On Aug 2, 2008, at 3:05 PM, Nate Lowrie wrote:

> Added menu item type support to the mnxml processing.  Attribute in  
> mnxml is MenuType.  Defaults to "" when attribute is not present, so  
> the change is backwards compatible.

        Thanks for doing this, but how did you manage to commit and wipe out  
the change I posted this morning? Either you should have merged my  
changes first, or svn should not have allowed you to do this.

> +                             type = menu._extractKey(itmatts, "MenuType", "")


        Reserved words, such as 'type', should never be used as names.  
Typically I use 'typ' or, for this case, 'mtype' or 'menuType'.

        I'll fix both of these and re-commit. But I'm very interested in  
learning why svn didn't catch that your version was old before  
allowing you to commit.

-- Ed Leafe





_______________________________________________
Post Messages to: [email protected]
Subscription Maintenance: http://leafe.com/mailman/listinfo/dabo-dev
Searchable Archives: http://leafe.com/archives/search/dabo-dev
This message: http://leafe.com/archives/byMID/[EMAIL PROTECTED]

Reply via email to