Skip to content

Improve Conditional/Copy/Deinterleave/Fill function descriptions in SimdLib.hpp - #824

Merged
ermig1979 merged 1 commit into
devfrom
cursor/improve-conditional-copy-fill-docs-a3f4
Aug 5, 2026
Merged

Improve Conditional/Copy/Deinterleave/Fill function descriptions in SimdLib.hpp#824
ermig1979 merged 1 commit into
devfrom
cursor/improve-conditional-copy-fill-docs-a3f4

Conversation

@ermig1979

Copy link
Copy Markdown
Owner

Summary

Align C++ wrapper Doxygen descriptions in SimdLib.hpp with the improved C API wording in SimdLib.h for Conditional, Copy, Deinterleave, and Fill helpers, and record the updates in release 7.2.165 notes.

Improved functions

  • Simd::ConditionalSquareGradientSum, Simd::ConditionalFill
  • Simd::Copy, Simd::CopyFrame
  • Simd::DeinterleaveUv, Simd::DeinterleaveBgr, Simd::DeinterleaveBgra, Simd::DeinterleaveRgb, Simd::DeinterleaveRgba
  • Simd::Fill, Simd::FillFrame, Simd::FillBgr, Simd::FillBgra, Simd::FillPixel

Notable doc fixes

  • Clarified ConditionalSquareGradientSum border exclusion, gradient formula, and that sum is zero-initialized.
  • Clarified ConditionalFill leaves non-matching destination pixels unchanged.
  • Documented CopyFrame/FillFrame interior rectangle semantics ([left, right) x [top, bottom)).
  • Clarified Deinterleave* optional empty outputs (View::None) and per-channel extraction formulas.
  • Clarified Fill* padding behavior and corrected FillBgra default alpha wording (0xFF).

Files

  • src/Simd/SimdLib.hpp — documentation only
  • docs/2026.html — release 7.2.165 Documentation/Improving entries

No test suite run (documentation-only change, as requested).

Open in Web Open in Cursor 

…imdLib.hpp

Align C++ wrapper Doxygen comments with the improved C API wording and
record the documentation updates under release 7.2.165.

Co-authored-by: Ihar Yermalayeu <ermig1979@gmail.com>
@ermig1979
ermig1979 marked this pull request as ready for review August 5, 2026 13:07
@ermig1979
ermig1979 merged commit 18d05d4 into dev Aug 5, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants