I've made the suggested changes below in the updated patch here:
http://users.ices.utexas.edu/~pbauman/tensor_tools.patch

Tested in devel mode both --enable-everything and --enable-everything
--enable-complex

OK for trunk?

On Thu, Aug 2, 2012 at 11:38 PM, Paul T. Bauman <ptbau...@gmail.com> wrote:

>
> On Thu, Aug 2, 2012 at 11:31 PM, Roy Stogner <royst...@ices.utexas.edu>wrote:
>
>>
>> libmesh_dot() is probably recent enough to move,
>> but if it's actually getting a serious namespace then we can probably
>> drop the libmesh_ pseudo-namespace and just call it TensorTools::dot()
>> (or TensorTools::inner_product?)
>>
>
> I like this, preferring inner_product.
>
>
>>
>> Anyone else have any preferences?  I could be talked into
>> TensorTools::norm_sq() if nobody else is using libmesh_norm() in user
>> code; that would have the bonus of being less confusing for people who
>> are unfamiliar with std::norm but familiar with the *correct*
>> definition of a norm.
>
>
> I also like norm_sq() - I initially fell into the "unfamiliar with
> std::norm" category...
>
------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
Libmesh-devel mailing list
Libmesh-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/libmesh-devel

Reply via email to