> Therefore, lfModifier doesn't have a constructor. Instead, there > are "Create" and "Initialize" methods that must be called to have a > functioning object. "Create" sets the coordinate scale, and > "Initialize" adds the callbacks to the callback chain.
Yes, that should be changed! > > All of this is very ugly and error-prone (even the Lensfun > developers happened to tumble over this). Besides, the API is > difficult to extend. Is the non-dependency on GLib's include files > really so important? > I think it is a good idea to avoid this dependency. > I'd like to see Create and Initialize being merged into a > constructor proper. What's the best way in C++ to make the > parameter list extensible? For example some day, I may want to have > the image's timestamp in the parameter list. How can one design the > API to make such changes painless? > Have to take a closer look if this will really work in our context, but AFAIK you can have multiple constructors similar to function overloading... Sebastian ------------------------------------------------------------------------------ Dive into the World of Parallel Programming The Go Parallel Website, sponsored by Intel and developed in partnership with Slashdot Media, is your hub for all things parallel software development, from weekly thought leadership blogs to news, videos, case studies, tutorials and more. Take a look and join the conversation now. http://goparallel.sourceforge.net/ _______________________________________________ Lensfun-users mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/lensfun-users
