[GRASS-dev] Re: [GRASS-SVN] r42587 - in grass-addons/gui/wxpython: . wx.path

Martin Landa landa.martin at gmail.com
Sat Jun 19 08:01:29 EDT 2010


Hi,

2010/6/19  <svn_grass at osgeo.org>:
> Author: rashadkm
> Date: 2010-06-19 11:55:19 +0000 (Sat, 19 Jun 2010)
> New Revision: 42587
>
> Added:
>   grass-addons/gui/wxpython/wx.path/
>   grass-addons/gui/wxpython/wx.path/mapdisplay_window.py
>   grass-addons/gui/wxpython/wx.path/wx.path.py
> Log:
> created an interface for v.net.path

please don't use full copies of wxGUI classes (similar for data
catalog) when you are implementing new features. Better is to define
subclasses based on the original wxGUI classes and to provided patches
via trac for superclasses from trunk if necessary. Otherwise the code
is unreadable, hard to maintain and timeconsuming task to merge it
with trunk.

Martin

-- 
Martin Landa <landa.martin gmail.com> * http://gama.fsv.cvut.cz/~landa


More information about the grass-dev mailing list