← Back to index | Original Bugzilla link

Bug 14212 – frexp for const and immutable fails to compile

Status
RESOLVED
Resolution
FIXED
Severity
regression
Priority
P1
Component
phobos
Product
D
Version
D2
Platform
All
OS
All
Creation time
2015-02-21T09:16:00Z
Last change time
2017-07-19T17:42:15Z
Assigned to
nobody
Creator
ilyayaroshenko

Comments

Comment #0 by ilyayaroshenko — 2015-02-21T09:16:59Z
Comment #1 by bugzilla — 2015-02-22T07:51:25Z
Would like some more information on this. Please post a code example that does not compile.
Comment #2 by ilyayaroshenko — 2015-02-22T07:56:10Z
I am sorry, my bad. This is really Phobos issue. The fix is there https://github.com/D-Programming-Language/phobos/pull/3017
Comment #3 by bearophile_hugs — 2015-02-22T08:50:11Z
See also Issue 6586
Comment #4 by bugzilla — 2015-02-22T09:29:05Z
What exactly is the problem?
Comment #5 by ilyayaroshenko — 2015-02-22T09:42:06Z
The `value` should be mutable variable since it is changed inside function. The fix adds `Unqual!T vf = value;` to fix this issue.
Comment #6 by github-bugzilla — 2015-02-27T16:30:47Z
Commit pushed to master at https://github.com/D-Programming-Language/phobos https://github.com/D-Programming-Language/phobos/commit/62acd72cec052169e7eb56d1c63c860ae92f9ef9 Merge pull request #3017 from 9il/frexp [2.067.0-b2][regression] fix Issues 14212 and 6586
Comment #7 by github-bugzilla — 2015-02-27T20:55:45Z
Commit pushed to 2.067 at https://github.com/D-Programming-Language/phobos https://github.com/D-Programming-Language/phobos/commit/cf57c7529a63ac970167c65cb659d5249a3a1ab9 Merge pull request #3017 from 9il/frexp [2.067.0-b2][regression] fix Issues 14212 and 6586
Comment #8 by github-bugzilla — 2015-04-11T12:24:48Z
Commit pushed to master at https://github.com/D-Programming-Language/phobos https://github.com/D-Programming-Language/phobos/commit/cf57c7529a63ac970167c65cb659d5249a3a1ab9 Merge pull request #3017 from 9il/frexp
Comment #9 by github-bugzilla — 2015-06-17T21:03:10Z
Commit pushed to stable at https://github.com/D-Programming-Language/phobos https://github.com/D-Programming-Language/phobos/commit/62acd72cec052169e7eb56d1c63c860ae92f9ef9 Merge pull request #3017 from 9il/frexp
Comment #10 by github-bugzilla — 2017-07-19T17:42:15Z
Commit pushed to dmd-cxx at https://github.com/dlang/phobos https://github.com/dlang/phobos/commit/62acd72cec052169e7eb56d1c63c860ae92f9ef9 Merge pull request #3017 from 9il/frexp