optional: Remove the value() method that returns non-const-ref.
optional: Remove the value() method that returns non-const-ref.

I'm not convinced that `opt.value() = foo` is a desirable pattern,
and worse still it confuses some compilers about which overload to
use.

Just remove it.

google/cppdap,google/cppdap,google/cppdap