[Potlatch-dev] Introducing Magic Roundabout tool

Steve Bennett stevagewp at gmail.com
Thu Feb 24 22:47:18 GMT 2011


On Fri, Feb 25, 2011 at 6:20 AM, Andy Allan <gravitystorm at gmail.com> wrote:
> Unfortunately it doesn't compile - I think you added a length()
> function to Elastic.as but haven't added it in! Also, DrawWay.as#328
> needs fixing since that doesn't compile either. Could you have a look
> as soon as you get the chance - I'll hold off from uploading my
> commits for the moment and just work on things locally.

Doh, I committed too far down the tree - forgot that Halcyon isn't in
the Potlatch2 dir. Done now.

> (Also, best to avoid committing code with compiler warnings - there's
> now some missing type declarations, class-scoped vars (I suspect you
> want these instance scoped), duplicate variable declarations - give me
> a shout if you're unsure or don't have time and I can sort them out
> for you)

Whoops, sorry about that. All fixed now.

I've discovered some slight flakiness, in that occasionally it
produces a bad result, and if you try to undo, you get an exception.
I'll need to understand a bit more how to use the undo system.

Steve



More information about the Potlatch-dev mailing list