c++ bindings



Hi Chris!

Well, the example.cpp file looks fine to me, the only thing is that you
don't need to initialize Bonobo in this example. so instead of,

	/* initialize CORBA/bonobo */
	CORBA_exception_init(&ev);
	orb = gnome_CORBA_init("gda-rolodex-app", VERSION, &argc, argv, 0,
&ev);
	CORBA_exception_free(&ev);

>>>>>>>>>>>>>> remove the line below
	if (bonobo_init(orb, NULL, NULL) == FALSE) g_error(_("Could not
initialize Bonobo\n"));


I'll have a look at the other sources very soon to see if it's ok, but
as far as I've seen, it looks ok. Oh! yes, now I remember I saw you're
also adding the _get_type functions. DON'T, since these functions are
related to the GTK+ type system and are never called directly, and I
think there won't be any useful use for it from C++ apps, so remove
these functions.

And now I'm realizing that I don't know if it's a good idea to wrap the
GDA_Value CORBA struct. I have to check if it's used at all in any
client app. And if it's used, the gda-value.[ch] should be added to the
libgda-client to wrap it in the C libs. This struct is now partially
wrapped by the Gda_Field struct.

Cheers






[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]