Bug 17524 – [The C Preprocessor vs D] "need to worry about"?

Status
RESOLVED
Resolution
FIXED
Severity
enhancement
Priority
P3
Component
dlang.org
Product
D
Version
D2
Platform
All
OS
All
Creation time
2017-06-19T12:35:04Z
Last change time
2017-08-16T13:23:26Z
Keywords
pull
Assigned to
No Owner
Creator
Pet

Comments

Comment #0 by qmnaidgolfrot — 2017-06-19T12:35:04Z
Page http://dlang.org/pretod.html contains "Parameters are passed by value, so need to worry about side effects". Do need to worry or do *not* need to worry?
Comment #1 by dlang-bugzilla — 2017-06-19T12:50:09Z
https://github.com/dlang/dlang.org/pull/1726 Pet, would you consider creating a GitHub account and suggesting edits directly via pull requests? GitHub allows suggesting small edits directly in the web browser - see the "improve this page" link on every page.
Comment #2 by github-bugzilla — 2017-06-19T13:35:51Z
Commits pushed to master at https://github.com/dlang/dlang.org https://github.com/dlang/dlang.org/commit/83016443332f7dc595980afad81b3b6f47d35f2e Fix Issue 17524 - [The C Preprocessor vs D] "need to worry about"? https://github.com/dlang/dlang.org/commit/7d8e147ae9336a5df33d1feeb54e54fd37f97664 Merge pull request #1726 from CyberShadow/pull-20170619-124653 Fix Issue 17524 - [The C Preprocessor vs D] "need to worry about"?
Comment #3 by github-bugzilla — 2017-08-14T23:56:04Z
Commits pushed to newCTFE at https://github.com/dlang/dlang.org https://github.com/dlang/dlang.org/commit/83016443332f7dc595980afad81b3b6f47d35f2e Fix Issue 17524 - [The C Preprocessor vs D] "need to worry about"? https://github.com/dlang/dlang.org/commit/7d8e147ae9336a5df33d1feeb54e54fd37f97664 Merge pull request #1726 from CyberShadow/pull-20170619-124653
Comment #4 by github-bugzilla — 2017-08-16T13:23:26Z
Commits pushed to stable at https://github.com/dlang/dlang.org https://github.com/dlang/dlang.org/commit/83016443332f7dc595980afad81b3b6f47d35f2e Fix Issue 17524 - [The C Preprocessor vs D] "need to worry about"? https://github.com/dlang/dlang.org/commit/7d8e147ae9336a5df33d1feeb54e54fd37f97664 Merge pull request #1726 from CyberShadow/pull-20170619-124653