[llvm-commits] paths on win32

Nick Lewycky nicholas at mxc.ca
Sun May 11 00:48:46 PDT 2008


There's a problem on win32 where Paths can be created with \'s as path 
separators in them. The win32 sys::Path implementation normally 
canonicalize them into /'s, but using the constructor will bypass that. 
This is a problem because clients expect to be able to take any valid 
win32 path (which may have any mix of / and \) and construct a valid 
Path with it, through the constructor.

This patch moves the constructor implementations out of 
include/llvm/System/Path.h and into lib/System/*/Path.inc. This fixes 
the bug, but there may be a small price in efficiency. Is this approach 
okay? (There are two other ways to fix the bug.)

Nick

-------------- next part --------------
A non-text attachment was scrubbed...
Name: win32-path.patch
Type: text/x-diff
Size: 2152 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20080511/8c7b5cb6/attachment.patch>


More information about the llvm-commits mailing list