Exiv2

Patch is working. This is the output of the XMP dump of the provided test file:

Xmp.mwg-kw.Keywords                                 XmpText      0 type="Struct" 
Xmp.mwg-kw.Keywords/mwg-kw:Hierarchy                XmpText      0 type="Bag" 
Xmp.mwg-kw.Keywords/mwg-kw:Hierarchy[1]             XmpText      0 type="Struct" 
Xmp.mwg-kw.Keywords/mwg-kw:Hierarchy[1]/mwg-kw:Keyword XmpText      3 cat
Xmp.mwg-kw.Keywords/mwg-kw:Hierarchy[1]/mwg-kw:Children XmpText      0 type="Bag" 
Xmp.mwg-kw.Keywords/mwg-kw:Hierarchy[1]/mwg-kw:Children[1] XmpText      0 type="Struct" 
Xmp.mwg-kw.Keywords/mwg-kw:Hierarchy[1]/mwg-kw:Children[1]/mwg-kw:Keyword XmpText      7 Siamese
Xmp.mwg-kw.Keywords/mwg-kw:Hierarchy[2]             XmpText      0 type="Struct" 
Xmp.mwg-kw.Keywords/mwg-kw:Hierarchy[2]/mwg-kw:Keyword XmpText      7 Felidae
Xmp.mwg-kw.Keywords/mwg-kw:Hierarchy[2]/mwg-kw:Children XmpText      0 type="Bag" 
Xmp.mwg-kw.Keywords/mwg-kw:Hierarchy[2]/mwg-kw:Children[1] XmpText      0 type="Struct" 
Xmp.mwg-kw.Keywords/mwg-kw:Hierarchy[2]/mwg-kw:Children[1]/mwg-kw:Keyword XmpText      6 Jaguar

It messes up the layout but I guess this is general problem with long identifiers of hierarchical data.

Patch submitted. r3633. Thanks to Alan for reporting and providing the patch. Thanks to Thomas for validating the fix.

Thanks for looking at this Thomas, it has been on my TODO list for more than a week. (#1042/#1043 have been time-consuming). I think we should about add the test file to the test harness.

Alan: is there any reason (example copyright) to prevent us from copying your test file (mwg-kw.test.jpg) into our test suite? It will probably be renamed exiv2-bug1040.jpg to fit the name pattern of the test files.

Very cool! Thank you Thomas and Robin!

Yes, go ahead and include the test image into the test suite. It can be considered to be in the public domain. Next time I'll be sure to include a Creative Commons statement to that effect.

Submitted: r3642 and r3543

Alan:
I've added a test to make ensure the output of the command $ exiv2 -px mwg-kw.test.jpg is always the same as Thomas reports in Item3 above.

I'm a little confused by the discussion about the spelling of hierarchy. I've also added a parallel test on the file you provided for #784

$ exiv2 -px exiv2.lr.jpg # renamed to exiv2-bug784.jpg
Xmp.dc.format                                XmpText    10  image/jpeg
Xmp.dc.rights                                LangAlt     1  lang="x-default" Public Domain. Do whatever you like with this image
Xmp.dc.subject                               XmpBag      6  1st, 2nd, next1, next2, root, root0
Xmp.xmpMM.DocumentID                         XmpText    32  004D48F936062EF5085A81BF96D4C494
Xmp.xmpMM.OriginalDocumentID                 XmpText    32  004D48F936062EF5085A81BF96D4C494
Xmp.xmpMM.InstanceID                         XmpText    44  xmp.iid:f74f0d02-e921-134e-8107-1dda17aad853
Xmp.xmpMM.History                            XmpText     0  type="Seq" 
Xmp.xmpMM.History[1]                         XmpText     0  type="Struct" 
Xmp.xmpMM.History[1]/stEvt:action            XmpText     5  saved
Xmp.xmpMM.History[1]/stEvt:instanceID        XmpText    44  xmp.iid:f74f0d02-e921-134e-8107-1dda17aad853
Xmp.xmpMM.History[1]/stEvt:when              XmpText    25  2015-03-24T20:35:55-05:00
Xmp.xmpMM.History[1]/stEvt:softwareAgent     XmpText    39  Adobe Photoshop Lightroom 4.4 (Windows)
Xmp.xmpMM.History[1]/stEvt:changed           XmpText     9  /metadata
Xmp.xmp.MetadataDate                         XmpText    25  2015-03-24T20:35:55-05:00
Xmp.crs.RawFileName                          XmpText    12  exiv2.lr.jpg
Xmp.lr.hierarchicalSubject                   XmpBag      2  root0|next1|next2, root|1st|2nd
894 rmills@rmillsmbp:~/gnu/exiv2/trunk/test $ 

I hope these are the correct tests and we can mark both issues #1040 and #784 as resolved.