[IoVM] File moveTo

[superbobry]

According to the docs:

Raises ... File nameConflict exception if the file nameString already exists.

The realities:
Io> f := File with("test.io") open close
==> File_0x9f73450:

Io> f exists
==> true
Io> File with(f path) exists
==> true
Io> File with("testAgain.io") open moveTo(f path)
==>  File_0x9f88808:

[stevedekorte]

Why shouldn't it be a name conflict if your trying to move a file on top of one that already exists?

[superbobry]

No idea. I was writing tests for Directory moveTo and since it's a wrapper around File moveTo I decided to test the behaviour, described in it's docstring, and it didn't work as described. That's the story :)

IO_METHOD(IoFile, moveTo_)
{
    /*doc File moveTo(pathString)
    Moves the file specified by the receiver's path to the
    new path pathString. Raises an File doesNotExist exception if the
    file does not exist or a File nameConflict exception if the file
    nameString already exists.
    */

    IoSymbol *newPath = IoMessage_locals_symbolArgAt_(m, locals, 0);
    const char *fromPath = CSTRING(DATA(self)->path);
    const char *toPath = CSTRING(newPath);

[stevedekorte]

Oh, sorry - I thought you were saying it did raise an exception.

[mikedld]

Seems that either code or documentation should be fixed since they strictly conflict with each other. Later in the code,

...
remove(toPath); // to make sure we do not get an error
...

[stevedekorte]

Ok, lets' change the docs.