[GRASS-dev] [GRASS-SVN] r54302 - grass/trunk/scripts/r.mask

[Pietro]

Hi,

On Sun, Dec 16, 2012 at 8:55 AM, Vaclav Petras <wenzeslaus at gmail.com> wrote:
> Hi,
>
> On 16 December 2012 01:11, Markus Metz <markus.metz.giswork at gmail.com> wrote:
>> On Sun, Dec 16, 2012 at 12:42 AM, Martin Landa <landa.martin at gmail.com> wrote:
>>> Hi Markus,
>>>
>>> 2012/12/16  <svn_grass at osgeo.org>:
>>>
>>>> r.mask: fix #1838
>>>
>>> great addition! Probably we could also merge `maskcats` with `cats`
>>> parameter. The same syntax could be used for raster and vectors.
>>>
>>> maskcats=" 1 2 3 thru 7 *"
>>>
>>> would become
>>>
>>> cats="1,2,3-7,*"
>>
>> This is not possible because r.reclass expects the (raster) maskcats
>> syntax, not the (vector) cats syntax. Each syntax is well established
>> and I don't think it's a good idea to sync the two (requiring too many
>> low-level changes that are neither bug fixes nor performance
>> improvements).
>>
>
> But it is interface improvement. It would be great for novices.

+1

Moreover I would like to have, the "same" struct that we have for
vectors: map_info, for the raster, something like: rast_info, using as
much as possible the same structure for history, categories, etc...

But maybe is an enhancement for grass8 :-)

Pietro