Thread overview
[Issue 6634] New: std.path.globMatch throws wrong assertion
Sep 09, 2011
Rainer Schuetze
Sep 18, 2011
Brad Roberts
Sep 20, 2011
Rainer Schuetze
Oct 08, 2011
Kenji Hara
September 09, 2011
http://d.puremagic.com/issues/show_bug.cgi?id=6634

           Summary: std.path.globMatch throws wrong assertion
           Product: D
           Version: D2
          Platform: Other
        OS/Version: Windows
            Status: NEW
          Severity: regression
          Priority: P2
         Component: Phobos
        AssignedTo: nobody@puremagic.com
        ReportedBy: r.sagitario@gmx.de


--- Comment #0 from Rainer Schuetze <r.sagitario@gmx.de> 2011-09-09 03:28:35 PDT ---
import std.path;

void main()
{
    globMatch("a.di", "*.d");
}

asserts with

core.exception.AssertError@std.path(2239): Assertion failure

The same happens when using fnmatch instead of globMatch. The problem is a
wrong assert at the end of both functions:
    assert(ni >= path.length);

The wrong assert has been there before, but was actually never executed because it was built into the runtime library in release mode only. Now that the functions are templates, code generation moved to the user project, so the asserts can actually trigger.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
September 18, 2011
http://d.puremagic.com/issues/show_bug.cgi?id=6634


Brad Roberts <braddr@puremagic.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |braddr@puremagic.com


--- Comment #1 from Brad Roberts <braddr@puremagic.com> 2011-09-17 17:16:28 PDT ---
Care to turn this into a pull request with a unittest?

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
September 20, 2011
http://d.puremagic.com/issues/show_bug.cgi?id=6634



--- Comment #2 from Rainer Schuetze <r.sagitario@gmx.de> 2011-09-19 22:53:22 PDT ---
https://github.com/D-Programming-Language/phobos/pull/267

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
October 08, 2011
http://d.puremagic.com/issues/show_bug.cgi?id=6634


Kenji Hara <k.hara.pg@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED


--- Comment #3 from Kenji Hara <k.hara.pg@gmail.com> 2011-10-08 14:57:32 PDT ---
https://github.com/D-Programming-Language/phobos/commit/2399de49d8ffee3c8246b83561cbfc9c3ccd3c90

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