[poppler] poppler Digest, Vol 64, Issue 9

[srinivas adicherla]

Hi,

By considering the suggestions from Carlos & Pino Toscano, I made some
changes to the previous patch.

          As Carlos mentioned, I added PDFDoc:getID() in the core and from
the glib bindings took the structure for PopplerDocumentId and returning it.

In addition to that I added two new properties in the PopplerDocument
namely, "permanent-id" , "update-id".

           As Carlos mentioned PDFDoc::saveAs() will not generate ID unless
we modify and save the file. If we don't modify and save it, it is nothing
but save a copy.So, I added PDFDoc::setID() in the core.

Please find two attachments with this mail containing the above said
changes.




On Sat, Jun 12, 2010 at 12:30 AM, <poppler-request at lists.freedesktop.org>wrote:

> Send poppler mailing list submissions to
>        poppler at lists.freedesktop.org
>
> To subscribe or unsubscribe via the World Wide Web, visit
>        http://lists.freedesktop.org/mailman/listinfo/poppler
> or, via email, send a message with subject or body 'help' to
>        poppler-request at lists.freedesktop.org
>
> You can reach the person managing the list at
>        poppler-owner at lists.freedesktop.org
>
> When replying, please edit your Subject line so it is more specific
> than "Re: Contents of poppler digest..."
>
>
> Today's Topics:
>
>   1. Re: Patch to Get/Set PDF ID in the trailer dictionary,    get
>      page labels . (Carlos Garcia Campos)
>
>
> ----------------------------------------------------------------------
>
> Message: 1
> Date: Fri, 11 Jun 2010 12:31:47 +0200
> From: Carlos Garcia Campos <carlosgc at gnome.org>
> Subject: Re: [poppler] Patch to Get/Set PDF ID in the trailer
>        dictionary,     get page labels .
> To: srinivas adicherla <srinivas.adicherla at gmail.com>
> Cc: poppler <poppler at lists.freedesktop.org>
> Message-ID: <1276251137-sup-4795 at charmaleon>
> Content-Type: text/plain; charset="utf8"
>
> Excerpts from srinivas adicherla's message of jue jun 10 12:13:25 +0200
> 2010:
> > Hi,
>
> Hi srinivas,
>
> >   This Patch does the following:
>
> Thanks for the patch, I some comments:
>
> >   1) gets and set the pdf IDs in the trailer dictionary.
>
> The code to get ID could be moved to the core, so that it can be used
> by other frontends, someting like PDFDoc::getID(). In the glib
> frontend, instead of returning an array we could use a struct so that
> we can also give names to the fields, since the first id is the
> permanent id and the second one is the id that changes when the
> document is updated. Something like
>
> typedef struct _PopplerDocumentId {
>        gchar permanent_id[32];
>        gchar update_id[32];
> } PopplerDocumentId;
>
> The method generate_and_set_id() shouldn't be necessary since
> PDFDoc::saveAS() already generates a new id.
>
> >   2) get all page labels & get page labels by index.
>
> We already have labels in the glib frontend, it's property of
> PopplerPage that can be obtained with
>
> g_object_get (page, "label", &label, NULL);
>
> I don't think we need that API in PopplerDocument, but we should
> probably expose it in the PopplerPage API to make it easier to
> find.
>
> >    Kindly let me know if You commit this patch.
> >
> >
> > Thanks
> > A Srinivas
> --
> Carlos Garcia Campos
> PGP key: http://pgp.mit.edu:11371/pks/lookup?op=get&search=0x523E6462
> -------------- next part --------------
> A non-text attachment was scrubbed...
> Name: signature.asc
> Type: application/pgp-signature
> Size: 198 bytes
> Desc: not available
> URL: <
> http://lists.freedesktop.org/archives/poppler/attachments/20100611/3ab6035b/attachment-0001.pgp
> >
>
> ------------------------------
>
> _______________________________________________
> poppler mailing list
> poppler at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/poppler
>
>
> End of poppler Digest, Vol 64, Issue 9
> **************************************
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/poppler/attachments/20100615/b5654320/attachment-0001.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: srinivas_poppler.patch
Type: text/x-patch
Size: 6628 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/poppler/attachments/20100615/b5654320/attachment-0002.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: srinivas_poppler_api.patch
Type: text/x-patch
Size: 7315 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/poppler/attachments/20100615/b5654320/attachment-0003.bin>