Hallöchen!
[email protected] writes:
> [...]
>
>> [...] Is the non-dependency on GLib's include files really so
>> important?
>
> I think it is a good idea to avoid this dependency.
Then, merging is difficult. I succeeded by the following:
1. Move the method bodies of lfExtDatabase out of lensfunprv.h to
database.cpp. I see no point in them being in lensfunprv.h in
the first place.
2. Add #include <glib.h> to various cpp files which previously got
it through lensfunprv.h.
3. Prepend the following to lensfun.h.in:
#ifndef __G_TYPES_H__
typedef char gchar;
typedef int gint;
typedef unsigned int guint;
typedef void* gpointer;
typedef const void *gconstpointer;
typedef unsigned int gsize;
typedef void (*GDestroyNotify) (gpointer data);
struct GPtrArray
{
gpointer *pdata;
guint len;
};
typedef gint (*GCompareFunc) (gconstpointer a, gconstpointer b);
GPtrArray* g_ptr_array_new (void);
struct GString
{
gchar *str;
gsize len;
gsize allocated_len;
};
#endif
4. Move lensfunprv.h into lensfun.h.in.
(3) is the ugly part of course.
>> 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,
Okay, I stay tuned.
Tschö,
Torsten.
--
Torsten Bronger Jabber ID: [email protected]
or http://bronger-jmp.appspot.com
------------------------------------------------------------------------------
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
