libc: only append exterr herald and text if exterr was recorded
ClosedPublic
Actions

Authored by kib on Jul 8 2025, 2:05 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Apr 5, 6:24 AM

	Unknown Object (File)
	Fri, Apr 3, 3:09 PM

	Unknown Object (File)
	Mon, Mar 23, 7:45 AM

	Unknown Object (File)
	Mon, Mar 23, 7:30 AM

	Unknown Object (File)
	Fri, Mar 13, 9:47 AM

	Unknown Object (File)
	Wed, Mar 11, 12:57 AM

	Unknown Object (File)
	Tue, Mar 10, 5:29 PM

	Unknown Object (File)
	Tue, Mar 10, 12:04 AM

View All 51 Files

Subscribers

imp

Details

Reviewers

markj
asomers
brooks

Commits

rG72c848089f3b: libc: only append exterr herald and text if exterr was recorded

Summary

__uexterr_format: return empty string on no error, instead of "No error"
err(): check for the first symbol of the exterr to decide if there is
anything to print.

Requested by:   markj

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kib created this revision.Jul 8 2025, 2:05 PM

Herald added a subscriber: imp. · View Herald TranscriptJul 8 2025, 2:05 PM

kib requested review of this revision.Jul 8 2025, 2:05 PM

markj accepted this revision.Jul 8 2025, 3:45 PM

This revision is now accepted and ready to land.Jul 8 2025, 3:45 PM

I verified that the test failures are gone now, thank you.

Closed by commit rG72c848089f3b: libc: only append exterr herald and text if exterr was recorded (authored by kib). · Explain WhyJul 8 2025, 3:56 PM

This revision was automatically updated to reflect the committed changes.

kib added a commit: rG72c848089f3b: libc: only append exterr herald and text if exterr was recorded.

Revision Contents
Changeset List

Path

Size

lib/

libc/

gen/

err.c

2 lines

uexterr_format.c

2 lines

Diff 158169

View Options

lib/libc/gen/err.c