Paths

Table of Contentst

ktls_test: fix -Wsign-compare issue
ClosedPublic
Actions

Authored by ngie on Mon, Aug 31, 2:38 AM.

Tags

None

Referenced Files

	F170844878: D59285.id185614.diff
	Mon, Sep 7, 12:11 AM

	F170821838: D59285.diff
	Sun, Sep 6, 8:34 PM

	Unknown Object (File)
	Sun, Sep 6, 1:09 AM

	Unknown Object (File)
	Sat, Sep 5, 11:27 PM

	Unknown Object (File)
	Sat, Sep 5, 6:27 PM

	Unknown Object (File)
	Sat, Sep 5, 6:26 PM

	Unknown Object (File)
	Sat, Sep 5, 9:20 AM

	Unknown Object (File)
	Fri, Sep 4, 7:37 PM

Subscribers

Details

Reviewers

markj

Commits

rG505103215ee2: tests/sys/kern/ktls_test: fix -Wsign-compare issue

Summary

Cast the size_t quantity used in a comparison to off_t to mute a
-Wsign-compare complaint that now occurs after ATF 0.22 [1].

MFC after: 2 weeks
Reported by: clang

https://github.com/freebsd/atf/pull/72

Test Plan

Test passed after this change.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

ngie created this revision.Mon, Aug 31, 2:38 AM

Herald added a subscriber: imp. · View Herald TranscriptMon, Aug 31, 2:38 AM

ngie requested review of this revision.Mon, Aug 31, 2:38 AM

Harbormaster completed remote builds in B76343: Diff 185465.Mon, Aug 31, 2:38 AM

ngie edited the test plan for this revision. (Show Details)Mon, Aug 31, 2:41 AM

ngie added subscribers: glebius, jhb, markj.

markj accepted this revision.Tue, Sep 1, 3:43 PM

This revision is now accepted and ready to land.Tue, Sep 1, 3:43 PM

Closed by commit rG505103215ee2: tests/sys/kern/ktls_test: fix -Wsign-compare issue (authored by ngie). · Explain WhyWed, Sep 2, 7:40 AM

This revision was automatically updated to reflect the committed changes.

ngie added a commit: rG505103215ee2: tests/sys/kern/ktls_test: fix -Wsign-compare issue.

Revision Contents
Changeset List

Path

Size

tests/

sys/

kern/

ktls_test.c

2 lines

Diff 185614

View Options

tests/sys/kern/ktls_test.c

tests/sys/kern/ktls_test: fix -Wsign-compare issueClosedPublicActions