fix INT128_MIN write

[belous-dp]

this PR fixes #1151

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.99%. Comparing base (41a6875) to head (39a57a4).
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1152      +/-   ##
==========================================
- Coverage   90.47%   89.99%   -0.48%     
==========================================
  Files           9        8       -1     
  Lines        2319     2288      -31     
==========================================
- Hits         2098     2059      -39     
- Misses        221      229       +8     

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

[nhz2]

It would be good to add a test with BigInt as well to avoid future regressions.

[belous-dp]

done

[nhz2]

Thank you for fixing this. I did a basic benchmark with:

using BenchmarkTools, CSV
@btime CSV.writecell($(zeros(UInt8,1000)), 1, 100, $(IOBuffer()), -1234567890, nothing)

And I don't see a major performance regression, so I think this is ready to be merged.

[quinnj]

Thanks! Sorry for hte slow review.