[Issue 8447] Self importing is allowed?

2012-12-22 Thread d-bugmail 
http://d.puremagic.com/issues/show_bug.cgi?id=8447


Dmitry Olshansky  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||WORKSFORME


--- Comment #2 from Dmitry Olshansky  2012-12-22 
13:13:23 PST ---
Going to close it for a moment as the codebase changed radically over time and
I'm not 100% sure self-importing was the core cause.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
--- You are receiving this mail because: ---

[Issue 8447] Self importing is allowed?

2012-10-04 Thread d-bugmail 
http://d.puremagic.com/issues/show_bug.cgi?id=8447


Andrej Mitrovic  changed:

   What|Removed |Added

 CC||andrej.mitrov...@gmail.com


--- Comment #1 from Andrej Mitrovic  2012-10-04 
20:29:10 PDT ---
Phobos has a couple of these self-imports. They all seem to be in unittest
blocks, which maybe makes copy-pasting samples to outside modules easier since
you automatically get a free import statement included.

Maybe we should allow them in function/unittest scope, and disallow them in
module scope. Or should we just ban them outright?

I'm running a test suite on a fix right now.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
--- You are receiving this mail because: ---