|
Boost : |
From: Daryle Walker (darylew_at_[hidden])
Date: 2001-06-18 10:54:41
This is from just a quick glance at the Tuples library under review.
1. The header files have the authors' e-mail addresses as well as their
names. Maybe the e-mails should be left out, so we don't have to change the
headers in cases the e-mails change. The authors should eventually be
listed in the People section of the documentation, and they could put their
e-mail addresses there.
2. There's a type called "nil" for a placeholder for unused positions.
This is confusing from the usage as a NULL pointer that the last reviewed
library had. The name should be more descriptive. Worse, the name "nil" is
a macro constant in at least one OS-provider's system headers.
-- Daryle Walker Mac, Internet, and Video Game Junkie darylew AT mac DOT com
Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk