Bug 15369 – [REG master] id.d(369): Error: Outside Unicode code space

Status
RESOLVED
Resolution
FIXED
Severity
regression
Priority
P1
Component
dmd
Product
D
Version
D2
Platform
All
OS
All
Creation time
2015-11-20T23:07:00Z
Last change time
2016-01-03T14:02:33Z
Keywords
pull, wrong-code
Assigned to
nobody
Creator
issues.dlang

Comments

Comment #0 by issues.dlang — 2015-11-20T23:07:43Z
If I build dmd with the current dmd master, I get this error id.d(369): Error: Outside Unicode code space For some reason, line 369 ends up being empty = Identifier.idPool("¥p"); whereas it's supposed to be empty = Identifier.idPool(""); Building dmd with 2.067 works fine (which is what we normally do right now AFAIK), otherwise this probably would have showed up on the autotester. So, while it may not be an immediate problem, presumably we need to fix it at some point here, since we don't want to be stuck having to build dmd with 2.067 forever.
Comment #1 by k.hara.pg — 2015-11-21T15:35:38Z
Reduced test case (Note that the reproduction ratio is not 100%): import core.stdc.stdio; struct Msgtable { const(char)[] ident; const(char)* name; }; Msgtable[] msgtable = [ { "empty", "" }, ]; void main() { auto id = msgtable[0].ident; auto p = msgtable[0].name; printf("empty -> p = %p >>>%s<<<\n", p, p); } Introduced in: https://github.com/D-Programming-Language/dmd/pull/5220
Comment #2 by k.hara.pg — 2015-11-21T15:37:15Z
This is git-head only regression. Same issue does not exist in 2.069.1 or earlier releases.
Comment #3 by k.hara.pg — 2015-11-22T02:30:05Z
Comment #4 by github-bugzilla — 2015-11-28T18:51:14Z
Commits pushed to master at https://github.com/D-Programming-Language/dmd https://github.com/D-Programming-Language/dmd/commit/f13d425262073764f5389cc3717b4faf20c18058 fix Issue 15369 - id.d(369): Error: Outside Unicode code space https://github.com/D-Programming-Language/dmd/commit/7b2a4fff5d7ba3cb35a2d5341f62b48d6ce79df3 Merge pull request #5280 from 9rnsr/fix15369 [REG master] Issue 15369 - id.d(369): Error: Outside Unicode code space
Comment #5 by github-bugzilla — 2016-01-03T14:02:33Z