Hi,
The merge of the latest features from Liam into the Experimental branch raises the question of best practices for merging. In particular, I am not sure that merging multiple features at one time is a good idea. Other projects seem to divide commits into the smallest possible pieces. I think that this is good practice as it better documents what is being committed, allows for better reviews, and makes it easier to cherry-pick from one branch to another.
In particular, the commit message:
"Merge in ponyscape features by Theo and worked on my LiamW"
doesn't give any information about what are the purpose of the changes. It takes reading the code and/or searching the web to find this information. (See: http://liamwhite1.deviantart.com/journal/ ) As far as I can tell, the commit:
1. Adds the new LPE's:
* Attach path * Bounding box * Ellipse * Fill between many * Fill between strokes * Joint type * Taper stroke
2. A tagging system.
3. Objects dialog.
4. Multi-path manipulator
5. Filter effects chooser.
I think these should have been committed separately with a line or two explaining what each does. Also, each LPE should be a separate commit.
As an additional question: is the experimental branch going to become 0.92? I think there have been too many changes to the experimental branch to try to merge it with trunk.
Tav