This is an archive of the discontinued LLVM Phabricator instance.

[flang][unittests] Fix recent snprintf() changes to use correct buffer lengths
ClosedPublic

Authored by klausler on Jul 13 2023, 11:04 AM.

Details

Summary

A recent patch replaced two sprintf() calls with snprintf(), but didn't
use the right length for the remaining space in the buffer.

Diff Detail

Event Timeline

klausler created this revision.Jul 13 2023, 11:04 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 13 2023, 11:04 AM
klausler requested review of this revision.Jul 13 2023, 11:04 AM
klausler updated this revision to Diff 541204.Jul 17 2023, 1:12 PM

clang-format

cseo accepted this revision.Jul 18 2023, 10:26 AM

LGTM.

This revision is now accepted and ready to land.Jul 18 2023, 10:26 AM