seq: fix check for rounding error/truncation
ClosedPublic
Actions

Authored by emaste on Jun 19 2023, 1:36 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Oct 2, 4:08 PM

	Unknown Object (File)
	Thu, Oct 1, 9:21 PM

	Unknown Object (File)
	Thu, Oct 1, 2:43 PM

	Unknown Object (File)
	Tue, Sep 29, 8:28 AM

	Unknown Object (File)
	Sun, Sep 27, 1:12 PM

	Unknown Object (File)
	Sun, Sep 20, 8:24 AM

	Unknown Object (File)
	Fri, Sep 11, 6:08 PM

	Unknown Object (File)
	Mon, Sep 7, 8:46 AM

View All Files

Subscribers

imp

Details

Reviewers

yuripv

Commits

rG70af4d57c18f: seq: fix check for rounding error/truncation
rGe54db9a9ccd5: seq: fix check for rounding error/truncation

Summary

Based on OpenBSD 30f0fd29ba6c:

We need to compare the printable version of the last value displayed, not the floating point representation. Otherwise, we may print the last value twice.

PR: 271964

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

emaste requested review of this revision.Jun 19 2023, 1:36 AM

emaste created this revision.

yuripv accepted this revision.Jun 19 2023, 11:10 AM

This revision is now accepted and ready to land.Jun 19 2023, 11:10 AM

Closed by commit rGe54db9a9ccd5: seq: fix check for rounding error/truncation (authored by emaste). · Explain WhyJun 19 2023, 3:29 PM

This revision was automatically updated to reflect the committed changes.

emaste added a commit: rGe54db9a9ccd5: seq: fix check for rounding error/truncation.

Herald added a subscriber: imp. · View Herald TranscriptJun 19 2023, 3:29 PM

emaste added a commit: rG70af4d57c18f: seq: fix check for rounding error/truncation.Jun 26 2023, 4:32 PM

Revision Contents
Changeset List

Path

Size

usr.bin/

seq/

seq.c

22 lines

Diff 123426

View Options

seq: fix check for rounding error/truncationClosedPublicActions