www.digitalmars.com         C & C++   DMDScript  

digitalmars.D.bugs - [Issue 11057] New: [REG2.064dev] New std.uni has icmp() partly broken

reply d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=11057

           Summary: [REG2.064dev] New std.uni has icmp() partly broken
           Product: D
           Version: D2
          Platform: All
        OS/Version: All
            Status: NEW
          Keywords: wrong-code
          Severity: regression
          Priority: P2
         Component: Phobos
        AssignedTo: nobody puremagic.com
        ReportedBy: kekeniro2 yahoo.co.jp


--- Comment #0 from kekeniro2 yahoo.co.jp 2013-09-17 10:09:50 PDT ---
The code below fails in the last assertion since 2.064dev introduces new
std.uni.
The implementation of Ystd.uni.icmp() gets complicated.
std.path.filenameCmp(aka. fcmp) doesn't use it and is simple.

CODE:
import std.string; // or std.uni

void main(){
    assert( icmp("A", "B") < 0 ); // OK
    assert( icmp("J", "K") < 0 ); // OK
    assert( icmp("L", "M") < 0 ); // OK

    assert( icmp("K", "L") < 0 ); // NG
}

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Sep 17 2013
next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=11057


Dmitry Olshansky <dmitry.olsh gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|wrong-code                  |pull


--- Comment #1 from Dmitry Olshansky <dmitry.olsh gmail.com> 2013-09-22
10:56:07 PDT ---
This is not wrong-code, that keyword is reserved for the compiler.
Pull:
https://github.com/D-Programming-Language/phobos/pull/1594

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Sep 22 2013
prev sibling next sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=11057



--- Comment #2 from github-bugzilla puremagic.com 2013-09-26 07:27:23 PDT ---
Commit pushed to master at https://github.com/D-Programming-Language/phobos

https://github.com/D-Programming-Language/phobos/commit/9df179b65deb81021dd5cf18e83e05aec419dbac
fix issue 11057 [REG2.064dev] New std.uni has icmp() partly broken

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Sep 26 2013
prev sibling parent d-bugmail puremagic.com writes:
http://d.puremagic.com/issues/show_bug.cgi?id=11057


Dmitry Olshansky <dmitry.olsh gmail.com> changed:

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


--- Comment #3 from Dmitry Olshansky <dmitry.olsh gmail.com> 2013-09-26
08:31:06 PDT ---
Fixed in git master.

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