This is an archive of the discontinued LLVM Phabricator instance.

docs/clang-tidy/checks/misc-argument-comment.rst
27 ↗	(On Diff #70015)	Good catch! It would be better to avoid double negation. E.g. `When non-zero, the check will take into account leading and trailing underscores....`. I'd also add `Default is zero.`, if it's not there. (A full-context diff would make it easier to see what's in the rest of the document).

This revision now requires changes to proceed.Sep 7 2016, 4:16 AM

rewritten description

docs/clang-tidy/checks/misc-argument-comment.rst
27 ↗	(On Diff #70015)	Sorry for not uploading the whole diff. I have rewritten it based on the comment but I liked the word 'ignore' so came up with an intermediate solution.

LG. Thank you!

This revision is now accepted and ready to land.Sep 20 2016, 7:37 AM

Looks like patch was not committed.

This revision was automatically updated to reflect the committed changes.

Revision Contents

Path

Size

clang-tools-extra/

trunk/

docs/

clang-tidy/

checks/

5 lines

	Show All 18 Lines

	The check tries to detect typos and suggest automated fixes for them.			The check tries to detect typos and suggest automated fixes for them.

	Options			Options
	-------			-------

	.. option:: StrictMode			.. option:: StrictMode

	When non-zero, the check will ignore leading and trailing underscores and			When zero (default value), the check will ignore leading and trailing
	case when comparing parameter names.			underscores and case when comparing names -- otherwise they are taken into
				account.