diff options
| author | VinÃcius dos Santos Oliveira <vini.ipsmaker@gmail.com> | 2014-03-14 04:21:08 +0000 |
|---|---|---|
| committer | VinÃcius dos Santos Oliveira <vini.ipsmaker@gmail.com> | 2014-03-14 04:21:08 +0000 |
| commit | ed6364b83dafb3d1b9cbedfa6675817833e9fdeb (patch) | |
| tree | 2ea82b23e6ea0809f93de68611e60f97d80d1b6d /src | |
| parent | Small code simplification (diff) | |
| download | inkscape-ed6364b83dafb3d1b9cbedfa6675817833e9fdeb.tar.gz inkscape-ed6364b83dafb3d1b9cbedfa6675817833e9fdeb.zip | |
Adding default argument to PathManipulator::isBSpline.
Default argument is used when function is called with missing arguments.
The default was chosen to 'true', because it is the safest value in this
context.
From now on, the function can be called like "_pm.isBSpline()", which is
more intuitive.
Still, I haven't studied the code enough to determine if this cache
behaviour (force the user to think if cache is correct) is appropriate.
(bzr r11950.7.2)
Diffstat (limited to 'src')
| -rw-r--r-- | src/ui/tool/path-manipulator.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/ui/tool/path-manipulator.h b/src/ui/tool/path-manipulator.h index 5186bfc95..594f56ff9 100644 --- a/src/ui/tool/path-manipulator.h +++ b/src/ui/tool/path-manipulator.h @@ -106,7 +106,7 @@ private: typedef boost::shared_ptr<NodeList> SubpathPtr; void _createControlPointsFromGeometry(); - bool isBSpline(bool recalculate); + bool isBSpline(bool recalculate = true); double BSplineHandlePosition(Handle *h); Geom::Point BSplineHandleReposition(Handle *h); Geom::Point BSplineHandleReposition(Handle *h,double pos); |