This CL is an alternative to replace http://codereview.appspot.com/3096042/. It solves the same problem, but I ...
13 years, 4 months ago
(2010-12-15 23:28:31 UTC)
#2
This CL is an alternative to replace
http://codereview.appspot.com/3096042/. It solves the same problem, but I like
this one better, because:
- removed some C code, new code is Go;
- can reuse some existing code from "os";
- it doesn't affect other GOOSes in any way;
- it gives us finer control over dll loading(*);
*) This is a list of dlls, that are loaded for basic program (without any
packages imported):
test
SHLWAPI
COMCTL32
RPCRT4
USER32
GDI32
ntdll
msvcrt
ADVAPI32
Secur32
KERNEL32
shell32
same list after this CL applied
test
ntdll
kernel32
the difference is one function CommandLineToArgv from shell32.dll, shell32.dll
pulls a lot of other dlls. It is true, all dlls will be loaded anyway once
syscall package is used, but we could optimize that by loading dlls only if they
are used (same could be said about network dlls).
Issue 3702041: code review 3702041: runtime: move windows goargs implementation from runtim...
(Closed)
Created 13 years, 4 months ago by brainman
Modified 13 years, 4 months ago
Reviewers:
Base URL:
Comments: 0