[vsipl++] [patch] New example showing how to use Tensor subviews
Stefan Seefeld
stefan at codesourcery.com
Fri Oct 9 04:30:15 UTC 2009
On 10/08/2009 07:02 PM, Don McCoy wrote:
> These are all good ideas. I've eliminated all namespace directives with
> the exception of 'vsip'. Note the need for 'using vsip_csl::operator<<'
> as a result. Is it possible to include that statement directly in
> vsip_csl/output.hpp? Or would it be considered unsafe to do it this way?
> The benefit would be that it avoids the obscure error message that
> results if the directive is left out.
Yeah, I understand and agree this is bad. I think what you are
suggesting is a good idea. It only exposes the operator<<, nothing else.
Thus it is as if we added it to vsip::, only that it comes from a
non-standard header.
Yes, I think that's fine, so please do what you suggest.
> Otherwise, all feedback applied. Can you verify I made all the necessary
> makefile changes to include the new subdirectory?
This looks fine. Thanks !
Thanks,
Stefan
--
Stefan Seefeld
CodeSourcery
stefan at codesourcery.com
(650) 331-3385 x718
More information about the vsipl++
mailing list