Page MenuHomeFreeBSD

atf: Guard against multiple evaluation.
ClosedPublic

Authored by des on May 10 2024, 6:12 AM.
Tags
None
Referenced Files
F86146237: D45148.diff
Sun, Jun 16, 3:04 AM
Unknown Object (File)
Thu, May 23, 8:10 PM
Unknown Object (File)
Sun, May 19, 8:38 PM
Unknown Object (File)
May 17 2024, 4:31 AM
Unknown Object (File)
May 15 2024, 10:12 AM
Unknown Object (File)
May 13 2024, 9:12 AM
Unknown Object (File)
May 13 2024, 6:56 AM
Unknown Object (File)
May 12 2024, 9:17 PM
Subscribers

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 57651
Build 54539: arc lint + arc unit

Event Timeline

des requested review of this revision.May 10 2024, 6:12 AM

BTW, ATF is under FreeBSD org at GitHub: https://github.com/freebsd/atf , we an fix there and import to base.

Cool, feel free to open an issue there, I don't want to have to wait weeks or months for a fix.

ngie added a subscriber: ngie.
ngie added inline comments.
contrib/atf/atf-c/macros.h
174

_(picking a line)_
Should this macro expansion use #_atf_expected, #_atf_actual instead of #expected, #actual? It shouldn't matter, except with non standards conforming C/C++ toolchains.

This revision is now accepted and ready to land.May 14 2024, 10:33 PM
des marked an inline comment as done.May 15 2024, 10:07 AM
des added inline comments.
contrib/atf/atf-c/macros.h
174

No, that wouldn't work since _atf_expected and _atf_actual aren't macro parameters, and I don't see why you'd want to.

This revision was automatically updated to reflect the committed changes.
des marked an inline comment as done.