#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> _______________________________________________ grass-dev mailing list grass-dev@lists.osgeo.org http://lists.osgeo.org/mailman/listinfo/grass-dev