Re: [Monotone-devel] nvm.options

[Timothy Brownawell]

On 07/21/2010 05:25 PM, Thomas Keller wrote:
> Am 21.07.10 01:10, schrieb Thomas Keller:
> > Am 20.07.10 04:58, schrieb Timothy Brownawell:
> > > On 07/18/2010 08:19 AM, Stephen Leake wrote:
> > > > Tim,
> > > >
> > > > Just letting you know I've been keeping up with nvm.options, and it all
> > > > looks very good so far.
> > >
> > > I think it's ready to merge now. See anything I missed?
> >
> > The current head (44c1dd7bbc276790c51463a83823d2e1339c7ff6) fails to
> > compile for me:
> >
> > base.hh: In function ‘void dump(const T&, std::string&) [with T = long
> > unsigned int]’:
> > sanity.hh:451:   instantiated from ‘void Musing<T>::gasp(std::string&)
> > const [with T = size_t]’
> > cmd.cc:640:   instantiated from here
> > base.hh:45: error: invalid application of ‘sizeof’ to incomplete type
> > ‘dump_must_be_specialized_for_this_type’
> > make[1]: *** [cmd.o] Error 1
> > make: *** [all] Error 2
>
> I added a specialization for size_t in
> 760adcf3915f59a0ba8e2c55ea652ef29deac477 and it compiles now.

That *breaks* it here, since apparently u64 and size_t are the same 
thing (while your size_t / unsigned long doesn't match u64 or u32).

I guess we need an autoconf thingy for this?

--
Timothy

Free public monotone hosting: http://mtn-host.prjek.net