Re: [GRASS-dev] [GRASS-SVN] r64850 - in grass-addons/grass7/gui/wxpython: . wx.mwprecip

2015-03-14 Thread Glynn Clements
Vaclav Petras wrote: > This should be one big try-except. Also, `except:` is a bad practice, this > will catch everything including perhaps even SyntaxError. You should > specify the error/exception type. If you want to catch all errors, use "except StandardError:". That includes anything which

Re: [GRASS-dev] [GRASS-SVN] r64850 - in grass-addons/grass7/gui/wxpython: . wx.mwprecip

2015-03-13 Thread Vaclav Petras
Well, this is interesting, I'm looking forward how this will develop. Here is code review of some minor issues. On Fri, Mar 13, 2015 at 1:24 PM, wrote: > > +def loadSettings(self, sett=None): > +if sett: > +self.settings = sett > + > +try: > +self.data