Skip to content

remove redundant semicolons#20772

Merged
opencv-pushbot merged 1 commit intoopencv:masterfrom
shengyu7697:remove-redundant-semicolon
Oct 4, 2021
Merged

remove redundant semicolons#20772
opencv-pushbot merged 1 commit intoopencv:masterfrom
shengyu7697:remove-redundant-semicolon

Conversation

@shengyu7697
Copy link
Copy Markdown
Contributor

@shengyu7697 shengyu7697 commented Sep 29, 2021 •
edited
Loading

Pull Request Readiness Checklist

See details at https://github.com/opencv/opencv/wiki/How_to_contribute#making-a-good-pull-request

  • I agree to contribute to the project under Apache 2 License.
  • To the best of my knowledge, the proposed patch is not based on a code under GPL or other license that is incompatible with OpenCV
  • The PR is proposed to proper branch
  • There is reference to original bug report and related work
  • There is accuracy test, performance test and test data in opencv_extra repository, if applicable
    Patch to opencv_extra has the same branch name.
  • The feature is well documented and sample code can be built with the project CMake

TolyaTalamanov
TolyaTalamanov reviewed Sep 30, 2021
View reviewed changes
TolyaTalamanov
TolyaTalamanov reviewed Sep 30, 2021
View reviewed changes
TolyaTalamanov
TolyaTalamanov reviewed Sep 30, 2021
View reviewed changes
TolyaTalamanov
TolyaTalamanov approved these changes Sep 30, 2021
View reviewed changes
Copy link
Copy Markdown
Contributor

@TolyaTalamanov TolyaTalamanov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

Could I ask you also remove extract space before {} ?

@shengyu7697 shengyu7697 force-pushed the remove-redundant-semicolon branch from 1affa5d to 4d87f60 Compare October 2, 2021 01:36
@shengyu7697
Copy link
Copy Markdown
Contributor Author

shengyu7697 commented Oct 2, 2021 •
edited
Loading

Ok, I removed extra spaces before {}

TolyaTalamanov
TolyaTalamanov approved these changes Oct 4, 2021
View reviewed changes
Copy link
Copy Markdown
Contributor

@TolyaTalamanov TolyaTalamanov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! 👍

@opencv-pushbot opencv-pushbot merged commit 5950259 into opencv:master Oct 4, 2021
@alalek alalek mentioned this pull request Oct 15, 2021
Merged
@alalek alalek mentioned this pull request Oct 27, 2021
Merged
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@TolyaTalamanov TolyaTalamanov TolyaTalamanov approved these changes

Assignees

@TolyaTalamanov TolyaTalamanov

Labels

category: g-api / gapi

Projects

None yet

Milestone

4.5.4

Development

Successfully merging this pull request may close these issues.

4 participants

@shengyu7697 @TolyaTalamanov @opencv-pushbot @alalek