<feed xmlns='http://www.w3.org/2005/Atom'>
<title>wireguard-openbsd/games/random, branch master</title>
<subtitle>WireGuard implementation for the OpenBSD kernel</subtitle>
<id>https://git.zx2c4.com/wireguard-openbsd/atom/games/random?h=master</id>
<link rel='self' href='https://git.zx2c4.com/wireguard-openbsd/atom/games/random?h=master'/>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/wireguard-openbsd/'/>
<updated>2016-03-07T12:07:55Z</updated>
<entry>
<title>- General changes:</title>
<updated>2016-03-07T12:07:55Z</updated>
<author>
<name>mestre</name>
<email>mestre@openbsd.org</email>
</author>
<published>2016-03-07T12:07:55Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/wireguard-openbsd/commit/?id=6fa5e1da3490487b9d3a614457ba0b7c041eb1ac'/>
<id>urn:sha1:6fa5e1da3490487b9d3a614457ba0b7c041eb1ac</id>
<content type='text'>
- Remove -? from getopt(3) options, but still keep (or add) -h where applicable
- Replace hardcoded program strings by getprogname(3)

- Specific changes:
- atc(6): this used -? and -u for usage(), remove both from game and manpage
- bcd(6): use __progname instead of getprogname(3), no need to include stdlib.h
- hunt(6): replace fputs(3) by fprintf(3)

OK tb@ after his suggestions
</content>
</entry>
<entry>
<title>As per style(9), remove remaining lint-style comments from games/</title>
<updated>2016-01-10T13:35:09Z</updated>
<author>
<name>mestre</name>
<email>mestre@openbsd.org</email>
</author>
<published>2016-01-10T13:35:09Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/wireguard-openbsd/commit/?id=7e027cb7af8339d869c5abaf2cab2bd54487de54'/>
<id>urn:sha1:7e027cb7af8339d869c5abaf2cab2bd54487de54</id>
<content type='text'>
OK tb@
</content>
</entry>
<entry>
<title>Some basic code maintenance in games/</title>
<updated>2016-01-07T16:00:31Z</updated>
<author>
<name>tb</name>
<email>tb@openbsd.org</email>
</author>
<published>2016-01-07T16:00:31Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/wireguard-openbsd/commit/?id=17641e3181064a4b256bd1af9f64d90af0ac5a36'/>
<id>urn:sha1:17641e3181064a4b256bd1af9f64d90af0ac5a36</id>
<content type='text'>
- in main() replace exit with return
- drop some /* NOTREACHED */ lint comments along the way.
- make more use of standard CFLAGS, esp. -Wimplicit-function-declaration
- add and sort some headers when needed
- add straightforward pledges to some programs used at compile time

discussed with and ok mestre@
</content>
</entry>
<entry>
<title>More headers removal and sorted the remaining alphabetically.</title>
<updated>2016-01-04T11:58:35Z</updated>
<author>
<name>mestre</name>
<email>mestre@openbsd.org</email>
</author>
<published>2016-01-04T11:58:35Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/wireguard-openbsd/commit/?id=bdf3894f0c761055c5cbceaae3fb7c235dd3294c'/>
<id>urn:sha1:bdf3894f0c761055c5cbceaae3fb7c235dd3294c</id>
<content type='text'>
Most noticeable that were removed are sys/types.h which will be included from
stdlib.h and in turn also brings along sys/cdefs.h. The other one is limits.h
that was used with the old idiom (denom * random() / LONG_MAX), although it was
only removed were applicable (some files still need that header).

OK tb@
</content>
</entry>
<entry>
<title>About 13 years ago when the idiom srandom(time()), and sometimes</title>
<updated>2016-01-03T14:38:16Z</updated>
<author>
<name>mestre</name>
<email>mestre@openbsd.org</email>
</author>
<published>2016-01-03T14:38:16Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/wireguard-openbsd/commit/?id=9ea8ec706f2f03d68b523542bd48faf2fabf1785'/>
<id>urn:sha1:9ea8ec706f2f03d68b523542bd48faf2fabf1785</id>
<content type='text'>
srandom(time()+getpid()), was changed by srandomdev(), but #include &lt;time.h&gt;
lived up until this day so remove it.

Additionally, earlier than that, 18 years ago, random(6) was one of the first
consumers of arc4random(3) family, and was pulling it from dev/rndvar.h but
these days we pull it from stdlib.h, which is already done, so while here
remove dev/rndvar.h also.

"seems comprehensive to me" deraadt@ and OK tb@
</content>
</entry>
<entry>
<title>Declare usage() functions as __dead void, if they don't return, on games section.</title>
<updated>2015-12-25T20:59:09Z</updated>
<author>
<name>mestre</name>
<email>mestre@openbsd.org</email>
</author>
<published>2015-12-25T20:59:09Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/wireguard-openbsd/commit/?id=f0628b46ae621bae7650f74df86c15fcd5e2698f'/>
<id>urn:sha1:f0628b46ae621bae7650f74df86c15fcd5e2698f</id>
<content type='text'>
Found another one in arithmetic(6) which also didn't return, and removed a function from number(6) which is not used anymore.

OK tb@
</content>
</entry>
<entry>
<title>replace setbuf with setvbuf, from Frederic Nowak</title>
<updated>2015-11-04T21:22:10Z</updated>
<author>
<name>tedu</name>
<email>tedu@openbsd.org</email>
</author>
<published>2015-11-04T21:22:10Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/wireguard-openbsd/commit/?id=33206277febef61dfb7bd1e4e5e039c0db0c0ce3'/>
<id>urn:sha1:33206277febef61dfb7bd1e4e5e039c0db0c0ce3</id>
<content type='text'>
</content>
</entry>
<entry>
<title>Pledge "stdio" for simple games.</title>
<updated>2015-10-14T08:12:12Z</updated>
<author>
<name>doug</name>
<email>doug@openbsd.org</email>
</author>
<published>2015-10-14T08:12:12Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/wireguard-openbsd/commit/?id=673e924c3636ca89a9458797e02ad35e9e67754e'/>
<id>urn:sha1:673e924c3636ca89a9458797e02ad35e9e67754e</id>
<content type='text'>
ok semarie@
</content>
</entry>
<entry>
<title>rcsid[] and sccsid[] and copyright[] are essentially unmaintained (and</title>
<updated>2009-10-27T23:59:19Z</updated>
<author>
<name>deraadt</name>
<email>deraadt@openbsd.org</email>
</author>
<published>2009-10-27T23:59:19Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/wireguard-openbsd/commit/?id=043fbe51c197dbbcd422e917b65f765d8b5f8874'/>
<id>urn:sha1:043fbe51c197dbbcd422e917b65f765d8b5f8874</id>
<content type='text'>
unmaintainable).  these days, people use source.  these id's do not provide
any benefit, and do hurt the small install media
(the 33,000 line diff is essentially mechanical)
ok with the idea millert, ok dms
</content>
</entry>
<entry>
<title>Use arc4random_buf() when requesting more than a single word of output</title>
<updated>2008-04-13T00:22:16Z</updated>
<author>
<name>djm</name>
<email>djm@openbsd.org</email>
</author>
<published>2008-04-13T00:22:16Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/wireguard-openbsd/commit/?id=66ad965f4873a0970dea06fb53c307b8385e5a94'/>
<id>urn:sha1:66ad965f4873a0970dea06fb53c307b8385e5a94</id>
<content type='text'>
Use arc4random_uniform() when the desired random number upper bound
is not a power of two

ok deraadt@ millert@
</content>
</entry>
</feed>
