| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
| |
Bug 779936
|
|
|
|
|
| |
It's not used by glibmm or gtkmm. It's unnecessary.
Use const_cast<>() directly instead.
|
|
|
|
| |
Bug 777791
|
|
|
|
| |
Bug 777791
|
|
|
|
|
|
|
| |
* gio/giomm.h: Add propertyaction.h
* gio/src/filelist.am: Add propertyaction.hg
* gio/src/propertyaction.[ccg|hg]: New files.
Required in the "Building Applications" chapter in the gtkmm tutorial.
|
| |
|
|
|
|
|
|
|
| |
Because the original code starts at 1, not 0, so this was not
equivalent.
This reverts commit b0a3e10d17f21254c15781bb7ff57736a9bea569.
|
| |
|
|
|
|
|
|
|
| |
Make it possible for named custom types to register additions to the
class init function and to register an instance init function.
An extra class init function is useful in Gtk::WidgetCustomDraw and
Gtk::WidgetCustomSnapshot. Bug 775348
|
|
|
|
|
|
| |
This matches e.g. simpleaction.hg and is what I originally intended.
https://bugzilla.gnome.org/show_bug.cgi?id=774444
|
|
|
|
|
|
| |
In case anyone is as easily confused as me…
https://bugzilla.gnome.org/show_bug.cgi?id=778575
|
|
|
|
|
|
|
| |
Clarify what is set and returned, and use lower case “nothing” to match
how that must be written in GVariant text format, à la GSettings, etc.
https://bugzilla.gnome.org/show_bug.cgi?id=778219
|
|
|
|
|
|
|
|
| |
It’s not exactly intuitive why the *_maybe() methods are kept in
ContainerBase, nor that Variant<VariantBase> inherits from the former
and is the way to get a maybe-typed Variant in glibmm. Let’s fix that!
https://bugzilla.gnome.org/show_bug.cgi?id=778219
|
|
|
|
|
|
|
| |
Donʼt use braces for single-line blocks, and do if the other side of an
if/else already did. Also, get rid of a couple of extraneous newlines.
https://bugzilla.gnome.org/show_bug.cgi?id=778219
|
| |
|
|
|
|
|
|
|
|
|
|
| |
If I’d realised there was a specific commit that protected this, I would
have reverted it, rather than doing it manually in my previous commit.
Completes reversion of commit b7431bca6ccbd9f918b5ef73bae65a1c7828e7e0 –
“Gio::SimpleAction: Make set_state() protected.”
https://bugzilla.gnome.org/show_bug.cgi?id=777953
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is needed to apply the requested state in a custom handler for
signal_change_state(). It was protected on an assumption that when the C
doc for g_simple_action_set_state() said “This should only be called by
the implementor of the action”, it meant the implementor of a subclass.
But the C docs for signal change-state are clear that it means the
implementor of the signal handler. Without such a user being able to
call set_state(), they can’t apply the new state, unless they subclass
SimpleAction, which is a lot to require for such a simple case as this.
https://bugzilla.gnome.org/show_bug.cgi?id=777953
|
|
|
|
| |
https://bugzilla.gnome.org/show_bug.cgi?id=777953
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Currently, it is included indirectly, via 3 other headers. This is
brittle. Without that, users of action.h would have to manually include
variant.h to use methods returning the forward-declared VariantBase,
which isn’t very helpful. Plus, the header uses types like Variant<>
templates and VariantContainerBase, which were never forward-declared.
Including it directly makes the dependency more obvious and less brittle
and spares us from having to write a bunch more forward-declarations.
https://bugzilla.gnome.org/show_bug.cgi?id=777953
|
| |
|
|
|
|
|
| |
Since we generate the .pdb files during the build, we should copy them to
make debugging glibmm on Windows easier.
|
|
|
|
|
| |
Because GNetworkMonitor requires GInitable.
Compare Gio::LoadableIcon which derives from Gio:Icon.
|
|
|
|
|
| |
The Variant<Glib::ustring> and Variant<std::string> specializations of
VariantBase::cast_dynamic<>() are no longer used by glibmm.
|
|
|
|
|
|
| |
* glib/glibmm/refptr.h: Remove clear().
* glib/glibmm/utility.h: Remove GLIBMM_INITIALIZE_STRUCT and ScopedPtr.
* glib/src/value_basictypes.[cc|h].m4: Remove Value<char>.
|
|
|
|
|
|
|
|
| |
* gio/src/gio_vfuncs.defs: Add GAsyncResult::is_tagged():
* gio/src/action.hg: get_state_hint_variant() returns VariantContainerBase.
* gio/src/actiongroup.hg: Remove the deprecated get_action_state_hint().
* gio/src/asyncresult.hg: Add is_tagged_vfunc().
* glib/src/checksum.hg: update(): The length parameter is a gssize.
|
| |
|
| |
|
|
|
|
|
|
|
| |
This puts several things in more logical places and adds a couple of
line breaks that make it easier to see the key sections in the header.
https://bugzilla.gnome.org/show_bug.cgi?id=774444
|
|
|
|
|
|
| |
It's used in add_action_with_bool() too.
https://bugzilla.gnome.org/show_bug.cgi?id=774444
|
|
|
|
|
|
| |
This brings it in line with the existing doc comment for ActivateSlot.
https://bugzilla.gnome.org/show_bug.cgi?id=774444
|
|
|
|
|
|
|
|
|
| |
It was not registering the parameter type for the action on create, so
when activating the action later and passing a parameter, an assertion
failed about the expected and passed parameter types/counts not matching
and the parameter was not usable.
https://bugzilla.gnome.org/show_bug.cgi?id=774444
|
|
|
|
|
| |
Remove Glib::VariantType::first() and next(). Replace them with
get_item_types(). Bug 775741
|
|
|
|
|
|
|
|
|
|
|
|
| |
Use Glib::ustring for values of file attributes of type
G_FILE_ATTRIBUTE_TYPE_STRING (UTF-8 encoded string). Changed methods:
* gio/src/file.hg: set_attribute_string()
* gio/src/fileinfo.hg: set/get_attribute_string(), set/get_display_name(),
set/get_edit_name(), set/get_content_type(), get_etag()
* gio/src/fileiostream.hg: get_etag()
* gio/src/fileoutputstream.hg: get_etag()
Bug 615950
|
|
|
|
|
|
|
| |
Store only a pointer to the private member data in the Dispatcher class.
The most important reason for the pimpl idiom (pointer to implementation)
in this case is that the deletion of the private data can be delayed until
it's safe to delete it. Bug 651942
|
|
|
|
|
|
|
|
|
| |
* glib/glibmm/init.[cc|h]: Add set/get_init_to_users_preferred_locale().
Let Glib::init() set the C andC++ global locale to the user's preferred locale,
or (if Glib::set_init_to_users_preferred_locale(false) has been called) set
the C++ locale to be equal to the C locale.
* glib/glibmm/ustring.h: Add to the documentation that Glib::init() sets the
global locale. Bug 661588
|
|
|
|
| |
Noticed by cppcheck.
|
| |
|
| |
|
|
|
|
| |
Noticed by cppcheck.
|
|
|
|
| |
Noticed by cppcheck.
|
|
|
|
| |
Noticed by cppcheck.
|
|
|
|
| |
Noticed by cppcheck.
|
|
|
|
| |
Noticed by cppcheck.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Rename activate_action_vfunc() to activate_action_full_vfunc() ad
rename change_action_state_vfunc() to change_action_state_full_vfunc(),
matching the underlying C vfuncs.
Now that this class derives from ActionGroup, this was causing warnings
such as this, at least with clang++:
/opt/gnome/include/giomm-2.52/giomm/remoteactiongroup.h:180:18: error: 'Gio::RemoteActionGroup::activate_action_vfunc' hides overloaded virtual
function [-Werror,-Woverloaded-virtual]
virtual void activate_action_vfunc(const Glib::ustring& action_name, const Glib::VariantBase& parameter, const Glib::VariantBase& platform_data);
^
/opt/gnome/include/giomm-2.52/giomm/actiongroup.h:574:18: note: hidden overloaded virtual function 'Gio::ActionGroup::activate_action_vfunc' declared
here: different number of parameters (2 vs 3)
virtual void activate_action_vfunc(const Glib::ustring& name, const Glib::VariantBase& parameter);
|
| |
|
|
|
|
|
| |
* glib/glibmm/dispatcher.cc: When a Windows HANDLE must be cast, cast it
to Glib::PollFD::fd_t instead of int. Bug 772074
|
|
|
|
|
|
|
|
| |
* glib/glibmm/main.[h|cc]: file descriptor to poll can be either of gint,
or gint64 type, depending on the platform. glibmm should follow this
rule as well.
https://bugzilla.gnome.org/show_bug.cgi?id=772074
|
|
|
|
| |
https://bugzilla.gnome.org/show_bug.cgi?id=774903
|
|
|
|
| |
Bug 775210
|