[GRASS-dev] [GRASS GIS] #2031: G_legal_filename() cleanup patch for GRASS 6

[GRASS GIS]

#2031: G_legal_filename() cleanup patch for GRASS 6
----------------------+-----------------------------------------------------
  Reporter:  neteler  |       Owner:  grass-dev@…              
      Type:  defect   |      Status:  closed                   
  Priority:  normal   |   Milestone:  6.4.4                    
 Component:  LibGIS   |     Version:  svn-releasebranch64      
Resolution:  fixed    |    Keywords:                           
  Platform:  All      |         Cpu:  All                      
----------------------+-----------------------------------------------------

Comment(by neteler):

 Replying to [comment:4 hamish]:
 > cell/mapname is by definition a filesystem name, so G_legal_filename()
 seems
 > perfectly legitimate safety check.

 See the ML discussion for this.

 > Also there is refactoring of v.random.surface and

 You mean r.random.surface? That's part of the change r32820.

 > r.surf.fractal, was that intended to be in this commit?

 That's a trivial change and part of the original change.

 > I suggest to revert -- and not commit large patches to relbr64 directly!
 > Especially when we are trying to get a release out the door!!

 I oppose to revert and the change r32820 has been tested for 6 years in
 trunk.
 There is no release soon, we do not even have an RC1 for 6.4 as no
 feedback
 there.

 thanks,
 Markus

-- 
Ticket URL: <http://trac.osgeo.org/grass/ticket/2031#comment:5>
GRASS GIS <http://grass.osgeo.org>