Boost C++ Libraries: Ticket #308: boost::rational documentation typos(?)
https://svn.boost.org/trac10/ticket/308
<pre class="wiki"><http://www.boost.org/libs/rational/
rational.html#Integer%20Type%20Requirements> states:
"I must be an integer-like type, that is the following expressions must
be valid for any two values n and m of type I, with the "expected"
semantics.
[...]
n / m (must truncate, and n/m must be positive if n and m are
positive)"
I doubt that that is what boost::rational requires.
int n = 1;
int m = 3;
int n_over_m = n/m;
if( n_over_m > 0)
{
std::cout << "all OK\n";
} else {
std::cout << "Can't use 'int' as base type for boost::rational\n";
}
<http://www.boost.org/libs/rational/rational.html#Constructors>
states:
"Rationals can be constructed from a pair (numerator, denominator)
of integers, or a single integer"
I think that the documentation should read:
"Rationals can be constructed from a pair (numerator, denominator)
of I's, or a single I"
</pre>en-usBoost C++ Libraries/htdocs/site/boost.png
https://svn.boost.org/trac10/ticket/308
Trac 1.4.3Jonathan TurkanisSat, 08 Oct 2005 22:32:59 GMT
<link>https://svn.boost.org/trac10/ticket/308#comment:1 </link>
<guid isPermaLink="false">https://svn.boost.org/trac10/ticket/308#comment:1</guid>
<description>
<pre class="wiki">Logged In: YES
user_id=811799
Thanks for your obvservations. I'm planning on rewriting the
implementation of rational for 1.34 (with the help of Andras
Erdei) and will completely rewrite the documentation. I'll
be sure to change the above points
</pre>
</description>
<category>Ticket</category>
</item>
<item>
<dc:creator>Daryle Walker</dc:creator>
<pubDate>Fri, 06 Jan 2006 15:11:33 GMT</pubDate>
<title>status changed
https://svn.boost.org/trac10/ticket/308#comment:2
https://svn.boost.org/trac10/ticket/308#comment:2
<ul>
<li><strong>status</strong>
<span class="trac-field-old">assigned</span> → <span class="trac-field-new">closed</span>
</li>
</ul>
<pre class="wiki">Logged In: YES
user_id=551024
I just submitted a fix; please re-open this bug if the
changes are not adequate. The CVS version of the
fixed file is:
/cvsroot/boost/boost/libs/rational/rational.html :1.8
</pre>
Ticket