[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