Thanks for the help. I will try to fix the issues as soon as possible. On Mon, Mar 16, 2009 at 9:58 AM, Torsten Schoenfeld <kaffeeti...@gmx.de>wrote:
> 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. I'm taking note. > > > • 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. Don't worry I will add unit test coverage in the next days. I'm also missing a lot of documentation and some examples. I really wanted to make the code public as we where lagging behind python and C# :) Once more, thanks for the help. -- Emmanuel Rodriguez
_______________________________________________ gtk-perl-list mailing list gtk-perl-list@gnome.org http://mail.gnome.org/mailman/listinfo/gtk-perl-list