Skip to content

Local objects optimization in calibration init#19565

Merged
alalek merged 3 commits intoopencv:masterfrom
cyyever:minor_fix
Mar 5, 2021
Merged

Local objects optimization in calibration init#19565
alalek merged 3 commits intoopencv:masterfrom
cyyever:minor_fix

Conversation

@cyyever
Copy link
Copy Markdown
Contributor

@cyyever cyyever commented Feb 19, 2021

No description provided.

@asmorkalov asmorkalov changed the title Minor fix Local objects optimization in calibration init Feb 19, 2021
@asmorkalov
Copy link
Copy Markdown
Contributor

@cyyever Please pay attention on CI status. BuildBot found trailing white spaces in the code.

@asmorkalov
Copy link
Copy Markdown
Contributor

@@cyyever Friendly reminder.

@cyyever
Copy link
Copy Markdown
Contributor Author

cyyever commented Mar 3, 2021

@@cyyever Friendly reminder.

I removed the tailing space.

Copy link
Copy Markdown
Member

@alalek alalek left a comment

Choose a reason for hiding this comment

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

Looks good to me! Thank you for contribution 👍

@alalek alalek merged commit b450dd7 into opencv:master Mar 5, 2021
@cyyever cyyever deleted the minor_fix branch March 5, 2021 15:06
sujitahirrao added a commit to sujitahirrao/opencv that referenced this pull request Mar 6, 2021
Merge pull request opencv#19565 from cyyever:minor_fix
@alalek alalek mentioned this pull request Apr 9, 2021
a-sajjad72 pushed a commit to a-sajjad72/opencv that referenced this pull request Mar 30, 2023
Local objects optimization in calibration init

* use emplace_back

* use {} initilization

* remove a tailing white space
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants