Re: Stable glibmm and gtkmm releases soon
- From: José Alburquerque <jaalburquerque gmail com>
- To: Murray Cumming <murrayc murrayc com>
- Cc: gtkmm-list <gtkmm-list gnome org>
- Subject: Re: Stable glibmm and gtkmm releases soon
- Date: Tue, 26 Mar 2013 11:17:31 -0400
On Tue, 2013-03-26 at 11:30 +0100, Murray Cumming wrote:
On Mon, 2013-03-25 at 22:16 -0400, José Alburquerque wrote:
3) From the GTlsFileDatabase docs [5] (see the description and the class
details), it seems that the interface derives from the GTlsDatabase
class. In glibmm it therefore presently derives from Gio::TlsDatabase.
I just want to make sure that the derivation of the interface in glibmm
should not be changed.
This is also odd. GTlsFileDatabase is an interface, that requires the
GTlsDatabase object type (not an interface).
You have GTlsDatabase deriving from, and implementing, TlsFileDatabase
(and Object), which seems wrong, so I have corrected that:
https://git.gnome.org/browse/glibmm/commit/?id=6b966ecb0435775f6a7f84dece93b779b7ee42d3
(Please check that you agree.)
I do. I think I got confused with the description of GTlsFileDatabase:
"GTlsFileDatabase is implemented by GTlsDatabase objects which load
their certificate information from a file. It is in interface which TLS
library specific subtypes implement."
I had not remembered that a type's prerequisite would list types that it
would derive from and not the other way around, sorry.
Actually, GTlsFileDatabase does not seem to be used in any public glib
or glibmm API (though TlsDatabase is). I suggest that we do not wrap it
just yet, to avoid us getting it wrong before we know how it should
really be used.
That's fine. I've removed TlsFileDatabase from the list of sources to
be built:
https://git.gnome.org/browse/glibmm/commit/?id=e28f5093dced2e8cafef5795c02657b7265b78ca
I guess it can be re-added (and maybe modified) in the future when it is
more clear how the class should be used.
--
José
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]