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
1=== modified file 'katze/katze-arrayaction.c'
2--- katze/katze-arrayaction.c 2013-12-26 17:41:17 +0000
3+++ katze/katze-arrayaction.c 2014-05-20 02:22:37 +0000
4@@ -695,7 +695,7 @@
5 * Note that the label is reasonably ellipsized for you,
6 * much like katze_image_menu_item_new_ellipsized().
7 *
8- * Return value: a new tool item
9+ * Return value: (transfer full): a new tool item
10 **/
11 GtkToolItem*
12 katze_array_action_create_tool_item_for (KatzeArrayAction* array_action,
13@@ -802,6 +802,14 @@
14 (action, proxy);
15 }
16
17+/**
18+ * katze_array_action_get_array:
19+ * @array_action: a #KatzeArrayAction
20+ *
21+ * Obtain the array which backs the array action.
22+ *
23+ * Return value: (transfer none): the KatzeArray used by @array_action
24+ **/
25 KatzeArray*
26 katze_array_action_get_array (KatzeArrayAction* array_action)
27 {
28
29=== modified file 'katze/katze-cellrenderercomboboxtext.c'
30--- katze/katze-cellrenderercomboboxtext.c 2013-10-25 21:49:56 +0000
31+++ katze/katze-cellrenderercomboboxtext.c 2014-05-20 02:22:37 +0000
32@@ -334,7 +334,7 @@
33 * value in the model, thus rendering a different string in each row
34 * of the #GtkTreeView
35 *
36- * Return value: the new cell renderer
37+ * Return value: (transfer full): the new cell renderer
38 **/
39 GtkCellRenderer *
40 katze_cell_renderer_combobox_text_new (void)
41
42=== modified file 'katze/katze-item.c'
43--- katze/katze-item.c 2014-03-02 20:21:29 +0000
44+++ katze/katze-item.c 2014-05-20 02:22:37 +0000
45@@ -275,7 +275,7 @@
46 *
47 * Creates a new #KatzeItem.
48 *
49- * Return value: a new #KatzeItem
50+ * Return value: (transfer full): a new #KatzeItem
51 **/
52 KatzeItem*
53 katze_item_new (void)
54@@ -825,7 +825,7 @@
55 *
56 * Since 0.4.3 meta data is copied.
57 *
58- * Return value: a new #KatzeItem
59+ * Return value: (transfer full): a new #KatzeItem
60 *
61 * Since: 0.1.3
62 **/
63
64=== modified file 'katze/katze-preferences.c'
65--- katze/katze-preferences.c 2014-03-14 19:03:51 +0000
66+++ katze/katze-preferences.c 2014-05-20 02:22:37 +0000
67@@ -103,7 +103,7 @@
68 *
69 * Creates a new preferences dialog.
70 *
71- * Return value: a new #KatzePreferences
72+ * Return value: (transfer full): a new #KatzePreferences
73 *
74 * Since: 0.2.1
75 **/
76@@ -225,6 +225,9 @@
77 *
78 * Adds a new category with the specified label to the dialog.
79 *
80+ * Return value: (transfer none): a new #GtkBox in the preferences widget to
81+ * hold the category's widgets
82+ *
83 * Since: 0.2.1
84 *
85 * Since 0.3.4 a #GtkBox is returned that can be packed into.
86
87=== modified file 'katze/katze-utils.c'
88--- katze/katze-utils.c 2014-02-23 14:51:05 +0000
89+++ katze/katze-utils.c 2014-05-20 02:22:37 +0000
90@@ -316,7 +316,7 @@
91 * Since 0.2.1 the proxy may contain a label if the platform
92 * has according widgets.
93 *
94- * Return value: a new widget
95+ * Return value: (transfer full): a new widget
96 **/
97 GtkWidget*
98 katze_property_proxy (gpointer object,
99@@ -744,7 +744,7 @@
100 * Creates an image menu item where the label is
101 * reasonably ellipsized for you.
102 *
103- * Return value: a new label widget
104+ * Return value: (transfer full): a new label widget
105 **/
106 GtkWidget*
107 katze_image_menu_item_new_ellipsized (const gchar* label)
108@@ -990,7 +990,7 @@
109 *
110 * Retrieve the object value of the specified property.
111 *
112- * Return value: an object
113+ * Return value: (transfer none): an object
114 **/
115 gpointer
116 katze_object_get_object (gpointer object,
117@@ -1092,7 +1092,7 @@
118 * If @other_widget is given, it will become insensitive if
119 * the input is not a valid URI.
120 *
121- * Returns: a #GtkEntry
122+ * Returns: (transfer full): a #GtkEntry
123 *
124 * Since: 0.3.6
125 */
126
127=== modified file 'midori/midori-bookmarks-db.c'
128--- midori/midori-bookmarks-db.c 2014-03-16 20:25:59 +0000
129+++ midori/midori-bookmarks-db.c 2014-05-20 02:22:37 +0000
130@@ -739,11 +739,11 @@
131
132 /**
133 * midori_bookmarks_db_import_array:
134- * @array: the main bookmark array
135+ * @bookmarks: the main bookmark array
136 * @array: #KatzeArray containing the items to import
137 * @parentid: the id of folder
138 *
139- * Imports the items of @array as childs of the folder
140+ * Imports the items of @array as children of the folder
141 * identfied by @parentid.
142 *
143 * Since: 0.5.2
144
145=== modified file 'midori/midori-locationaction.c'
146--- midori/midori-locationaction.c 2014-04-07 21:53:49 +0000
147+++ midori/midori-locationaction.c 2014-05-20 02:22:37 +0000
148@@ -1707,7 +1707,7 @@
149 }
150
151 /**
152- * midori_location_action_set_security_hint:
153+ * midori_location_action_set_secondary_icon:
154 * @location_action: a #MidoriLocationAction
155 * @icon: a stock ID, or an icon name
156 *
157@@ -1736,7 +1736,7 @@
158 }
159
160 /**
161- * midori_location_action_primary_icon:
162+ * midori_location_set_action_primary_icon:
163 * @location_action: a #MidoriLocationAction
164 * @icon: a list of icon names, preferred icon first
165 * @tooltip: The tooltip to show
166
167=== modified file 'midori/midori-view.c'
168--- midori/midori-view.c 2014-04-28 22:01:42 +0000
169+++ midori/midori-view.c 2014-05-20 02:22:37 +0000
170@@ -3232,7 +3232,7 @@
171 }
172
173 /**
174- * midori_view_new_with_item:
175+ * midori_view_new_from_view:
176 * @view: a predating, related #MidoriView, or %NULL
177 * @item: a #KatzeItem, or %NULL
178 * @settings: a #MidoriWebSettings, or %NULL

Subscribers

People subscribed via source and target branches

to all changes: