Differential D20122

Use D_PARTISGPT rather than bare 255
ClosedPublic
Actions

Authored by imp on May 1 2019, 5:33 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, May 17, 12:49 PM

	Unknown Object (File)
	Sun, May 17, 12:03 AM

	Unknown Object (File)
	Thu, May 14, 4:26 AM

	Unknown Object (File)
	Fri, May 8, 6:37 AM

	Unknown Object (File)
	Apr 17 2026, 10:04 PM

	Unknown Object (File)
	Apr 14 2026, 5:27 PM

	Unknown Object (File)
	Apr 14 2026, 11:24 AM

	Unknown Object (File)
	Apr 13 2026, 10:39 PM

View All Files

Subscribers

None

Details

Reviewers

tsoome
• ian
kevans
bcran

Commits

rS346984: Use D_PARTISGPT rather than bare 255

Summary

These three cases dovetail with other places in the code where we use
or set D_PARTISGPT when we mean that the partitioning scheme is
GPT. Use this #define to make the code easier to undertand.

NOTE: My reading of grep -r 255 src/stand says I caught them all. Prove me wrong.

Bonus question: Do we use a different #define that happens to be 255 when we mean this one. I don't think so.

Test Plan

md5 says the .o is the same before and afer :).

Diff Detail

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 24031
Build 22917: arc lint + arc unit

Event Timeline

imp created this revision.May 1 2019, 5:33 AM

Harbormaster completed remote builds in B24031: Diff 56899.May 1 2019, 5:33 AM

imp added reviewers: tsoome, • ian, kevans, bcran.May 1 2019, 5:34 AM

LGTM

This revision is now accepted and ready to land.May 1 2019, 5:35 AM

imp edited the summary of this revision. (Show Details)May 1 2019, 5:36 AM

Closed by commit rS346984: Use D_PARTISGPT rather than bare 255 (authored by imp). · Explain WhyMay 1 2019, 5:42 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

stand/

common/

disk.c

6 lines

Diff 56899

View Options

Use D_PARTISGPT rather than bare 255ClosedPublicActions