Skip to content

Pack of formal changes when dealing with Mat::data#3094

Merged
opencv-pushbot merged 2 commits intoopencv:masterfrom
Adil-Ibragimov:minor_fixes_in_Mat-data
Aug 13, 2014
Merged

Pack of formal changes when dealing with Mat::data#3094
opencv-pushbot merged 2 commits intoopencv:masterfrom
Adil-Ibragimov:minor_fixes_in_Mat-data

Conversation

@Adil-Ibragimov
Copy link
Copy Markdown
Contributor

Several type of formal refactoring when dealing with Mat::data

  1. someMatrix.data -> someMatrix.ptr()
  2. someMatrix.data + someMatrix.step * lineIndex -> someMatrix.ptr( lineIndex )
  3. (SomeType*) someMatrix.data -> someMatrix.ptr()
  4. someMatrix.data -> !someMatrix.empty() ( or !someMatrix.data -> someMatrix.empty() ) in logical expressions

1. someMatrix.data -> someMatrix.prt()
2. someMatrix.data + someMatrix.step * lineIndex -> someMatrix.ptr( lineIndex )
3. (SomeType*) someMatrix.data -> someMatrix.ptr<SomeType>()
4. someMatrix.data -> !someMatrix.empty() ( or !someMatrix.data -> someMatrix.empty() ) in logical expressions
@vpisarev vpisarev self-assigned this Aug 13, 2014
@vpisarev
Copy link
Copy Markdown
Contributor

👍

@opencv-pushbot opencv-pushbot merged commit 9ac06a9 into opencv:master Aug 13, 2014
@Adil-Ibragimov Adil-Ibragimov deleted the minor_fixes_in_Mat-data branch August 14, 2014 07:39
BKNio pushed a commit to BKNio/opencv that referenced this pull request Sep 24, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants