casts issues

16 views
Skip to first unread message

dmitry....@gmail.com

unread,
Jul 17, 2014, 5:05:49 AM7/17/14
to nt2...@googlegroups.com
Hi All,

1. [casts_downgrade] - uses [nt2::groups] while [as_group] typedef uses [tag::group_] instead of [tag::groups_]. Is this by design? If is, we need to include <nt2/include/functions/group.hpp> header.
2. The same with [nt2::toints] and [nt2::touints]
3. I have problem with nt2 compilation on Visual Studio 2013 while processing <nt2/table.hpp>
------

1>.. \nt2\build\include\nt2\operator\functions\container\casts.hpp(138): error C2953: 'nt2::ext::casts_intfloat<Expr,From,To,boost::enable_if_c<boost::dispatch::meta::is_integral<To>::value&&boost::dispatch::meta::is_unsigned<To>::value,void>::type>' : class template has already been defined

1> .. \nt2\build\include\nt2\operator\functions\container\casts.hpp(122) : see declaration of 'nt2::ext::casts_intfloat<Expr,From,To,boost::enable_if_c<boost::dispatch::meta::is_integral<To>::value&&boost::dispatch::meta::is_unsigned<To>::value,void>::type>'

------
As far I research this - this depends on order of <cast.hpp> - <casts.hpp> includes - the latter include fail.

Revision: 40b1138d85f0e71b3ae5555bf6b0a5601a19a2db Author: Joel Falcou <....> Date: 6/23/2014 03:38:23
Visual Studio Pro 2013 Upd 1
CMake 3.0, vs2013 x64 configuration, no additional settings besides default
Windows 8.1 Pro x64

Mathias Gaunard

unread,
Jul 18, 2014, 5:00:10 AM7/18/14
to nt2...@googlegroups.com
Hi,

It's indeed a bug, casts was just copy-pasted from cast and slightly
adapted.

Fixed in ff9f3938a6cb7c05a8dc29660b0ae4ec6417a53d.
<https://github.com/MetaScale/nt2/commit/ff9f3938a6cb7c05a8dc29660b0ae4ec6417a53d>

Thanks for the report.
> --
> You received this message because you are subscribed to the Google
> Groups "nt2-dev" group.
> To unsubscribe from this group and stop receiving emails from it, send
> an email to nt2-dev+u...@googlegroups.com
> <mailto:nt2-dev+u...@googlegroups.com>.
> For more options, visit https://groups.google.com/d/optout.

Reply all
Reply to author
Forward
0 new messages