This is an archive of the discontinued LLVM Phabricator instance.

docs/ReleaseNotes.rst
153–154 ↗	(On Diff #52083)	Was this added in 3.8 or is it being added in 3.9? That URL gives me 404 not found. If it is new to 3.9, it should be farther up in the file.

fowles added inline comments.Mar 30 2016, 2:20 PM

docs/ReleaseNotes.rst
153–154 ↗	(On Diff #52083)	I don't know. It was committed yesterday.

LegalizeAdulthood added inline comments.Mar 30 2016, 2:33 PM

docs/ReleaseNotes.rst
153–154 ↗	(On Diff #52083)	Hrm. I see the page describing it here: http://clang.llvm.org/extra/clang-tidy/checks/readability-avoid-const-params-in-decls.html but I think those docs are continuously rebuilt from trunk. The release specific URL you added does not work and if you look in the base directory for that URL: http://llvm.org/releases/3.8.0/tools/clang/tools/extra/docs/clang-tidy/checks/ you will not find a file for readability-avoid-const-params-in-decls I suspect it means that the documentation was added since the 3.8 release and therefore the check isn't actually available in the 3.8 release either. Yep, looking at the github mirror for branch release_38, this check isn't present. See the mirror on github to verify this. Therefore, I conclude that the check should be listed in the what's new for 3.9 section and not what was added between 3.7 and 3.8.

move release note to 3.9 section

Closed by commit rL265007: note for top-level consts in function decls tidy (authored by alexfh). · Explain WhyMar 31 2016, 5:12 AM

This revision was automatically updated to reflect the committed changes.

alexfh added inline comments.Mar 31 2016, 5:12 AM

docs/ReleaseNotes.rst
73 ↗	(On Diff #52126)	Typo: "delcartions". Fixed in a follow up.

Thank you for the patch, btw!

Revision Contents

Path

Size

clang-tools-extra/

trunk/

docs/

ReleaseNotes.rst

4 lines

Diff 52187

clang-tools-extra/trunk/docs/ReleaseNotes.rst

	Show First 20 Lines • Show All 62 Lines • ▼ Show 20 Lines
	explain them more clearly, and provide more accurate fix-its for the issues			explain them more clearly, and provide more accurate fix-its for the issues
	identified. The improvements since the 3.8 release include:			identified. The improvements since the 3.8 release include:

	- New ``modernize-raw-string-literal`` check			- New ``modernize-raw-string-literal`` check

	This check selectively replaces string literals containing escaped			This check selectively replaces string literals containing escaped
	characters with raw string literals.			characters with raw string literals.

				- New ``readability-avoid-const-params-in-decls`` check

				This check warns about top-level const parameters in function delcartions.

	Clang-tidy changes from 3.7 to 3.8			Clang-tidy changes from 3.7 to 3.8
	^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^			^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

	The 3.8 release didn't include release notes for :program:`clang-tidy`. In the			The 3.8 release didn't include release notes for :program:`clang-tidy`. In the
	3.8 release many new checks have been added to :program:`clang-tidy`:			3.8 release many new checks have been added to :program:`clang-tidy`:

	- Checks enforcing certain rules of the `CERT Secure Coding Standards			- Checks enforcing certain rules of the `CERT Secure Coding Standards
	<https://www.securecoding.cert.org/confluence/display/seccode/SEI+CERT+Coding+Standards>`_:			<https://www.securecoding.cert.org/confluence/display/seccode/SEI+CERT+Coding+Standards>`_:
	▲ Show 20 Lines • Show All 88 Lines • Show Last 20 Lines

This is an archive of the discontinued LLVM Phabricator instance.

note for top-level consts in function decls tidyClosedPublic

Details

Diff Detail

Event Timeline

Revision Contents

Diff 52187

clang-tools-extra/trunk/docs/ReleaseNotes.rst

note for top-level consts in function decls tidy
ClosedPublic