[GRASS-dev] Re: [GRASS GIS] #676: r.watershed: different output map options do not allow for fully qualified map names

[GRASS GIS]

#676: r.watershed: different output map options do not allow for fully qualified
map names
-----------------------+----------------------------------------------------
  Reporter:  mlennert  |       Owner:  grass-dev at lists.osgeo.org
      Type:  defect    |      Status:  new                      
  Priority:  normal    |   Milestone:  6.4.0                    
 Component:  Raster    |     Version:  6.4.0 RCs                
Resolution:            |    Keywords:  r.watershed              
  Platform:  Linux     |         Cpu:  Unspecified              
-----------------------+----------------------------------------------------
Comment (by glynn):

 Replying to [comment:8 mlennert]:

 > > > And if yes, should they use G_legal_filename().
 > >
 > > Modules shouldn't normally use this function. It's meant for use by
 low-level library functions.
 >
 > Can we just delete the check then ? In any case there will be an error
 at creation if the map name is illegal, or ?

 The library functions which create maps, support files, etc should be
 calling G_legal_filename() as needed.

 > > If modules are passing map names to it, the modules should be fixed.
 >
 > Grepping for G_legal_filename in grass6 (trunk and 64 releasebranch)
 throws up lots of occurences which at first sight seem to pass map names.
 None left in grass7, so I guess you have already cleaned up ?
 >
 > Should this also be done in 6.* ?

 Ideally, yes.

-- 
Ticket URL: <https://trac.osgeo.org/grass/ticket/676#comment:11>
GRASS GIS <http://grass.osgeo.org>