date_time's embedded constrained_value uses uninitialized member

[Brad Spencer <spencer@…>]

At first I thought this was a spurious warning from gcc-4.3, but upon further inspection, I agree that the constrained_value class that comes as part of date_time is using its value_ member before initializing it when constructed with an out-of-range value.

I checked what the unofficial Boost Constrained Value proposed library does, and it seems to "initialize, then check". It's reality easy to apply the same approach here (see the attached patch). There may be a better approach overall, but this prevents value_'s value from being passed to a function (even if that function "doesn't use" the value) before it is initialized.

[Andrey Semashev]

Already fixed in trunk.