Committing code review suggestions

Co-authored-by: Juliette <[email protected]>

Author: rodrigoprimo

src/Standards/Generic/Sniffs/NamingConventions/CamelCapsFunctionNameSniff.php

@@ -113,6 +113,7 @@ protected function processTokenWithinScope(File $phpcsFile, $stackPtr, $currScop
 
         $methodName = $phpcsFile->getDeclarationName($stackPtr);
         if ($methodName === null) {
+            // Live coding or parse error. Bow out.
             return;
         }
 
@@ -187,6 +188,7 @@ protected function processTokenOutsideScope(File $phpcsFile, $stackPtr)
     {
         $functionName = $phpcsFile->getDeclarationName($stackPtr);
         if ($functionName === null) {
+            // Live coding or parse error. Bow out.
             return;
         }
 
@@ -203,7 +205,7 @@ protected function processTokenOutsideScope(File $phpcsFile, $stackPtr)
             $phpcsFile->addError($error, $stackPtr, 'FunctionDoubleUnderscore', $errorData);
         }
 
-        // Ignore first underscore in functions prefixed with "_".
+        // Ignore leading underscores in the method name.
         $functionName = ltrim($functionName, '_');
 
         if (Common::isCamelCaps($functionName, false, true, $this->strict) === false) {

src/Standards/Generic/Tests/NamingConventions/CamelCapsFunctionNameUnitTest.1.inc

@@ -185,8 +185,8 @@ enum Suit: string implements Colorful, CardGame {
 }
 
 interface MyInterface {
-    function getSomeValue(); // Ok.
-    function get_some_value(); // Not ok.
+    public function getSomeValue();
+    public function get_some_value();
 }
 
 class MyClass {