Skip to content

Add ONVIF SEI as a character type for future usage#397

Merged
mithydolphin merged 7 commits intomasterfrom
add_onvif_sei_type_as_char
Mar 3, 2025
Merged

Add ONVIF SEI as a character type for future usage#397
mithydolphin merged 7 commits intomasterfrom
add_onvif_sei_type_as_char

Conversation

@mithydolphin
Copy link
Contributor

Introduced 'O' as a character identifier for ONVIF SEI in the SEI type handler.

Change-Id: I5ef7c34deaee910fdb561a32ce488438e4aa04fc

Describe your changes

Please include a summary of the change, a relevant motivation and context.

Issue ticket number and link

  • Fixes #(issue)

Checklist before requesting a review

  • I have performed a self-review of my own code
  • I have verified that the code builds perfectly fine on my local system
  • I have added tests that prove my fix is effective or that my feature works
  • I have commented my code, particularly in hard-to-understand areas
  • I have verified that my code follows the style already available in the repository
  • I have made corresponding changes to the documentation

Introduced 'O' as a character identifier for ONVIF SEI in the SEI type handler.

Change-Id: I5ef7c34deaee910fdb561a32ce488438e4aa04fc
@mithydolphin mithydolphin force-pushed the add_onvif_sei_type_as_char branch from 6326cbb to e7ec125 Compare February 26, 2025 16:03
Copy link
Contributor

@bjornvolcker bjornvolcker left a comment

Choose a reason for hiding this comment

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

Much better!

Copy link

@lusikamalo2 lusikamalo2 left a comment

Choose a reason for hiding this comment

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

OK.

@mithydolphin mithydolphin merged commit 08fee34 into master Mar 3, 2025
3 checks passed
@mithydolphin mithydolphin deleted the add_onvif_sei_type_as_char branch March 18, 2025 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

3 participants