Steven Schveighoffer wrote: > Steven Schveighoffer Wrote: > >> I'll have to double check, I thought I copied your code verbatim (I did >> switch around the arguments to be minA, maxA, minB, maxB to fit my test >> harness, and I changed the uint_32_t to uint). I'll check tomorrow at work >> where the computer I used to test is. >> > > I think I see the issue, I was using an older version of your highbit, at > some point you changed it, but didn't repeat it in this solution, so I > searched for it on the newsgroup and found your first version. That differs > from later versions you posted, but I can't find where you identified there > was a bug. > > I'll test again tomorrow, but it looks like that probably accounts for the > problem. > > The version I was using: > http://www.digitalmars.com/webnews/newsgroups.php?art_group=digitalmars.D&article_id=108788 > > The later version: > http://www.digitalmars.com/webnews/newsgroups.php?art_group=digitalmars.D&article_id=108855 > > It looks like in that second post that Ali had the same problem I did. > > Next time, you should include your whole code each time, especially parts you > changed. > Oups, I hadn't noticed that I sent two different versions (except for the C<->D translation). Sorry about that.
Note that there is no bug: both versions are correct. The first one
uses 1-based indexes, while the second uses 0-based indexes an
processes 0 differently.
Jerome
--
mailto:jeber...@free.fr
http://jeberger.free.fr
Jabber: jeber...@jabber.fr
signature.asc
Description: OpenPGP digital signature
