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)
	Wed, Dec 4, 5:45 AM

	Unknown Object (File)
	Tue, Nov 26, 1:09 PM

	Unknown Object (File)
	Nov 23 2024, 7:05 AM

	Unknown Object (File)
	Oct 4 2024, 7:34 PM

	Unknown Object (File)
	Oct 4 2024, 1:48 PM

	Unknown Object (File)
	Oct 1 2024, 11:34 PM

	Unknown Object (File)
	Oct 1 2024, 11:00 PM

	Unknown Object (File)
	Sep 29 2024, 6:50 PM

View All 61 Files

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...