Re: gtkmm API freeze on July 30th



> > Documentation:
> > - AboutDialog::get_program_name() and others: refers to the dialog
> > 'owning' the string and warning the user not to modify it

I fixed the documentation for the methods where I found this.

> > - Action::create_menu(): does the user need to free the menu?

It's a widget, so it should be destroyed or added to a container that
owns it, I guess. It's the same as for the other ::create_*() functions
there.

> > - EntryCompletion::get_completion_prefix(): "remove 'or 0 if...'"

Fixed.

> > - IconTheme::choose_icon(): "icon_names 0-terminated array of icon
> > names to lookup.", also "Returns: A Gtk::IconInfo structure containing
> > information about the icon, or 0 if the icon wasn't found. Free with
> > gtk_icon_info_free()"

Fixed.

> > - IconTheme::list_contexts(): I have no idea what 'contexts' are, it'd
> > be nice to explain it somewhere.

Yeah, I filed a bug:
http://bugzilla.gnome.org/show_bug.cgi?id=461249

>   Also: "Returns: A G::List list
> > holding the names of all the contexts in the theme. You must first
> > free each element in the list with Glib::free(), then free the list
> > itself with Glib::list_free()"

Fixed.

> > IconView::set_tooltip_cell() & set_tooltip_item(): "See also
> > gtk_tooltip_set_tip_area()."

Fixed.

> > - PageSetup::save_to_file(), PrintSettings::to_file(): has
> > documentation for non-existant 'error' parameter

Fixed.

> > - Printer::request_details(): documentation refers to the GTK+ signal
> > name "Gtk::Printer::details-acquired"

Fixed.

> > - CellLayout::get_cells() contains the following text in
> > documentation: "The list, but not the renderers has been newly
> > allocated and should be freed with Glib::list_free() when no longer
> > needed"

Fixed.

> > - Widget::error_bell(): refers to Gtk::Settings:gtk-error-bell GTK+
> > property, which isn't wrapped in gtkmm.  Documentation also refers to
> > the C function gdk_window_beep()

Fixed. And I added several new Gtk::Settings properties.

> > - Widget::get_tooltip_markup(), Widget::get_tooltip_text(): incorrect
> > text "The tooltip text, or 0. You should free the returned string with
> > Glib::free() when done."

Fixed.

I do wonder why the C gtk_* functions are not being converted
automatically to C++ method names, as I think usually happens.
 
-- 
Murray Cumming
murrayc murrayc com
www.murrayc.com
www.openismus.com




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