search

google-code-export / gpick

Automatically exported from code.google.com/p/gpick
2 stars 0 forks source link

GPL import discards some of the first items #113

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
This is easily replicable with a small palette.
It appears to be related to having less than 3 digits for R G B values
(eg '40 255   0' rather than '040 255 000').

The first attachment is a simple 17-color palette which fails to load correctly 
(first loaded color is #5)
The attached 'fixed.gpl' file is loaded correctly. It's an edited version of 
the first attachment where every RGB intensity is 0-padded to 3 places.

Original issue reported on code.google.com by fintic...@gmail.com on 24 Dec 2013 at 12:43

Attachments:

GoogleCodeExporter commented 9 years ago 
I believe uiExport.cpp:125 is the culprit, since lines that start with a space 
contain colors, but this is not recognized by the conditional (only recognizes 
lines starting with digits)

Original comment by fintic...@gmail.com on 24 Dec 2013 at 12:57

GoogleCodeExporter commented 9 years ago 
Fixed in changeset 285:19d5ee0b1943. I'll switch to my old account to close 
this.

Original comment by fintic...@gmail.com on 24 Dec 2013 at 8:46

GoogleCodeExporter commented 9 years ago 
Closing.

Original comment by 00a...@gmail.com on 24 Dec 2013 at 8:48