[Boost-bugs] [ boost-Bugs-1588359 ] "Bug" in comment

Bugs item #1588359, was opened at 2006-10-31 18:54 Message generated for change (Tracker Item Submitted) made by Item Submitter You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=107586&aid=1588359&group_id=7586 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Alexis Wilke (alexis_wilke) Assigned to: Nobody/Anonymous (nobody) Summary: "Bug" in comment Initial Comment: This is very low priority! Hey guys, In this file: http://boost.org/boost/integer.hpp there is the following comment which says '9=unsigned long' and the code says '9=unsigned char'. Should someone get rid of that comment or fix it? // specializatons: 1=long, 2=int, 3=short, 4=signed char, // 6=unsigned long, 7=unsigned int, 8=unsigned short, 9=unsigned long // no specializations for 0 and 5: requests for a type > long are in error template<> struct int_least_helper<1> { typedef long least; }; template<> struct int_least_helper<2> { typedef int least; }; template<> struct int_least_helper<3> { typedef short least; }; template<> struct int_least_helper<4> { typedef signed char least; }; template<> struct int_least_helper<6> { typedef unsigned long least; }; template<> struct int_least_helper<7> { typedef unsigned int least; }; template<> struct int_least_helper<8> { typedef unsigned short least; }; template<> struct int_least_helper<9> { typedef unsigned char least; }; ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=107586&aid=1588359&group_id=7586 ------------------------------------------------------------------------- Using Tomcat but need to do more? Need to support web services, security? Get stuff done quickly with pre-integrated technology to make your job easier Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642 _______________________________________________ Boost-bugs mailing list Boost-bugs@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/boost-bugs

SourceForge.net wrote:
Bugs item #1588359, was opened at 2006-10-31 18:54 Message generated for change (Tracker Item Submitted) made by Item Submitter You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=107586&aid=1588359&group_id=7586
Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Alexis Wilke (alexis_wilke) Assigned to: Nobody/Anonymous (nobody) Summary: "Bug" in comment
Initial Comment: This is very low priority!
Hey guys,
In this file:
http://boost.org/boost/integer.hpp
there is the following comment which says '9=unsigned long' and the code says '9=unsigned char'. Should someone get rid of that comment or fix it?
This is somewhat of a code smell. Better, IMO, to get rid of the individual comments and to replace them with a single one, saying that number "n" stands for type "T" in each line - this is then immune from any transcription errors or changes to the code.
participants (2)
-
Paul Giaccone
-
SourceForge.net