[bind10-dev] BIND 10 trac3160, updated. 6faa17ef4f22d11d3fc653680d073ac61197d6b3 [3160] Log text representation of the AbstractRRset (contd.)

[Michal 'vorner' Vaner]

Hello

On Mon, Sep 16, 2013 at 03:31:14PM +0530, Mukund Sivaraman wrote:
> On Mon, Sep 16, 2013 at 09:17:49AM +0200, Michal 'vorner' Vaner wrote:
> > Why is this change needed? Shouldn't .arg use the generic << operator? Shouldn't
> > rrsets print themselves with that? I think that was the original intention and
> > if it doesn't work, it is broken and we should fix that instead of working
> > around it.
> 
> You know how we are facing issues with compilers and Boost. This work is
> currently in a branch.

Yes, it is in branch. But I think it is better to point out issues sooner rather
than later.

> These changes are for issues with the latest version of stock Boost
> (according to Jeremy) and SunStudio compiler. boost::lexical_cast seems
> to be horribly broken on it:

We use lexical_cast for conversion to string, mostly. Would it make sense to
create our own version? As far as I understand what lexical_cast does, this
could make the trick for many uses (especially the one in .arg):

template<class T>
std::string to_string(const T& value) {
  std::string result;
  std::string_stream stream(result);
  stream << value;
  return (result);
}

This is, obviously, less generic, but removes bunch of layers I suspect boost
has, so the chance of hitting errors is lower.

With regards

-- 
Hello, I'm an extension to the infamous signature virus, called spymail.
Could you please copy me into your signature and send back what you were
doing last night between 10pm and 3am?

Michal 'vorner' Vaner
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <https://lists.isc.org/pipermail/bind10-dev/attachments/20130916/77c5fb11/attachment.bin>