Skip to content

core(persistence): base64 tests#15058

Merged
opencv-pushbot merged 4 commits intoopencv:masterfrom
alalek:core_fix_base64_packed_struct_master
Jul 18, 2019
Merged

core(persistence): base64 tests#15058
opencv-pushbot merged 4 commits intoopencv:masterfrom
alalek:core_fix_base64_packed_struct_master

Conversation

@alalek
Copy link
Copy Markdown
Member

@alalek alalek commented Jul 16, 2019

port into master #15050

relates #14977

Part of tests are disabled due missing base64 encoders (+json base64 decoder).

allow_multiple_commits=1

Usually it is more convenient to use operator `>>` instead of this method.
@param fmt Specification of each array element. See @ref format_spec "format specification"
@param vec Pointer to the destination array.
@param len Number of bytes to read (buffer size limit). If it is greater than number of
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
@param len Number of bytes to read (buffer size limit). If it is greater than number of
@param len Number of bytes to read (buffer size limit). If it is greater than size of

Usually it is more convenient to use operator `>>` instead of this method.
@param fmt Specification of each array element. See @ref format_spec "format specification"
@param vec Pointer to the destination array.
@param len Number of bytes to read (buffer size limit). If it is greater than number of
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
@param len Number of bytes to read (buffer size limit). If it is greater than number of
@param len Number of bytes to read (buffer size limit). If it is greater than size of

@opencv-pushbot opencv-pushbot merged commit d4501f0 into opencv:master Jul 18, 2019
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