Bug 11057 – [REG2.064dev] New std.uni has icmp() partly broken

Status
RESOLVED
Resolution
FIXED
Severity
regression
Priority
P2
Component
phobos
Product
D
Version
D2
Platform
All
OS
All
Creation time
2013-09-17T10:09:00Z
Last change time
2013-09-26T08:31:06Z
Keywords
pull
Assigned to
dmitry.olsh
Creator
kekeniro2

Comments

Comment #0 by kekeniro2 — 2013-09-17T10:09:50Z
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 }
Comment #1 by dmitry.olsh — 2013-09-22T10:56:07Z
This is not wrong-code, that keyword is reserved for the compiler. Pull: https://github.com/D-Programming-Language/phobos/pull/1594
Comment #2 by github-bugzilla — 2013-09-26T07:27:23Z
Comment #3 by dmitry.olsh — 2013-09-26T08:31:06Z
Fixed in git master.