Am 02.03.2015 um 21:44 schrieb Sebastian Kraft: > > One thing I have never really understood is why lensfun is C++ but > relies on glib (C libray) for array and list handling... Can you imagine > any good reason for that? If no, maybe it is best to replace GPtrArry by > a std::vector or something similar. >
I had a closer look at the PtrArray usage. It would not make sense to replace it IMO. The whole library is a mix of C and C++ calls, therefore starting to replace it in one part means to replace it nearly everywhere. In the end it is a complete rewrite. So lets better focus on the main issue, the syntax of lfModifier::Create/Initialize. Therefore, it should not be necessary to change the division into lfModifier and lfExtModifier in my opinion. It is not the most intuitive solution, but it is usually not visible to the programmer using the API. 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
