Now all escaping required for the C++ code generation is done when the
.cpp/.h files are generated. Previously, only newlines were escaped
automatically. This was a) inconsistent and b) leaked c++ details into
the xml data.
In addition, the escaping is now done in one central place, harmonizing
the three previous implementations.
Pre-existing c++ escape sequences in the XML files have been replaced by
their literal equivalent so that the resulting python doc sting remains
unchanged.
It's not recommended any more to keep the export macro in the PreCompiled.h because many IDEs fail to fetch the export macro and raise a parsing error instead.
A regression from #9506 where `self.multiply` was removed but not all references to it were updated.
I'm not a Python guy but from what I can see the behaviour is the same - `offsetvector` is still an `App.Vector` which `self.multiply` used to use as well.
Tested with a project that previously didn't work, and now shows a nice `Arc`-type dressup in a circular pocket.
This is a rename that got missed in #6637. I ran into this bug when trying to post a job using the `linuxcnc` post processor on a Windows 11 machine, running the weekly build titled `FreeCAD_weekly-builds-33576-2023-07-13-conda-Windows-x86_64-py310.7z`.
While working on https://github.com/FreeCAD/FreeCAD/pull/9867 I noticed my patch
showed up with a lot of linting issues in code I did not touch, to a point
where the view was very cluttered by lintian issues. Here is my second
try to reduce the number of issues discovered by the linter. Some
issues are still left, as I fail to see how to sensibly reduce the number of
parameters or local variable used.
The issue only happen when splitting jobs on tools (orderby == Tool), and when
USE_TLO was active and the preamble include G49. The first move is then done
before tool height is set, and can cause damage if the existing tool height is set
to more than the gap between the spindle and the table or work piece, when the machine
take a sudden dive straight down.
Removed move between G49 and first G43, to ensure all moves are done after G43
correctly set tool height compensation.
Rewrote code to introduce new method fixtureSetup() to ensure all orderby alternatives
behave the same way.
Fixes#9866.
While working on https://github.com/FreeCAD/FreeCAD/pull/9867 I noticed my patch
showed up with a lot of linting issues in code I did not touch, to a point
where the view was very cluttered by lintian issues. Here is my
try to reduce the number of issues discovered by the linter. Some
issues are left, as I fail to see how to sensibly reduce the number of
parameters or local variable used.
