Fixed a few spelling errors in clang/include/clang/StaticAnalyzer/Checkers/CheckerBase.td

[giannicrivello]

Summary

Fixed spelling errors in said file. I am also new to LLVM contribution so I wanted to test the waters and make a small commit before taking on bigger projects.

[NoQ]

Nice, thanks!

For non-trivial patches it's recommended to include full diff context via -U999999 (https://llvm.org/docs/Phabricator.html#requesting-a-review-via-the-web-interface) so that reviewers could easily see what does the surrounding code do.

clang/include/clang/StaticAnalyzer/Checkers/CheckerBase.td

30

This one's probably intentional!

[giannicrivello]

In D138964#3958986, @NoQ wrote:

Nice, thanks!

For non-trivial patches it's recommended to include full diff context via -U999999 (https://llvm.org/docs/Phabricator.html#requesting-a-review-via-the-web-interface) so that reviewers could easily see what does the surrounding code do.

Ok will do, thanks. So how would I move forward from here?

clang/include/clang/StaticAnalyzer/Checkers/CheckerBase.td

30

I see what they did there. lol

[NoQ]

So how would I move forward from here?

There's the "Update Diff" button on the right! You can use it to address review comments until everyone's happy.

[tbaeder]

clang/include/clang/StaticAnalyzer/Checkers/CheckerBase.td

102

Sure that "name" and "fullname" here aren't actual field names in the code?

[giannicrivello]

Addressed the intentional spelling errors, also fixed a few more grammar errors

clang/include/clang/StaticAnalyzer/Checkers/CheckerBase.td

 def Boolean : CmdLineOptionTypeEnum<2>;
 
 /// Describes the state of the entry. We wouldn't like to display, for example,
 /// developer only entries for a list meant for end users.
 class DevelopmentStageEnum<bits<1> val> {
   bits<1> Val = val;
 }
 
-/// Alpha entries are under development, might be incomplete, incorrect and
+/// Alpha entries are under development, might be incomplet, inkorrekt and
 /// unstable.
 def InAlpha : DevelopmentStageEnum<0>;
 
 /// Released entries are stable, produce minimal, if any false positives,
-/// and emits reports that explain the occurance of the bug understandably and
+/// and emits reports that explain the occurrence of the bug understandably and
 /// thoroughly.
 def Released : DevelopmentStageEnum<1>;
 
 /// Marks the entry hidden. Hidden entries won't be displayed in
 /// -analyzer-checker-option-help.
 class HiddenEnum<bit val> {
   bit Val = val;
 }

 
 class Documentation<DocumentationEnum val> {
   bits<1> Documentation = val.Documentation;
 }
 
 /// Describes a checker. Every builtin checker has to be registered with the use
 /// of this class (out-of-trunk checkers loaded from plugins obviously don't).
 /// Note that a checker has a name (e.g.: 'NullDereference'), and a full name,
 /// that is auto-generated with the help of the ParentPackage field, that also
 /// includes package names (e.g.: 'core.NullDereference').
 /// Example:
 ///   def DereferenceChecker : Checker<"NullDereference">,
 ///     HelpText<"Check for dereferences of null pointers">;
 class Checker<string name = ""> {
   string              CheckerName = name;
   string              HelpText;
   // This field is optional.