Fix docstrings

m-kuhn · m-kuhn · commit c30d7b98d73d · 2017-05-02T19:43:29.000+02:00
diff --git a/src/core/qgsexpressioncontext.h b/src/core/qgsexpressioncontext.h
@@ -126,6 +126,7 @@ class CORE_EXPORT QgsExpressionContextScope
        * \param name variable name (should be unique within the QgsExpressionContextScope)
        * \param value initial variable value
        * \param readOnly true if variable should not be editable by users
+       * \param isStatic true if the variable will not change during the lifteime of an iterator.
        */
       StaticVariable( const QString &name = QString(), const QVariant &value = QVariant(), bool readOnly = false, bool isStatic = false )
         : name( name )
@@ -164,10 +165,10 @@ class CORE_EXPORT QgsExpressionContextScope
      */
     QString name() const { return mName; }
 
-    /** Convenience method for setting a variable in the context scope by name and value. If a variable
+    /** Convenience method for setting a variable in the context scope by \a name name and \a value. If a variable
      * with the same name is already set then its value is overwritten, otherwise a new variable is added to the scope.
-     * \param name variable name
-     * \param value variable value
+     * If the \a isStatic parameter is set to true, this variable can be cached during the execution
+     * of QgsExpression::prepare().
      * \see addVariable()
      */
     void setVariable( const QString &name, const QVariant &value, bool isStatic = false );
diff --git a/tests/src/core/testqgsexpressioncontext.cpp b/tests/src/core/testqgsexpressioncontext.cpp
@@ -111,8 +111,14 @@ class TestQgsExpressionContext : public QObject
           return new ModifiableFunction( mVal );
         }
 
+        /**
+         * This function is not static, it's value changes with every invocation.
+         */
         virtual bool isStatic( const QgsExpression::NodeFunction *node, QgsExpression *parent, const QgsExpressionContext *context ) const override
         {
+          Q_UNUSED( node )
+          Q_UNUSED( parent )
+          Q_UNUSED( context )
           return false;
         }
 

Original file line number	Diff line number	Diff line change
`@@ -111,8 +111,14 @@ class TestQgsExpressionContext : public QObject`
`111`	`111`	`return new ModifiableFunction( mVal );`
`112`	`112`	`}`
`113`	`113`
	`114`	`+ /**`
	`115`	`+ * This function is not static, it's value changes with every invocation.`
	`116`	`+ */`
`114`	`117`	`virtual bool isStatic( const QgsExpression::NodeFunction node, QgsExpression parent, const QgsExpressionContext *context ) const override`
`115`	`118`	`{`
	`119`	`+ Q_UNUSED( node )`
	`120`	`+ Q_UNUSED( parent )`
	`121`	`+ Q_UNUSED( context )`
`116`	`122`	`return false;`
`117`	`123`	`}`
`118`	`124`