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

Fix clang-tidy warning in indexed_bounds_check_fn::operator() #178

Merged
merged 1 commit into from
Mar 14, 2024

Conversation

braxtons12
Copy link
Contributor

Fix use-after-move of loc

Closes #176

Copy link

codecov bot commented Mar 14, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.06%. Comparing base (8ae438b) to head (702f989).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #178   +/-   ##
=======================================
  Coverage   98.06%   98.06%           
=======================================
  Files          69       69           
  Lines        2426     2426           
=======================================
  Hits         2379     2379           
  Misses         47       47           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@tcbrindle
Copy link
Owner

It looks like our Asan CI test has suddenly started timing out for some reason (it was fine yesterday...), but this PR looks good to me. Thanks @braxtons12!

@tcbrindle tcbrindle merged commit cfa01fc into tcbrindle:main Mar 14, 2024
30 of 31 checks passed
@braxtons12
Copy link
Contributor Author

braxtons12 commented Mar 14, 2024

It looks like our Asan CI test has suddenly started timing out for some reason (it was fine yesterday...), but this PR looks good to me. Thanks @braxtons12!

I saw that. I don't think I've ever actually seen a bad_alloc in the wild, really curious.
Given this was removing an instance of what could be argued to be UB, my "knee jerk" assumption would be that an optimization is no longer happening, and that's causing it to stall.
But given the Asan build is a debug build and UBsan builds passed before this, that rules that out.
Really curious.

@tcbrindle
Copy link
Owner

Yeah, I'm mystified.

As a check, I re-ran the Asan build with on the main branch -- which passed when I ran it yesterday! -- and it also timed out. So it's definitely not this PR :)

What's really weird is that sometimes the check will hang whilst in the building phase (specifically, linking libcatch) and sometimes it will make it as far as testing before hanging and eventually terminating with a bad_alloc.

I really don't understand what's going on, so I've just disabled the sanitizer builds for now and hopefully it'll mysteriously start working again in a couple of days! 🤷🏻‍♂️

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.

move of loc in assert.hpp causes clang-tidy error
2 participants