On 4 June 2012 20:35, Jon Nordby <[email protected]> wrote: > On 4 June 2012 18:50, Jon Nordby <[email protected]> wrote: >> On 2 June 2012 17:21, Martin Renold <[email protected]> wrote: >>> hi Jon >>> >>> Nice progress with brushlib! >>> No objection to convert MyPaint's .myb to json as planned. >> >> Thanks! >> I have pushed a branch "json" to the mainline repository now. This has >> the code for supporting the new format in MyPaint, migrates the stock >> brushes to this format and implements loading in libmypaint. >> >>> About dict vs list: IMO if we have a high-level structure like json, only a >>> dict makes sense. The index is only required internally, for fast C array >>> access. >> >> I agree. I will make this change in above mentioned branch shortly. > > I've now made the change and updated the branch. I would like to push > this branch (without the two TEMP commits) now to complete the > migration to new format. I have tested it a bit, but I would like that > it gets some testing by others before I push it to master.
I've rebased, retested and pushed this change now. So in master the brushes are now in JSON format. Two steps more are required to complete the migration fully 1. Implement save as JSON in brushlib (libmypaint) 2. Use the brushlib for brush settings serialization. This latter is mainly to use libmypaint in the same way as other consumers would, to prove and test that it works as intended and to not special-case MyPaint. -- Jon Nordby - www.jonnor.com _______________________________________________ Mypaint-discuss mailing list [email protected] https://mail.gna.org/listinfo/mypaint-discuss
