This is an archive of the discontinued LLVM Phabricator instance.

[clang-tidy] Update doc: ClangTidyCheck is no more DefaultConstructible.
AbandonedPublic

Authored by curdeius on Sep 17 2014, 5:55 AM.

Reviewers

klimek
bkramer

Summary

Updated documentation on creating a new check for clang-tidy.
Now, a new check should define its constructor.

curdeius retitled this revision from to [clang-tidy] Update doc: ClangTidyCheck is no more DefaultConstructible..

curdeius updated this object.

curdeius edited the test plan for this revision. (Show Details)

curdeius added a subscriber: Unknown Object (MLST).

Obsoleted by rev. 218570.

Path

Size

docs/

3 lines

	Show First 20 Lines • Show All 227 Lines • ▼ Show 20 Lines
	.. code-block:: c++			.. code-block:: c++

	#include "../ClangTidy.h"			#include "../ClangTidy.h"

	namespace clang {			namespace clang {
	namespace tidy {			namespace tidy {

	class MyCheck : public ClangTidyCheck {			class MyCheck : public ClangTidyCheck {
				public:
				MyCheck(StringRef Name, ClangTidyContext *Context)
				: ClangTidyCheck(Name, Context) {}
	};			};

	} // namespace tidy			} // namespace tidy
	} // namespace clang			} // namespace clang

	Next, you need to decide whether it should operate on the preprocessor level or			Next, you need to decide whether it should operate on the preprocessor level or
	on the AST level. Let's imagine that we need to work with the AST in our check.			on the AST level. Let's imagine that we need to work with the AST in our check.
	In this case we need to override two methods:			In this case we need to override two methods:
	▲ Show 20 Lines • Show All 117 Lines • Show Last 20 Lines