Page MenuHomeFreeBSD
Differential D51723

libutil: Backward compatibility for expand_number()
ClosedPublic
Actions

Authored by des on Mon, Aug 4, 5:02 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Aug 12, 9:22 PM
Unknown Object (File)
Sun, Aug 10, 8:10 AM
Unknown Object (File)
Sat, Aug 9, 10:43 AM
Unknown Object (File)
Fri, Aug 8, 6:29 PM
Unknown Object (File)
Wed, Aug 6, 8:44 PM
Subscribers
imp
kib
ziaee

Details

Reviewers
jhb
olce
Commits
rGab1c6874e500: libutil: Backward compatibility for expand_number()
Summary

Reimplement expand_number() in terms of expand_unsigned(), which takes
a pointer to uint64_t like expand_number() did before. Provide a macro
that picks the correct version based on the type of the argument.

Fixes: 2e0caa7c7e14

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 65948
Build 62831: arc lint + arc unit

Event Timeline

des created this revision.Mon, Aug 4, 5:02 PM
Herald added a reviewer: olce. · View Herald TranscriptMon, Aug 4, 5:02 PM
Herald added a subscriber: imp. · View Herald Transcript
des requested review of this revision.Mon, Aug 4, 5:02 PM
Harbormaster completed remote builds in B65947: Diff 159713.Mon, Aug 4, 5:02 PM
des mentioned this in D51542: libutil: Really fix expand_number(3).Mon, Aug 4, 5:02 PM
des updated this revision to Diff 159714.Mon, Aug 4, 5:04 PM

typo

Harbormaster completed remote builds in B65948: Diff 159714.Mon, Aug 4, 5:04 PM
jhb added a comment.Mon, Aug 4, 5:14 PM

Looks good to me

kib added a subscriber: kib.Mon, Aug 4, 9:52 PM

Don't you need to export expand_unsigned in version def?

lib/libutil/libutil.h
94

_Generic use should be braced by the C dialect check, since this is a public header.

des updated this revision to Diff 159842.Wed, Aug 6, 12:32 PM

complete

Harbormaster completed remote builds in B66006: Diff 159842.Wed, Aug 6, 12:32 PM
des edited the summary of this revision. (Show Details)Wed, Aug 6, 12:32 PM
des updated this revision to Diff 159843.Wed, Aug 6, 12:36 PM
des edited the summary of this revision. (Show Details)

nit

Harbormaster completed remote builds in B66007: Diff 159843.Wed, Aug 6, 12:36 PM
des updated this revision to Diff 159854.Wed, Aug 6, 1:35 PM

documentation

Herald added a subscriber: ziaee. · View Herald TranscriptWed, Aug 6, 1:35 PM
Harbormaster completed remote builds in B66015: Diff 159854.Wed, Aug 6, 1:35 PM
des marked an inline comment as done.Wed, Aug 6, 1:39 PM
des edited the summary of this revision. (Show Details)
jhb accepted this revision.Wed, Aug 6, 2:48 PM
This revision is now accepted and ready to land.Wed, Aug 6, 2:48 PM
Closed by commit rGab1c6874e500: libutil: Backward compatibility for expand_number() (authored by des). · Explain WhyWed, Aug 6, 8:44 PM
This revision was automatically updated to reflect the committed changes.
des added a commit: rGab1c6874e500: libutil: Backward compatibility for expand_number().

Revision Contents
Changeset List

PathSize
lib/
libutil/
20 lines
5 lines

Diff 159714

View Options

lib/libutil/expand_number.c

Loading...
View Options

lib/libutil/libutil.h

Loading...