[GRASS-windows] Re: [winGRASS] v.db.addcol

[Glynn Clements]

Moritz Lennert wrote:

> >> > Call remove() first.
> >> >
> >> I've found the following other places where I have the feeling that this
> >> needs to be done, but would like confirmation that these are correct:
> >
> > [snip]
> >
> > I think that we need a G_rename_file() function (G_rename() already
> > exists; that renames maps etc). Ideally, the remove() call should only
> > be used on Windows.
> 
> Does this look like what you meant ?

Yes.

> I imagine I should add some error checking in there ?

I would make G_rename_file() return the result of rename(), so that it
behaves as a drop-in replacement.

There's no need to check the result of remove().

-- 
Glynn Clements <glynn at gclements.plus.com>