-
Notifications
You must be signed in to change notification settings - Fork 311
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[SofaKernel] Improve File:line info in error message (for python and xml error reporting) #314
[SofaKernel] Improve File:line info in error message (for python and xml error reporting) #314
Conversation
…he problematic file line number in the message.
[ci-build] |
[ci-build] |
Note: the build fail on OSX is due to API change in boost 1.64 that makes the Multithreading plugin fail to build. I would not pay attention to it until Multithreading is fixed. |
[ci-build] |
Fixed the conflict...let's rebuild |
Just a SOFA_FILE_INFO2 to replace with SOFA_FILE_INFO_COPIED_FROM in Binding_SofaModule.cpp |
great @damienmarchal , unfortunately ci is down for the day ! |
[ci-build] |
The CI finished but the dashboard is not up-to date. |
…ation [SofaKernel] Improve File:line info in error message (for python and xml error reporting) (cherry picked from commit 6e9ac71)
Currently when object are unable to be created there is no filename/line information on where the problem come from.
In this PR I implemented the missing <<operator in messaging.h to report those line number.
Now you can write:
To have the file name & line number associated in the message emitting information. If you don't the emitting information will be the source code file&line location.
If updated the xml load to take this feature into account.
The python loader is updated in the PR #304
This PR:
Reviewers will merge only if all these checks are true.