January 30, 2017 [Issue 8471] std.stdio.readf should be @trusted | ||||
|---|---|---|---|---|
| ||||
 | https://issues.dlang.org/show_bug.cgi?id=8471 --- Comment #13 from Jakub Łabaj <uaaabbjjkl@gmail.com> --- I understand now, thanks! You can find my profile here: https://github.com/byebye. I've create a simple PR: https://github.com/dlang/phobos/pull/5040 for similar issue involving formattedRead, where I was wondering if I should restrict arguments to pointers. So it seems the solution would be exactly the same in both cases. -- | |||
Permalink
ReplyFebruary 02, 2017 [Issue 8471] std.stdio.readf should be @trusted | ||||
|---|---|---|---|---|
| ||||
 | https://issues.dlang.org/show_bug.cgi?id=8471 --- Comment #14 from github-bugzilla@puremagic.com --- Commits pushed to master at https://github.com/dlang/phobos https://github.com/dlang/phobos/commit/882a1fb8f62b872f3463e9fca2b2e1e04d36effb Fix issue #8471 - allow only pointers as readf parameters https://github.com/dlang/phobos/commit/0059fc3263b9ebd62ef7c6c7b3eed4b20f313364 Merge pull request #5076 from byebye/issue_8471 Fix issue #8471 - allow only pointers as readf parameters -- | |||
February 02, 2017 [Issue 8471] std.stdio.readf should be @trusted | ||||
|---|---|---|---|---|
| ||||
| https://issues.dlang.org/show_bug.cgi?id=8471 github-bugzilla@puremagic.com changed: What |Removed |Added ---------------------------------------------------------------------------- Status|REOPENED |RESOLVED Resolution|--- |FIXED -- | |||
February 05, 2017 [Issue 8471] std.stdio.readf should be @trusted | ||||
|---|---|---|---|---|
| ||||
| https://issues.dlang.org/show_bug.cgi?id=8471 Jakub Łabaj <uaaabbjjkl@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |REOPENED Resolution|FIXED |--- --- Comment #15 from Jakub Łabaj <uaaabbjjkl@gmail.com> --- It's not fixed yet, my commit doesn't really make readf @safe/@trusted (sorry for the mess). -- | |||
February 24, 2017 [Issue 8471] std.stdio.readf should be @trusted | ||||
|---|---|---|---|---|
| ||||
| https://issues.dlang.org/show_bug.cgi?id=8471 --- Comment #16 from github-bugzilla@puremagic.com --- Commits pushed to newCTFE at https://github.com/dlang/phobos https://github.com/dlang/phobos/commit/882a1fb8f62b872f3463e9fca2b2e1e04d36effb Fix issue #8471 - allow only pointers as readf parameters https://github.com/dlang/phobos/commit/0059fc3263b9ebd62ef7c6c7b3eed4b20f313364 Merge pull request #5076 from byebye/issue_8471 -- | |||
February 24, 2017 [Issue 8471] std.stdio.readf should be @trusted | ||||
|---|---|---|---|---|
| ||||
| https://issues.dlang.org/show_bug.cgi?id=8471 github-bugzilla@puremagic.com changed: What |Removed |Added ---------------------------------------------------------------------------- Status|REOPENED |RESOLVED Resolution|--- |FIXED -- | |||
February 26, 2017 [Issue 8471] std.stdio.readf should be @trusted | ||||
|---|---|---|---|---|
| ||||
 | https://issues.dlang.org/show_bug.cgi?id=8471 greenify <greeenify@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |REOPENED CC| |greeenify@gmail.com Resolution|FIXED |--- --- Comment #17 from greenify <greeenify@gmail.com> --- (wrongly closed) -- | |||
March 17, 2017 [Issue 8471] std.stdio.readf should be @trusted | ||||
|---|---|---|---|---|
| ||||
 | https://issues.dlang.org/show_bug.cgi?id=8471 Bolpat <qs.il.paperinik@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |qs.il.paperinik@gmail.com --- Comment #18 from Bolpat <qs.il.paperinik@gmail.com> --- As Andrei pointed out, readf has its signature for historic reasons. Please check out my pull: https://github.com/dlang/phobos/pull/5247 If I'm correct, we can deprecate formattedRead and with it readf with pointers. Tell me if I'm wrong, but there is no justification to make readf @trusted. It accesses a __gshared global and is not @safe for good reasons. -- | |||
March 18, 2017 [Issue 8471] std.stdio.readf should be @trusted | ||||
|---|---|---|---|---|
| ||||
 | https://issues.dlang.org/show_bug.cgi?id=8471 --- Comment #19 from Andrei Alexandrescu <andrei@erdani.com> --- @Bolpat (In reply to Bolpat from comment #18) > As Andrei pointed out, readf has its signature for historic reasons. Please check out my pull: https://github.com/dlang/phobos/pull/5247 If I'm correct, we can deprecate formattedRead and with it readf with pointers. > > Tell me if I'm wrong, but there is no justification to make readf @trusted. It accesses a __gshared global and is not @safe for good reasons. Using __gshared is not problematic because the reading primitives are interlocked. -- | |||
March 22, 2017 [Issue 8471] std.stdio.readf should be @trusted | ||||
|---|---|---|---|---|
| ||||
| https://issues.dlang.org/show_bug.cgi?id=8471 --- Comment #20 from github-bugzilla@puremagic.com --- Commits pushed to stable at https://github.com/dlang/phobos https://github.com/dlang/phobos/commit/882a1fb8f62b872f3463e9fca2b2e1e04d36effb Fix issue #8471 - allow only pointers as readf parameters https://github.com/dlang/phobos/commit/0059fc3263b9ebd62ef7c6c7b3eed4b20f313364 Merge pull request #5076 from byebye/issue_8471 -- | |||
Copyright © 1999-2021 by the D Language Foundation