More test results...

[Benjamin Kosnik]

> I'm still unclear as to why we're getting that error.  Isn't the
> complier and linker supposed to use the "gnu link once" technology to
> eliminate redundant instantiations?  As far as I can tell, those are
> implicit instantiations and the compiler is required to handle them in
> the appropriate way.  The constructs used are daily constructs, which
> means, it isn't just the library which is affected but also user code.
> I think the issue has to be investigated at the compiler and linker
> level.

I am uncertain of this, and defer to Jason.

> Independently of what is happening in the compiler, I think your patch
> is good because it will improve the overall footprint.  There is
> however a minor nit: there should be corresponding definitions of the
> static data members in src/limits.cc.  I'm not in a position to do any
> check-in, so please could do that change check in?  Thanks in advance.

Right! Forgot about that. I'll check in shortly, thanks.

-benjamin