Re: [boost] Re: Patch for function/function_base.hpp

From: Douglas Gregor (gregod_at_[hidden])
Date: 2003-02-13 07:56:53

On Thursday 13 February 2003 04:38 am, Daniel Frey wrote:
> Ah, that's the reason. But given my recent discomfort about
> unmaintainable code, look at it again:
>
>   #  if BOOST_WORKAROUND(__HP_aCC, <= 33900)
>       template<bool cond, typename T> struct enable_if;
>   #  else
>       template<bool, typename T> struct enable_if;
>   #  endif
>
> Does this really makes sense? Shouldn't we just keep one version with
> names for template parameters? AFAICS this should work for all compilers
> and it could be a general boost coding guideline to always provide names
> for template parameters. Comments?

FWIW, I didn't apply the patch directly but instead changed it to what you 
ask:
  template<bool cond, typename T> struct enable_if;

I'd be fine with making this a coding guideline...

        Doug