Boost.Array gives MS warnings C4510 and C4610

[Paul A. Bristow]

Boost.Array gives MS warnings C4510 and C4610 in simple use.

boost::array<const double, 99> stuff = { 9.206343, ....}

Unless anyone can see that this should be fixed 'properly', a patch to push'n'pop and disable these annoying warnings is attached.

[Paul A. Bristow]

suppress warnings in Boost.array

[Steven Watanabe]

Why array<const double, 99> rather than const array<double, 99>?

Also, array<const double, 99> does not quite work correctly, because iterator::value_type is const double rather than double.

[Paul A. Bristow]

Replying to steven_watanabe:

Why array<const double, 99> rather than const array<double, 99>?

Also, array<const double, 99> does not quite work correctly, because iterator::value_type is const double rather than double.

You are right, as always.

Asking for an array of const doubles still *feels* resonable.

So, since I am sure I am not the only one to suffer from constant const confusion, it might be useful to document that const array is best, and briefly why?

[anonymous]

Replying to pbristow:

Asking for an array of const doubles still *feels* resonable.

So, since I am sure I am not the only one to suffer from constant const confusion, it might be useful to document that const array is best, and briefly why?

Sure. Now that I have a working doc toolchain again...

[Marshall Clow]

Some of this should be fixed in the 1.42 release candidate. Paul, can you check and see if this still occurs there? If so, I can add more warning suppression.

[Paul A. Bristow]

Replying to marshall:

I haven't got 1.42 rc yet, but the code appears unchanged in trunk?

I am still not clear if the warning should be suppressed (I suspect not suppressing is best) but the array docs should warn that to get an array with const values

boost::array<const double, 2> stuff = { 1.2, 3.4}; warnings

so use instead

const boost::array<double, 2> stuff = { 1.2, 3.4}; No warnings.

As Steven says, this is "because iterator::value_type is const double rather than double. "

(But the docs appear to me to be in XML :-( so editing may be troublesome?).

[Marshall Clow]

Replying to pbristow:

Replying to marshall:

I haven't got 1.42 rc yet, but the code appears unchanged in trunk?

Well, the code has been changed in the trunk. There was some warning suppression (though not specifically 4510 and 4610) added to that file after this bug was opened. I wanted to make sure (not having access to VC9) that this didn't take care of the problems reported here.

[Marshall Clow]

(In [59476]) Applied patches to fix #2886

[dimentiy]

I've got those two in 1.41 and can't avoid it. Push and pop around the #include <boost/array.hpp> don't work (vc80). :(