Emmanuel Rodriguez wrote:
I finally released to CPAN the first release of libchamplain's bindings (Download here http://search.cpan.org/perldoc?Champlain). Libchamplain consists of a Clutter actor (a canvas) that supports numerous free map sources such as OpenStreetMap <http://www.openstreetmap.org/>, OpenArialMap<http://www.openaerialmap.org/>and Maps for free <http://www.maps-for-free.com/>.
A few other things that caught my eye:• gperl_croak_gerror ignores its first argument (for historical reasons), so the convention is to simply pass in NULL.
• You have no tests. libchamplain's API appears to be very well designed, so the wrappers are mostly trivial. But still, writing good bindings "blindly", i.e. without tests, is error-prone.