![]() |
Boost : |
From: Paul Giaccone (paulg_at_[hidden])
Date: 2006-11-01 04:26:46
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.
Boost list run by bdawes at acm.org, david.abrahams at rcn.com, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk