Flag CV_PCA_DATA_AS_COL missing in core.hpp (Bug #3874)

I also noted that many macros in the current documentation are still named with the prefix CV_ although they don't exist anymore (which is not the case for the macros of PCA).

Actually afaik the current doc for 2.4.9 should still mention macros with the CV_ constant before while the macro's will also be available without the CV_. However in the 3.0 dev branch all CV_ prefixes are removed. This is one of the changes I noticed during making pull requests.

Can you show some examples of macro's that are wrong to your interpretation in the docs?

I've meant the trunk-documentation at http://docs.opencv.org/trunk, and maybe my statement was too bold, not 'many' but 'several' are still wrong or wrong documented. For example search for CV_ in the calib3d module and you'll find some, e.g. the flags for findFundamentalMat() don't have the prefix CV_ anymore in the code, but are still listed as this in the documentation. Probably one will find some more examples here.

However, my bug-report above actually points to a different issue (sorry, I should have opened another bug-report for the wrong documentation): here the old CV_ notion still exists but is in a different header, so actually one only needs to define the new macros without CV_ and replace all of these occurences.

Okidoki! The second problem I am somewhat missing, however the first should indeed be adressed.

And are you sure that it is really not there, I still find some of the parameters defined in code, for example these:

https://github.com/Itseez/opencv/blob/master/modules/calib3d/include/opencv2/calib3d/calib3d_c.h#L79

Yes they are there, but these are the macros for the "old" C-Api, which apparantly should be retained as is. So yes, if you include calib3d_c.h then you are fine, but actually one shouldn't need to do that but use the new-style-macros without the CV_ -prefix.

Aw okay I see your point. Basically it should be possible to manually add those headers to get the c support but you dont want the standard description use the C style macros.

Yes, the non-'CV_'-prefixed versions are just missing, maybe this part of the code just hasn't been revised yet. Currently, I bypass this by including the *c.h headers to get the old 'CV' macros.

Update: the findFundamentalMat has been updated during a recent pr, so I am outdated here...

I am guessing this is the biggest downside to the changes made. 3.0 still needs a big update on the documentation to be reasonable to work with in my opinion.
Feel free to make suggestions using pull requests! ;)