Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

two small fixes (backport #2601) #2611

Merged
merged 4 commits into from
May 10, 2023
Merged

two small fixes (backport #2601) #2611

merged 4 commits into from
May 10, 2023

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented May 10, 2023

This is an automatic backport of pull request #2601 done by Mergify.


Mergify commands and options

More conditions and actions can be found in the documentation.

You can also trigger Mergify actions by commenting on this pull request:

  • @Mergifyio refresh will re-evaluate the rules
  • @Mergifyio rebase will rebase this PR on its base branch
  • @Mergifyio update will merge the base branch into this PR
  • @Mergifyio backport <destination> will backport this PR on <destination> branch

Additionally, on Mergify dashboard you can:

  • look at your merge queues
  • generate the Mergify configuration with the config editor.

Finally, you can contact us on https://mergify.com

neheb added 4 commits May 10, 2023 12:10
Allows the compiler to actually diagnose formats.

Signed-off-by: Rosen Penev <rosenp@gmail.com>
(cherry picked from commit 37184fd)
Typically done with noexcept.

Signed-off-by: Rosen Penev <rosenp@gmail.com>
(cherry picked from commit 4c66b44)
Signed-off-by: Rosen Penev <rosenp@gmail.com>
(cherry picked from commit 954153a)
Signed-off-by: Rosen Penev <rosenp@gmail.com>
(cherry picked from commit c319699)
@mergify mergify bot mentioned this pull request May 10, 2023
@ghost
Copy link

ghost commented May 10, 2023

👇 Click on the image for a new way to code review

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map legend

@kmilos kmilos requested a review from neheb May 10, 2023 12:15
@codecov
Copy link

codecov bot commented May 10, 2023

Codecov Report

Merging #2611 (18ea9f5) into 0.28.x (cdcfac7) will increase coverage by 0.00%.
The diff coverage is 36.00%.

@@           Coverage Diff           @@
##           0.28.x    #2611   +/-   ##
=======================================
  Coverage   63.92%   63.93%           
=======================================
  Files         103      103           
  Lines       22309    22296   -13     
  Branches    10795    10787    -8     
=======================================
- Hits        14262    14255    -7     
+ Misses       5825     5823    -2     
+ Partials     2222     2218    -4     
Impacted Files Coverage Δ
src/jpgimage.cpp 68.41% <ø> (ø)
src/rafimage.cpp 21.13% <0.00%> (ø)
src/safe_op.hpp 96.66% <ø> (ø)
src/enforce.hpp 85.71% <75.00%> (+20.71%) ⬆️
src/bmffimage.cpp 72.56% <100.00%> (ø)
src/image_int.hpp 100.00% <100.00%> (ø)
src/jp2image.cpp 70.47% <100.00%> (ø)

@neheb neheb merged commit 54700c4 into 0.28.x May 10, 2023
110 checks passed
@neheb neheb deleted the mergify/bp/0.28.x/pr-2601 branch May 10, 2023 13:27
@kevinbackhouse kevinbackhouse added this to the v0.28.1 milestone Nov 4, 2023
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.

None yet

2 participants