Paths

Table of Contentst

Differential D45148

atf: Guard against multiple evaluation.
ClosedPublic
Actions

Authored by des on May 10 2024, 6:12 AM.

Tags

None

Referenced Files

	F113065149: D45148.diff
	Wed, Mar 26, 2:15 AM

	F113055853: D45148.diff
	Tue, Mar 25, 11:33 PM

	Unknown Object (File)
	Sat, Mar 22, 6:49 AM

	Unknown Object (File)
	Sat, Mar 15, 8:54 AM

	Unknown Object (File)
	Sat, Mar 15, 12:22 AM

	Unknown Object (File)
	Fri, Mar 14, 11:14 PM

	Unknown Object (File)
	Fri, Mar 14, 8:37 PM

	Unknown Object (File)
	Fri, Mar 14, 6:22 PM

View All 38 Files

Subscribers

Details

Reviewers

kevans
ngie

Group Reviewers

Commits

rG0829f9334cdd: atf: Guard against multiple evaluation.
rGd16bc4fda821: atf: Guard against multiple evaluation.
rG745ee870031e: atf: Guard against multiple evaluation.
rGa7beca6fb113: atf: Guard against multiple evaluation.

Summary

Note that the ATF-C++ macros have the same issue, but they are not as
easily fixed.

MFC after: 3 days

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

des created this revision.May 10 2024, 6:12 AM

Herald added a subscriber: imp. · View Herald TranscriptMay 10 2024, 6:12 AM

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

Harbormaster completed remote builds in B57651: Diff 138333.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 accepted this revision.May 14 2024, 10:33 PM

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.

Closed by commit rGa7beca6fb113: atf: Guard against multiple evaluation. (authored by des). · Explain WhyMay 15 2024, 10:09 AM

This revision was automatically updated to reflect the committed changes.

des marked an inline comment as done.

des added a commit: rGa7beca6fb113: atf: Guard against multiple evaluation..

des added a commit: rG745ee870031e: atf: Guard against multiple evaluation..May 20 2024, 9:09 AM

des added a commit: rGd16bc4fda821: atf: Guard against multiple evaluation..May 20 2024, 9:35 AM

des added a commit: rG0829f9334cdd: atf: Guard against multiple evaluation..May 20 2024, 8:38 PM

Revision Contents
Changeset List

Path

Size

contrib/

atf/

atf-c/

151 lines

Diff 138569

contrib/atf/atf-c/macros.h

Loading...