umass: Simplify umass_std_transform to eliminate fake success
Needs ReviewPublic
Actions

Authored by imp on Sun, Mar 23, 9:50 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, Mar 31, 4:47 AM

	Unknown Object (File)
	Mon, Mar 31, 4:47 AM

	Unknown Object (File)
	Mon, Mar 31, 4:47 AM

	Unknown Object (File)
	Mon, Mar 31, 4:47 AM

	Unknown Object (File)
	Mon, Mar 31, 4:47 AM

	Unknown Object (File)
	Sat, Mar 29, 6:13 AM

	Unknown Object (File)
	Fri, Mar 28, 4:07 PM

	Unknown Object (File)
	Fri, Mar 28, 1:21 PM

View All 10 Files

Subscribers

None

Details

Reviewers

mav
ken
markj
chs

Summary

Now that nothing returns (2) to fake the success of the command,
eliminate that magic number by eliminating the case.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 63076
Build 59960: arc lint + arc unit

Event Timeline

imp requested review of this revision.Sun, Mar 23, 9:50 PM

imp created this revision.

Harbormaster completed remote builds in B63076: Diff 152568.Sun, Mar 23, 9:50 PM

imp added a parent revision: D49468: umass: Fail SYNCHRONIZE_CACHE for UFI.Sun, Mar 23, 9:50 PM

imp added a child revision: D49470: umass: make *_transform() return a bool.

imp added reviewers: mav, ken, markj.Mon, Mar 24, 1:04 AM

imp added a reviewer: chs.Wed, Apr 2, 8:04 PM

Revision Contents
Changeset List

Path

Size

sys/

dev/

usb/

storage/

umass.c

19 lines

Diff 152568

View Options

sys/dev/usb/storage/umass.c

umass: Simplify umass_std_transform to eliminate fake successNeeds ReviewPublicActions