isDir won't throw FileException with dirEntries

[Jonathan M Davis]

On Tuesday, August 30, 2011 11:04 Nick Treleaven wrote:
> On 30/08/2011 19:02, Nick Treleaven wrote:
> > On 30/08/2011 18:54, Jonathan M Davis wrote:
> >> On Tuesday, August 30, 2011 10:39 Nick Treleaven wrote:
> >>> Hi,
> >>> With the attached source file on Windows, dmd 2.054, I'm not getting an
> >>> exception when the path doesn't exist. If I uncomment the foreach line,
> >>> the exception is thrown. Should I file this in bugzilla?
> >> 
> >> So, what exactly is the problem? Is the issue that isDir isn't
> >> throwing or
> >> that dirEntries isn't throwing? isDir should definitely throw if the
> >> path doesn't exist, but I'm not sure that dirEntries will.
> > 
> > The problem is that isDir doesn't throw - the documentation says:
> > "Throws: FileException if the given file does not exist. "
> > 
> > If I remove the foreach/dirEntries then isDir does throw.
> 
> BTW, I meant to say 'comment the foreach line' in my first post, sorry
> for the confusion...

So, you're saying that

import std.file;

void main(string[] args)
{
 auto path = args[1];

 if (!path.isDir())
 {
 }

 //foreach (DirEntry f; dirEntries(path, SpanMode.depth))
 {
 }
}

fails to throw on your system, whereas

import std.file;

void main(string[] args)
{
 auto path = args[1];

 if (!path.isDir())
 {
 }

 foreach (DirEntry f; dirEntries(path, SpanMode.depth))
 {
 }
}

_does_ throw? If so, that's a compiler bug. The foreach should have _zero_ 
effect on the statements before it - save for stuff like scope statements.

- Jonathan M Davis