Boost C++ Libraries: Ticket #8129: Conversion warning in hashed_index.hpp

attachment set

mbradle@… — Wed, 20 Feb 2013 22:44:14 GMT

attachment → ex.13.7.cpp

test cpp file

attachment set

mbradle@… — Wed, 20 Feb 2013 22:44:51 GMT

attachment → test.txt

transcript of compilation and warning

attachment set

mbradle@… — Wed, 20 Feb 2013 22:46:31 GMT

attachment → diff.txt

Change that removed the warning (possible patch)

version changed

mbradle@… — Wed, 20 Feb 2013 22:51:08 GMT

version Boost 1.53.0 → Boost Development Trunk

Joaquín M López Muñoz — Thu, 21 Feb 2013 07:28:11 GMT

Hi Brad,

Actually, your proposed patch is not entirely correct: n should be promoted to a floating point type, not mlf to integral. Please try the following:

      float     fbc=static_cast<float>(1+static_cast<double>(n)/mlf);

If this clears the warning I'll commit the change to the repository. Thank you.

mbradle@… — Thu, 21 Feb 2013 08:26:51 GMT

Replying to joaquin:

Actually, your proposed patch is not entirely correct: n should be promoted to a floating point type, not mlf to integral. Please try the following:
      float     fbc=static_cast<float>(1+static_cast<double>(n)/mlf);
If this clears the warning I'll commit the change to the repository. Thank you.

Hi,

Yes, this does clear the warning, and I do see that n should be promoted, not mlf. Thanks, and best wishes.

Brad

status changed; resolution set

Joaquín M López Muñoz — Thu, 21 Feb 2013 10:11:34 GMT

status new → closed
resolution → fixed

(In [83058]) fixed #8129