igorcoroli / javacv

Automatically exported from code.google.com/p/javacv
GNU General Public License v2.0
0 stars 0 forks source link

Cvmat isContinuous(), total() method missing #215

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Both of that methods are missing from bindings. Would be nice to add them.

Original issue reported on code.google.com by gorud...@gmail.com on 8 Jul 2012 at 11:54

GoogleCodeExporter commented 8 years ago
Please read http://code.google.com/p/javacv/wiki/WhereIsFeatureXOfOpenCV

Original comment by samuel.a...@gmail.com on 9 Jul 2012 at 1:19

GoogleCodeExporter commented 8 years ago
FYI, `total() == rows * cols` and `isContinous() == (step == 0 || step == cols 
* CV_ELEM_SIZE(type))`, so just use that. IMO, the C++ API has too much fluff 
and is too badly designed to bother.

Original comment by samuel.a...@gmail.com on 14 Jul 2012 at 2:23

GoogleCodeExporter commented 8 years ago
Now that you mention it though, it looks like OpenCV decided to add some 
depth(), channels(), elemSize() functions similar to what I had already added 
in JavaCV, so I guess I'll be adding isContinuous() and total() as well, just 
to be a bit more consistent... thanks for pointing this out!

Original comment by samuel.a...@gmail.com on 16 Jul 2012 at 2:29

GoogleCodeExporter commented 8 years ago
No problem, if I will see more issues I'll report. Thanks for fixing them.

Original comment by gorud...@gmail.com on 17 Jul 2012 at 6:27