reformatting in more files

[badasahog]

As far as I can tell these files haven't been touched in a while, so there shouldn't be any major conflicts.

Only one change was functional (switching to memset). Other than that, everything here is purely cosmetic.

[codecov]

Codecov Report

Attention: Patch coverage is 97.56098% with 2 lines in your changes missing coverage. Please review.

Project coverage is 98.77%. Comparing base (e0d40b1) to head (06805a5).
Report is 1 commits behind head on master.

Files with missing lines	Patch %	Lines
src/openmp-utils.c	90.90%	2 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #7198   +/-   ##
=======================================
  Coverage   98.77%   98.77%           
=======================================
  Files          81       81           
  Lines       15212    15212           
=======================================
  Hits        15026    15026           
  Misses        186      186           

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

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[github-actions]

• HEAD=ReformattingAndCleanup stopped early for DT[by,verbose=TRUE] improved in #6296
  

Generated via commit 06805a5

Download link for the artifact containing the test results: ↓ atime-results.zip

Task	Duration
R setup and installing dependencies	2 minutes and 44 seconds
Installing different package versions	39 seconds
Running and plotting the test cases	2 minutes and 35 seconds

[MichaelChirico]

Hi @badasahog I would prefer to focus on the new segfault #7183. We'll have to revert the breaking change if the new issue can't be fixed.

[badasahog]

I think that's in a separate part of the code?

Also, do you have a minimal example I could use to recreate the segfault?

All tests cleared on my end.

[MichaelChirico]

Did you test under ASAN? The issue is shown clearly in the logs linked in the issue. Let's please continue the discussion there.

[badasahog]

now that that's done, is this ready to go?

[MichaelChirico]

Revert the changes to nafill.c and uniqlist.c which might conflict with pending PRs: #5692, #5682, #5576, #5575, #4372

[badasahog]

are there any other files that don't have outstanding PRs?

[MichaelChirico]

You can check: https://gist.github.com/MichaelChirico/64e981b4359a36d2279bf55c39c7d3dd