Added all *.cpp samples reference to the documentation#1206
Added all *.cpp samples reference to the documentation#1206opencv-pushbot merged 1 commit intoopencv:2.4from StevenPuttemans:feature_3169
Conversation
|
@StevenPuttemans please fix whitespace problems |
|
@apavlenko will do so later today 👍 |
|
Steven, you did a great job! Thanks! |
|
Pushed the fix! |
|
It seems still some whitespace were left :) must have missed two. Fixed! @apavlenko it seems that there are 20 warnings about a label matrixexpressions being missing. /home/user/slave/builds/precommit_docs/src/opencv/modules/core/doc/basic_structures.rst:274: WARNING: undefined label: matrixexpressions (if the link has no caption the label must precede a section header) Should I remove them? Because I have no idea to what it should point or how I can add a fix for this! EDIT: looking deeper into the problem, it seems that the doc can work with matrixexpressions, but it should in fact be matrix-expressions I am guessing. Will look into docs for it. |
|
I guess it was my own fault, by adding a sample tag behind the link. We will see if moving it one up will have fixed the linking. |
|
It would also better if you use linux style slashes, they work on most OSes |
|
Can you give an example of what you mean by that? |
|
opencv_source_code/samples/cpp/3calibration.cpp |
|
ok but I havent defined it as link, so it wont be clickable? Is it worth adapting all the files then? |
|
universal slashes allow copy-paste path to console or so... |
|
Hmm guess you have a point there :) Will update them today. |
…ssed in feature #3169 - Used lunix style slashes - works on most systems - Removed all trailing whitespaces
|
Added linux style slashes. Considered it is also usefull to not create errors, since it is also used as escape character in LaTeX styled documents besides easy copying to systems. Merged all commits into single usefull one. |
|
👍 |
|
👍 |
|
@StevenPuttemans Steven, I've just noticed that you used some unknown ReST directive: |
|
👍 hehe too bad :) I was expecting it to except every kind of form you want. Will do it later today probably. |
|
It's |
Added a link in documentation pointing to samples containing the functionality.
Did it only for the samples that have functionality in the 2.4 branch.
Suggestion was made in this feature request:
http://code.opencv.org/issues/3169
If this works out and gets merged, will consider doing same for other sample folders also.
Also want to know if it is usefull to add C samples, since we want to demotivate people using the interface?