[Poppler-bugs] [Bug 44790] Various minor introspection improvements

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=44790

Evan Nemerson <evan at coeus-group.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #55588|0                           |1
        is obsolete|                            |

--- Comment #2 from Evan Nemerson <evan at coeus-group.com> 2012-01-15 11:06:18 PST ---
Created attachment 55609
  --> https://bugs.freedesktop.org/attachment.cgi?id=55609
glib: various minor introspection and documentation improvements

(In reply to comment #1)
> Comment on attachment 55588 [details] [review]
> glib: various minor introspection improvements
> 
> Review of attachment 55588 [details] [review]:
> -----------------------------------------------------------------
> 
> Thanks for the patch! just a few comments

Thanks for the quick review.

> ::: glib/poppler-attachment.h
> @@ +39,2 @@
> >   * @count: number of bytes in @buf.
> > + * @data: (closure closure): user data passed to poppler_attachment_save_to_callback()
> 
> This should be (closure), (closure closure) is the callback.

Oops.

> @@ +41,1 @@
> >   * @error: GError to set on error, or NULL
> 
> We should probably annotate this as (out), no?

No, that's not necessary. GObject introspection will do the right thing here.

> ::: glib/poppler-document.cc
> @@ +2497,4 @@
> >   * Returns the #PopplerFormField for the given @id. It must be freed with
> >   * g_object_unref()
> >   *
> > + * Return value: (transfer full): a new #PopplerFormField or NULL if
> 
> Use %NULL, please.

I guess there is no reason not to fix this while fixing something else on the
same line...

> ::: glib/poppler-media.h
> @@ +38,2 @@
> >   * @count: number of bytes in @buf.
> > + * @data: (closure closure): user data passed to poppler_media_save_to_callback()
> 
> Same here, this should be (closure).

Again, oops.

> @@ +40,1 @@
> >   * @error: GError to set on error, or NULL
> 
> And here we could annotate this as (out)

Yes, but it's unnecessary. However s/NULL/%NULL/. Actually, I'll just fix all
the results from grep -P '^ \*.+ NULL' *.{cc,h}

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.