Merge lp:~gue5t/midori/gir-2 into lp:midori

Proposed by gue5t gue5t
Status: Merged
Approved by: Cris Dywan
Approved revision: 6702
Merged at revision: 6755
Proposed branch: lp:~gue5t/midori/gir-2
Merge into: lp:midori
Diff against target: 178 lines (+25/-14)
8 files modified
katze/katze-arrayaction.c (+9/-1)
katze/katze-cellrenderercomboboxtext.c (+1/-1)
katze/katze-item.c (+2/-2)
katze/katze-preferences.c (+4/-1)
katze/katze-utils.c (+4/-4)
midori/midori-bookmarks-db.c (+2/-2)
midori/midori-locationaction.c (+2/-2)
midori/midori-view.c (+1/-1)
To merge this branch: bzr merge lp:~gue5t/midori/gir-2
Reviewer Review Type Date Requested Status
Cris Dywan Approve
Review via email: mp+220146@code.launchpad.net

Commit message

Fix some symbol names and transfer annotations in doc comments

Description of the change

This fixes several wrong or missing parts of our documentation which the g-ir-scanner didn't like. No changes to executable code.

To post a comment you must log in.
Revision history for this message
Cris Dywan (kalikiana) wrote :

gir doesn't seem too happy with this

katze/katze-item.c:278: Warning: Katze: katze_item_new: return value: Invalid non-constant of bare structure or union; register as boxed type (skip)

review: Needs Fixing
Revision history for this message
gue5t gue5t (gue5t) wrote :

> gir doesn't seem too happy with this
>
> katze/katze-item.c:278: Warning: Katze: katze_item_new: return value: Invalid
> non-constant of bare structure or union; register as boxed type (skip)

I asked in #introspection and apparently g-ir-scanner does not intuit which types are GObject classes from source code (G_DEFINE_TYPE) but requires to build a binary and run it to dump introspection data, so in our current build setup (running g-ir-scanner on source rather than an introspection dump) it doesn't realize that KatzeItem is a GObject class. This isn't, as far as I know, a sign that our annotations are incorrect. Ultimately we need to set up the buildsystem stuff to produce the introspection binary, but in the meantime I'd like to get this merged just so we have more annotations.

Revision history for this message
Cris Dywan (kalikiana) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'katze/katze-arrayaction.c'
--- katze/katze-arrayaction.c 2013-12-26 17:41:17 +0000
+++ katze/katze-arrayaction.c 2014-05-20 02:22:37 +0000
@@ -695,7 +695,7 @@
695 * Note that the label is reasonably ellipsized for you,695 * Note that the label is reasonably ellipsized for you,
696 * much like katze_image_menu_item_new_ellipsized().696 * much like katze_image_menu_item_new_ellipsized().
697 *697 *
698 * Return value: a new tool item698 * Return value: (transfer full): a new tool item
699 **/699 **/
700GtkToolItem*700GtkToolItem*
701katze_array_action_create_tool_item_for (KatzeArrayAction* array_action,701katze_array_action_create_tool_item_for (KatzeArrayAction* array_action,
@@ -802,6 +802,14 @@
802 (action, proxy);802 (action, proxy);
803}803}
804804
805/**
806 * katze_array_action_get_array:
807 * @array_action: a #KatzeArrayAction
808 *
809 * Obtain the array which backs the array action.
810 *
811 * Return value: (transfer none): the KatzeArray used by @array_action
812 **/
805KatzeArray*813KatzeArray*
806katze_array_action_get_array (KatzeArrayAction* array_action)814katze_array_action_get_array (KatzeArrayAction* array_action)
807{815{
808816
=== modified file 'katze/katze-cellrenderercomboboxtext.c'
--- katze/katze-cellrenderercomboboxtext.c 2013-10-25 21:49:56 +0000
+++ katze/katze-cellrenderercomboboxtext.c 2014-05-20 02:22:37 +0000
@@ -334,7 +334,7 @@
334 * value in the model, thus rendering a different string in each row334 * value in the model, thus rendering a different string in each row
335 * of the #GtkTreeView335 * of the #GtkTreeView
336 *336 *
337 * Return value: the new cell renderer337 * Return value: (transfer full): the new cell renderer
338 **/338 **/
339GtkCellRenderer *339GtkCellRenderer *
340katze_cell_renderer_combobox_text_new (void)340katze_cell_renderer_combobox_text_new (void)
341341
=== modified file 'katze/katze-item.c'
--- katze/katze-item.c 2014-03-02 20:21:29 +0000
+++ katze/katze-item.c 2014-05-20 02:22:37 +0000
@@ -275,7 +275,7 @@
275 *275 *
276 * Creates a new #KatzeItem.276 * Creates a new #KatzeItem.
277 *277 *
278 * Return value: a new #KatzeItem278 * Return value: (transfer full): a new #KatzeItem
279 **/279 **/
280KatzeItem*280KatzeItem*
281katze_item_new (void)281katze_item_new (void)
@@ -825,7 +825,7 @@
825 *825 *
826 * Since 0.4.3 meta data is copied.826 * Since 0.4.3 meta data is copied.
827 *827 *
828 * Return value: a new #KatzeItem828 * Return value: (transfer full): a new #KatzeItem
829 *829 *
830 * Since: 0.1.3830 * Since: 0.1.3
831 **/831 **/
832832
=== modified file 'katze/katze-preferences.c'
--- katze/katze-preferences.c 2014-03-14 19:03:51 +0000
+++ katze/katze-preferences.c 2014-05-20 02:22:37 +0000
@@ -103,7 +103,7 @@
103 *103 *
104 * Creates a new preferences dialog.104 * Creates a new preferences dialog.
105 *105 *
106 * Return value: a new #KatzePreferences106 * Return value: (transfer full): a new #KatzePreferences
107 *107 *
108 * Since: 0.2.1108 * Since: 0.2.1
109 **/109 **/
@@ -225,6 +225,9 @@
225 *225 *
226 * Adds a new category with the specified label to the dialog.226 * Adds a new category with the specified label to the dialog.
227 *227 *
228 * Return value: (transfer none): a new #GtkBox in the preferences widget to
229 * hold the category's widgets
230 *
228 * Since: 0.2.1231 * Since: 0.2.1
229 *232 *
230 * Since 0.3.4 a #GtkBox is returned that can be packed into.233 * Since 0.3.4 a #GtkBox is returned that can be packed into.
231234
=== modified file 'katze/katze-utils.c'
--- katze/katze-utils.c 2014-02-23 14:51:05 +0000
+++ katze/katze-utils.c 2014-05-20 02:22:37 +0000
@@ -316,7 +316,7 @@
316 * Since 0.2.1 the proxy may contain a label if the platform316 * Since 0.2.1 the proxy may contain a label if the platform
317 * has according widgets.317 * has according widgets.
318 *318 *
319 * Return value: a new widget319 * Return value: (transfer full): a new widget
320 **/320 **/
321GtkWidget*321GtkWidget*
322katze_property_proxy (gpointer object,322katze_property_proxy (gpointer object,
@@ -744,7 +744,7 @@
744 * Creates an image menu item where the label is744 * Creates an image menu item where the label is
745 * reasonably ellipsized for you.745 * reasonably ellipsized for you.
746 *746 *
747 * Return value: a new label widget747 * Return value: (transfer full): a new label widget
748 **/748 **/
749GtkWidget*749GtkWidget*
750katze_image_menu_item_new_ellipsized (const gchar* label)750katze_image_menu_item_new_ellipsized (const gchar* label)
@@ -990,7 +990,7 @@
990 *990 *
991 * Retrieve the object value of the specified property.991 * Retrieve the object value of the specified property.
992 *992 *
993 * Return value: an object993 * Return value: (transfer none): an object
994 **/994 **/
995gpointer995gpointer
996katze_object_get_object (gpointer object,996katze_object_get_object (gpointer object,
@@ -1092,7 +1092,7 @@
1092 * If @other_widget is given, it will become insensitive if1092 * If @other_widget is given, it will become insensitive if
1093 * the input is not a valid URI.1093 * the input is not a valid URI.
1094 *1094 *
1095 * Returns: a #GtkEntry1095 * Returns: (transfer full): a #GtkEntry
1096 *1096 *
1097 * Since: 0.3.61097 * Since: 0.3.6
1098 */1098 */
10991099
=== modified file 'midori/midori-bookmarks-db.c'
--- midori/midori-bookmarks-db.c 2014-03-16 20:25:59 +0000
+++ midori/midori-bookmarks-db.c 2014-05-20 02:22:37 +0000
@@ -739,11 +739,11 @@
739739
740/**740/**
741 * midori_bookmarks_db_import_array:741 * midori_bookmarks_db_import_array:
742 * @array: the main bookmark array742 * @bookmarks: the main bookmark array
743 * @array: #KatzeArray containing the items to import743 * @array: #KatzeArray containing the items to import
744 * @parentid: the id of folder744 * @parentid: the id of folder
745 *745 *
746 * Imports the items of @array as childs of the folder746 * Imports the items of @array as children of the folder
747 * identfied by @parentid.747 * identfied by @parentid.
748 *748 *
749 * Since: 0.5.2749 * Since: 0.5.2
750750
=== modified file 'midori/midori-locationaction.c'
--- midori/midori-locationaction.c 2014-04-07 21:53:49 +0000
+++ midori/midori-locationaction.c 2014-05-20 02:22:37 +0000
@@ -1707,7 +1707,7 @@
1707}1707}
17081708
1709/**1709/**
1710 * midori_location_action_set_security_hint:1710 * midori_location_action_set_secondary_icon:
1711 * @location_action: a #MidoriLocationAction1711 * @location_action: a #MidoriLocationAction
1712 * @icon: a stock ID, or an icon name1712 * @icon: a stock ID, or an icon name
1713 *1713 *
@@ -1736,7 +1736,7 @@
1736}1736}
17371737
1738/**1738/**
1739 * midori_location_action_primary_icon:1739 * midori_location_set_action_primary_icon:
1740 * @location_action: a #MidoriLocationAction1740 * @location_action: a #MidoriLocationAction
1741 * @icon: a list of icon names, preferred icon first1741 * @icon: a list of icon names, preferred icon first
1742 * @tooltip: The tooltip to show1742 * @tooltip: The tooltip to show
17431743
=== modified file 'midori/midori-view.c'
--- midori/midori-view.c 2014-04-28 22:01:42 +0000
+++ midori/midori-view.c 2014-05-20 02:22:37 +0000
@@ -3232,7 +3232,7 @@
3232}3232}
32333233
3234/**3234/**
3235 * midori_view_new_with_item:3235 * midori_view_new_from_view:
3236 * @view: a predating, related #MidoriView, or %NULL3236 * @view: a predating, related #MidoriView, or %NULL
3237 * @item: a #KatzeItem, or %NULL3237 * @item: a #KatzeItem, or %NULL
3238 * @settings: a #MidoriWebSettings, or %NULL3238 * @settings: a #MidoriWebSettings, or %NULL

Subscribers

People subscribed via source and target branches

to all changes: