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)
	Sun, Aug 9, 12:29 AM

	Unknown Object (File)
	Sat, Aug 8, 6:34 PM

	Unknown Object (File)
	Sat, Aug 8, 1:29 PM

	Unknown Object (File)
	Sat, Aug 8, 12:03 AM

	Unknown Object (File)
	Sun, Aug 2, 11:30 AM

	Unknown Object (File)
	Sun, Aug 2, 3:16 AM

	Unknown Object (File)
	Sun, Aug 2, 1:17 AM

	Unknown Object (File)
	Sat, Aug 1, 7:54 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