http://codereview.chromium.org/56107/diff/1/5 File src/d8-posix.cc (right):
http://codereview.chromium.org/56107/diff/1/5#newcode587 Line 587: if (errno == EEXIST) { On 2009/03/31 12:28:02, Søren Gjesse wrote: > Is this is to handle from paths with '..' e.g. xxx/yyy/../yyy, or is it not > supposed to happen? Can I just pretend that's what I was thinking about all along? Test added to ensure that it actually works this way. http://codereview.chromium.org/56107/diff/1/3 File src/d8-windows.cc (right): http://codereview.chromium.org/56107/diff/1/3#newcode39 Line 39: os_templ->Set(String::New("setenv"), FunctionTemplate::New(SetEnvironment)); On 2009/03/31 12:28:02, Søren Gjesse wrote: > Why did you leave setenv here? It is not implemented either. Good point. Removed. http://codereview.chromium.org/56107 --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
