Skip to content

Conversation

9rnsr
Copy link
Contributor

@9rnsr 9rnsr commented Jul 21, 2015

Points:

  • Update outdated descriptions.
  • Follow the latest compiler behaviors.
  • Add inter-page links to associate related sections.

)

$(P If the bounds check in $(D @system) or $(D @trusted) code disabled, still
the code correctness have to be guaranteed by the code author.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggestion: If the bounds check in $(D @system) or $(D @trusted) code is disabled, the code correctness must still be guaranteed by the code author.

@Hackerpilot
Copy link
Contributor

I have a few suggestions for how the wording can be improved, but other than that, this is a good change.

@9rnsr
Copy link
Contributor Author

9rnsr commented Jul 23, 2015

@Hackerpilot Thanks a lot for your wording suggestions!

with a compile time switch.
)
$(P An out of bounds memory access will cause undefined behavior,
therefore array bounds check is normally enabled in $(D @safe) function.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/function/functions/

wilzbach added a commit to wilzbach/dlang.org that referenced this pull request Dec 16, 2017
wilzbach added a commit to wilzbach/dlang.org that referenced this pull request Dec 16, 2017
wilzbach added a commit to wilzbach/dlang.org that referenced this pull request Dec 16, 2017
wilzbach added a commit to wilzbach/dlang.org that referenced this pull request Dec 16, 2017
wilzbach added a commit to wilzbach/dlang.org that referenced this pull request Dec 16, 2017
wilzbach added a commit to wilzbach/dlang.org that referenced this pull request Dec 16, 2017
wilzbach added a commit to wilzbach/dlang.org that referenced this pull request Dec 16, 2017
wilzbach added a commit to wilzbach/dlang.org that referenced this pull request Dec 16, 2017
wilzbach added a commit to wilzbach/dlang.org that referenced this pull request Dec 16, 2017
wilzbach added a commit to wilzbach/dlang.org that referenced this pull request Dec 16, 2017
@wilzbach
Copy link
Contributor

Okay I think this PR contains great value and this I did split this PR up into five small PRs. The PRs contain have all feedback applied and should be in a size which is easy to review.

See: #2001, #2002, #2003, #2004, #2005

@wilzbach wilzbach closed this Dec 16, 2017
wilzbach added a commit to wilzbach/dlang.org that referenced this pull request Dec 16, 2017
wilzbach added a commit to wilzbach/dlang.org that referenced this pull request Dec 16, 2017
wilzbach added a commit to wilzbach/dlang.org that referenced this pull request Dec 18, 2017
wilzbach added a commit to wilzbach/dlang.org that referenced this pull request Dec 18, 2017
wilzbach added a commit to wilzbach/dlang.org that referenced this pull request Dec 18, 2017
wilzbach added a commit to wilzbach/dlang.org that referenced this pull request Dec 18, 2017
wilzbach added a commit to wilzbach/dlang.org that referenced this pull request Dec 18, 2017
dlang-bot added a commit that referenced this pull request Dec 18, 2017
Revive #1040 - spec/attributes.d
merged-on-behalf-of: Sebastian Wilzbach <sebi.wilzbach@gmail.com>
dlang-bot added a commit that referenced this pull request Dec 18, 2017
Revive #1040 - spec/{enum,expression}.d + dcompiler.d
merged-on-behalf-of: Sebastian Wilzbach <sebi.wilzbach@gmail.com>
dlang-bot added a commit that referenced this pull request Dec 18, 2017
Revive #1040 - spec/arrays.d
merged-on-behalf-of: Sebastian Wilzbach <sebi.wilzbach@gmail.com>
andralex added a commit that referenced this pull request Dec 20, 2017
Revive #1040 - spec/{interface,template}.d + glossary.d
wilzbach added a commit to wilzbach/dlang.org that referenced this pull request Jan 7, 2018
CyberShadow added a commit that referenced this pull request Jan 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants