--- a/src/eric7/Documentation/Source/eric7.Plugins.CheckerPlugins.CodeStyleChecker.Simplify.SimplifyNodeVisitor.html Tue Aug 23 19:33:07 2022 +0200 +++ b/src/eric7/Documentation/Source/eric7.Plugins.CheckerPlugins.CodeStyleChecker.Simplify.SimplifyNodeVisitor.html Tue Aug 23 19:35:42 2022 +0200 @@ -60,6 +60,10 @@ <td>Constructor</td> </tr> <tr> +<td><a href="#SimplifyNodeVisitor.__bodyContainsContinue">__bodyContainsContinue</a></td> +<td>Private method to check, if a list of statements contain a 'continue' statement.</td> +</tr> +<tr> <td><a href="#SimplifyNodeVisitor.__check101">__check101</a></td> <td>Private method to check for duplicate isinstance() calls.</td> </tr> @@ -140,6 +144,10 @@ <td>Private method to check for all if-blocks which only check if a key is in a dictionary.</td> </tr> <tr> +<td><a href="#SimplifyNodeVisitor.__check123">__check123</a></td> +<td>Private method to check for complicated dictionary access with default value.</td> +</tr> +<tr> <td><a href="#SimplifyNodeVisitor.__check181">__check181</a></td> <td>Private method to check for assignments that could be converted into an augmented assignment.</td> </tr> @@ -220,6 +228,34 @@ <td>Private method to check for bare numeric function arguments.</td> </tr> <tr> +<td><a href="#SimplifyNodeVisitor.__check901">__check901</a></td> +<td>Private method to check for unnecessary bool conversion.</td> +</tr> +<tr> +<td><a href="#SimplifyNodeVisitor.__check904">__check904</a></td> +<td>Private method to check for dictionary initialization.</td> +</tr> +<tr> +<td><a href="#SimplifyNodeVisitor.__check905">__check905</a></td> +<td>Private method to check for list initialization by splitting a string.</td> +</tr> +<tr> +<td><a href="#SimplifyNodeVisitor.__check906">__check906</a></td> +<td>Private method to check for unnecessary nesting of os.path.join().</td> +</tr> +<tr> +<td><a href="#SimplifyNodeVisitor.__check907">__check907</a></td> +<td>Private method to check for Union type annotation with None.</td> +</tr> +<tr> +<td><a href="#SimplifyNodeVisitor.__check909">__check909</a></td> +<td>Private method to check for reflexive assignments.</td> +</tr> +<tr> +<td><a href="#SimplifyNodeVisitor.__expressionUsesVariable">__expressionUsesVariable</a></td> +<td>Private method to check, if a variable is used by an expression.</td> +</tr> +<tr> <td><a href="#SimplifyNodeVisitor.__getDuplicatedIsinstanceCall">__getDuplicatedIsinstanceCall</a></td> <td>Private method to get a list of isinstance arguments which could be combined.</td> </tr> @@ -229,7 +265,7 @@ </tr> <tr> <td><a href="#SimplifyNodeVisitor.__isConstantIncrease">__isConstantIncrease</a></td> -<td>Private method check the given expression for being a constant increase.</td> +<td>Private method to check an expression for being a constant increase.</td> </tr> <tr> <td><a href="#SimplifyNodeVisitor.__isExceptionCheck">__isExceptionCheck</a></td> @@ -252,6 +288,10 @@ <td>Private method negate the given Compare node.</td> </tr> <tr> +<td><a href="#SimplifyNodeVisitor.getOsPathJoinArgs">getOsPathJoinArgs</a></td> +<td></td> +</tr> +<tr> <td><a href="#SimplifyNodeVisitor.visit_Assign">visit_Assign</a></td> <td>Public method to process an Assign node.</td> </tr> @@ -288,6 +328,10 @@ <td>Public method to process an IfExp node.</td> </tr> <tr> +<td><a href="#SimplifyNodeVisitor.visit_Subscript">visit_Subscript</a></td> +<td>Public method to process a Subscript node.</td> +</tr> +<tr> <td><a href="#SimplifyNodeVisitor.visit_Try">visit_Try</a></td> <td>Public method to process a Try node.</td> </tr> @@ -320,6 +364,32 @@ callback function to register an error </dd> </dl> +<a NAME="SimplifyNodeVisitor.__bodyContainsContinue" ID="SimplifyNodeVisitor.__bodyContainsContinue"></a> +<h4>SimplifyNodeVisitor.__bodyContainsContinue</h4> +<b>__bodyContainsContinue</b>(<i>stmts</i>) + +<p> + Private method to check, if a list of statements contain a 'continue' statement. +</p> +<dl> + +<dt><i>stmts</i> (list of ast.stmt)</dt> +<dd> +list of statements +</dd> +</dl> +<dl> +<dt>Return:</dt> +<dd> +flag indicating a continue statement +</dd> +</dl> +<dl> +<dt>Return Type:</dt> +<dd> +bool +</dd> +</dl> <a NAME="SimplifyNodeVisitor.__check101" ID="SimplifyNodeVisitor.__check101"></a> <h4>SimplifyNodeVisitor.__check101</h4> <b>__check101</b>(<i>node</i>) @@ -616,6 +686,20 @@ reference to the AST node to be checked </dd> </dl> +<a NAME="SimplifyNodeVisitor.__check123" ID="SimplifyNodeVisitor.__check123"></a> +<h4>SimplifyNodeVisitor.__check123</h4> +<b>__check123</b>(<i>node</i>) + +<p> + Private method to check for complicated dictionary access with default value. +</p> +<dl> + +<dt><i>node</i> (ast.If)</dt> +<dd> +reference to the AST node to be checked +</dd> +</dl> <a NAME="SimplifyNodeVisitor.__check181" ID="SimplifyNodeVisitor.__check181"></a> <h4>SimplifyNodeVisitor.__check181</h4> <b>__check181</b>(<i>node</i>) @@ -901,6 +985,120 @@ reference to the AST node to be checked </dd> </dl> +<a NAME="SimplifyNodeVisitor.__check901" ID="SimplifyNodeVisitor.__check901"></a> +<h4>SimplifyNodeVisitor.__check901</h4> +<b>__check901</b>(<i>node</i>) + +<p> + Private method to check for unnecessary bool conversion. +</p> +<dl> + +<dt><i>node</i> (ast.Call)</dt> +<dd> +reference to the AST node to be checked +</dd> +</dl> +<a NAME="SimplifyNodeVisitor.__check904" ID="SimplifyNodeVisitor.__check904"></a> +<h4>SimplifyNodeVisitor.__check904</h4> +<b>__check904</b>(<i>node</i>) + +<p> + Private method to check for dictionary initialization. +</p> +<dl> + +<dt><i>node</i> (ast.Assign)</dt> +<dd> +reference to the AST node to be checked +</dd> +</dl> +<a NAME="SimplifyNodeVisitor.__check905" ID="SimplifyNodeVisitor.__check905"></a> +<h4>SimplifyNodeVisitor.__check905</h4> +<b>__check905</b>(<i>node</i>) + +<p> + Private method to check for list initialization by splitting a string. +</p> +<dl> + +<dt><i>node</i> (ast.Call)</dt> +<dd> +reference to the AST node to be checked +</dd> +</dl> +<a NAME="SimplifyNodeVisitor.__check906" ID="SimplifyNodeVisitor.__check906"></a> +<h4>SimplifyNodeVisitor.__check906</h4> +<b>__check906</b>(<i>node</i>) + +<p> + Private method to check for unnecessary nesting of os.path.join(). +</p> +<dl> + +<dt><i>node</i> (ast.Call)</dt> +<dd> +reference to the AST node to be checked +</dd> +</dl> +<a NAME="SimplifyNodeVisitor.__check907" ID="SimplifyNodeVisitor.__check907"></a> +<h4>SimplifyNodeVisitor.__check907</h4> +<b>__check907</b>(<i>node</i>) + +<p> + Private method to check for Union type annotation with None. +</p> +<dl> + +<dt><i>node</i> (ast.Subscript)</dt> +<dd> +reference to the AST node to be checked +</dd> +</dl> +<a NAME="SimplifyNodeVisitor.__check909" ID="SimplifyNodeVisitor.__check909"></a> +<h4>SimplifyNodeVisitor.__check909</h4> +<b>__check909</b>(<i>node</i>) + +<p> + Private method to check for reflexive assignments. +</p> +<dl> + +<dt><i>node</i> (ast.Assign)</dt> +<dd> +reference to the AST node to be checked +</dd> +</dl> +<a NAME="SimplifyNodeVisitor.__expressionUsesVariable" ID="SimplifyNodeVisitor.__expressionUsesVariable"></a> +<h4>SimplifyNodeVisitor.__expressionUsesVariable</h4> +<b>__expressionUsesVariable</b>(<i>expr, var</i>) + +<p> + Private method to check, if a variable is used by an expression. +</p> +<dl> + +<dt><i>expr</i> (ast.expr)</dt> +<dd> +expression node to be checked +</dd> +<dt><i>var</i> (str)</dt> +<dd> +variable name to be checked for +</dd> +</dl> +<dl> +<dt>Return:</dt> +<dd> +flag indicating the expression uses the variable +</dd> +</dl> +<dl> +<dt>Return Type:</dt> +<dd> +bool +</dd> +</dl> <a NAME="SimplifyNodeVisitor.__getDuplicatedIsinstanceCall" ID="SimplifyNodeVisitor.__getDuplicatedIsinstanceCall"></a> <h4>SimplifyNodeVisitor.__getDuplicatedIsinstanceCall</h4> <b>__getDuplicatedIsinstanceCall</b>(<i>node</i>) @@ -957,17 +1155,16 @@ </dl> <a NAME="SimplifyNodeVisitor.__isConstantIncrease" ID="SimplifyNodeVisitor.__isConstantIncrease"></a> <h4>SimplifyNodeVisitor.__isConstantIncrease</h4> -<b>__isConstantIncrease</b>(<i>expression</i>) +<b>__isConstantIncrease</b>(<i>expr</i>) <p> - Private method check the given expression for being a constant - increase. + Private method to check an expression for being a constant increase. </p> <dl> -<dt><i>expression</i> (ast.AugAssign)</dt> +<dt><i>expr</i> (ast.AugAssign)</dt> <dd> -reference to the expression node +reference to the node to be checked </dd> </dl> <dl> @@ -1124,6 +1321,10 @@ ast.Compare </dd> </dl> +<a NAME="SimplifyNodeVisitor.getOsPathJoinArgs" ID="SimplifyNodeVisitor.getOsPathJoinArgs"></a> +<h4>SimplifyNodeVisitor.getOsPathJoinArgs</h4> +<b>getOsPathJoinArgs</b>(<i></i>) + <a NAME="SimplifyNodeVisitor.visit_Assign" ID="SimplifyNodeVisitor.visit_Assign"></a> <h4>SimplifyNodeVisitor.visit_Assign</h4> <b>visit_Assign</b>(<i>node</i>) @@ -1250,6 +1451,20 @@ reference to the IfExp node </dd> </dl> +<a NAME="SimplifyNodeVisitor.visit_Subscript" ID="SimplifyNodeVisitor.visit_Subscript"></a> +<h4>SimplifyNodeVisitor.visit_Subscript</h4> +<b>visit_Subscript</b>(<i>node</i>) + +<p> + Public method to process a Subscript node. +</p> +<dl> + +<dt><i>node</i> (ast.Subscript)</dt> +<dd> +reference to the Subscript node +</dd> +</dl> <a NAME="SimplifyNodeVisitor.visit_Try" ID="SimplifyNodeVisitor.visit_Try"></a> <h4>SimplifyNodeVisitor.visit_Try</h4> <b>visit_Try</b>(<i>node</i>)