Skip to content

Do not use deprecated ReleaseCleared in protobuf library.#20609

Merged
opencv-pushbot merged 1 commit intoopencv:3.4from
vrabaud:3.4
Aug 27, 2021
Merged

Do not use deprecated ReleaseCleared in protobuf library.#20609
opencv-pushbot merged 1 commit intoopencv:3.4from
vrabaud:3.4

Conversation

@vrabaud
Copy link
Copy Markdown
Contributor

@vrabaud vrabaud commented Aug 26, 2021

This is to make code work with arenas (ReleaseCleared is incompatible).
The cleaning of the memory is also simpler.

ReleaseCleared not working with arenas is described here:
https://developers.google.com/protocol-buffers/docs/reference/cpp/google.protobuf.repeated_field#RepeatedPtrField.ReleaseCleared.details
Arenas are described here:
https://developers.google.com/protocol-buffers/docs/reference/arenas

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.
  • [ x 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

This is to make code work with protobuf arenas for memory
management (ReleaseCleared is incompatible).
The cleaning of the memory is also simpler.
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.

Thank you 👍

@opencv-pushbot opencv-pushbot merged commit ea7f139 into opencv:3.4 Aug 27, 2021
@alalek alalek mentioned this pull request Aug 28, 2021
@alalek alalek mentioned this pull request Oct 15, 2021
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