[LLVMdev] Little win32/Signals.cpp patch

[Jeff Cohen]

The compiler is not finding them anywhere.  If they were in the global
namespace they'd be found even without ::.  What :: does is force the
compiler to ignore definitions in other scopes that would take
precedence over the global namespace.  But if there were any such
(incorrect) definitions being found, it would not be giving a "not
found" error.

The best solution for now is to include <iostream> and replace
std::remove() with Path::destroy_file().


On Fri, 24 Sep 2004 10:33:26 -0500
Misha Brukman <brukman at uiuc.edu> wrote:

> On Fri, Sep 24, 2004 at 05:32:08PM +0200, Paolo Invernizzi wrote:
> > Jeff Cohen wrote:
> > 
> > ><algorithm> works too.
> > > 
> > >
> > For std::remove yes... but...
> > 
> > d:\home\arathorn\sandbox\llvm\llvm\lib\System\platform\Signals.cpp(179) 
> > : error C2065: 'stderr' : undeclared identifier
> > d:\home\arathorn\sandbox\llvm\llvm\lib\System\platform\Signals.cpp(179) 
> > : error C3861: 'fprintf': identifier not found, even with 
> > argument-dependent lookup
> 
> These should all be prepended with ::, methinks, to label them as coming
> from the global namespace...
> 
> For one, as example I'm seeing the following in Unix/Path.cpp:
> 
>   int fd = ::creat(path.c_str(), S_IRUSR | S_IWUSR);
> 
> -- 
> Misha Brukman :: http://misha.brukman.net :: http://llvm.cs.uiuc.edu
> 
> _______________________________________________
> LLVM Developers mailing list
> LLVMdev at cs.uiuc.edu         http://llvm.cs.uiuc.edu
> http://mail.cs.uiuc.edu/mailman/listinfo/llvmdev