Differential D23289

random(3): Abstract routines into _r versions on explicit state
ClosedPublic
Actions

Authored by cem on Jan 21 2020, 2:07 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, Dec 30, 11:39 AM

	Unknown Object (File)
	Mon, Dec 29, 7:49 PM

	Unknown Object (File)
	Sat, Dec 27, 3:40 PM

	Unknown Object (File)
	Mon, Dec 22, 3:05 PM

	Unknown Object (File)
	Fri, Dec 19, 5:29 PM

	Unknown Object (File)
	Wed, Dec 17, 9:29 PM

	Unknown Object (File)
	Sun, Dec 14, 12:49 AM

	Unknown Object (File)
	Tue, Dec 9, 4:55 PM

Subscribers

Details

Reviewers

jilles
kevans
vangyzen
markm

Group Reviewers

Commits

rS357066: random(3): Abstract routines into _r versions on explicit state

Summary

The existing APIs simply pass the implicit global state to the _r variants.

No functional change.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

cem created this revision.Jan 21 2020, 2:07 AM

Harbormaster completed remote builds in B28804: Diff 67079.Jan 21 2020, 2:07 AM

cem added a parent revision: D23288: random(3): Abstract state into a single context object.Jan 21 2020, 2:09 AM

cem added a child revision: D23290: rand(3): Replace implementation with one backed by random(3) algorithm.

markm accepted this revision.Jan 21 2020, 6:11 PM

This revision is now accepted and ready to land.Jan 21 2020, 6:11 PM

I had some concerns about providing *_r variants of these that match names from glibc but aren't compatible, but I see the ultimate plan is that these aren't intended to be exported from libc at all- this seems OK.

In D23289#510876, @kevans wrote:

I had some concerns about providing *_r variants of these that match names from glibc but aren't compatible, but I see the ultimate plan is that these aren't intended to be exported from libc at all- this seems OK.

Hm, I didn't realize glibc had these routines. They picked different argument orderings and state struct name (and field names), but otherwise look pretty similar. Yeah, I didn't intend to export these routines from libc and don't see a great reason to.

Closed by commit rS357066: random(3): Abstract routines into _r versions on explicit state (authored by cem). · Explain WhyJan 24 2020, 1:39 AM

This revision was automatically updated to reflect the committed changes.

cem added a commit: rS357066: random(3): Abstract routines into _r versions on explicit state.

Herald added a subscriber: imp. · View Herald TranscriptJan 24 2020, 1:39 AM

Revision Contents
Changeset List

Path

Size

head/

lib/

libc/

stdlib/

46 lines

196 lines

Diff 67234

head/lib/libc/stdlib/random.h

Loading...

head/lib/libc/stdlib/random.c

Loading...