3

在 Boost ICL 中，当我在区间集上调用 cardinality() 或 size() 函数时，返回类型为 size_t，与区间类型无关。在 32 位机器上，这是一个 32 位无符号整数。但是，如果我的区间是 int64_t 类型，则基数很容易溢出 32 位整数。我在这里遗漏了一些明显的东西还是这个库的严重缺陷？

编辑：添加示例

以下代码在 64 位但在 32 位机器上编译和运行没有错误，它会抛出断言。

#include <boost/icl/interval_set.hpp>

int main()
{
    boost::icl::interval_set<int64_t> is;
    is.add(boost::icl::interval<int64_t>::closed(1, 4294967297LL));
    assert(boost::icl::cardinality(is) == 4294967297LL);
}

编辑：我在 Ubuntu 13.10 上使用 boost::icl 版本 1.49.0

编辑：

这不是一个 32/64 位的问题，因为以下代码也不适用于 64 位

#include <boost/icl/interval_set.hpp>
int main()
{
    boost::icl::interval_set<double> is;
    is.add(boost::icl::interval<double>::closed(1, 1.5));
    assert(boost::icl::cardinality(is) == 0.5);
}
4

1 回答 1

2

在 Ubuntu 14.04.1 LTS 上使用 Boost 1_54 复制

这似乎确实是一个错误。要修复的专业是

template <class Type> 
struct get_size_type<Type, false, false, false>
{ 
    typedef std::size_t type; 
};

在icl/type_traits/size_type_of.hpp. 不知何故，这些天 ICL 开发人员似乎没有使用 -m32 进行测试。

我已经成功将其替换为

// BEGIN SEHE WAS HERE
template <class Type> 
struct get_size_type<Type, std::enable_if<not boost::is_arithmetic<Type>::value, mpl::false_>::type::value, false, false>
{ 
    typedef std::size_t type; 
};

template <class Type> 
struct get_size_type<Type, std::enable_if<boost::is_arithmetic<Type>::value, mpl::false_>::type::value, false, false>
{ 
    typedef typename std::common_type<Type, std::size_t>::type type; 
};
// END SEHE WAS HERE

遗憾的是，该特征对 SFINAE 不是很友好，因此使用boolSFINAE 的第一个模板参数进行破解。改进可能是：

  • 仅使用boost类型特征
  • 使用 Boost Integer 的整数值扣除，而不是common_type<...>整数类型

我已经在 DoTheRightThing(TM)interval_set<double>以及interval_set<uint64_t>g++ -m32 和 -m64 上对此进行了测试。

我会在邮件列表上报告这个。

于 2014-12-01T09:46:18.457 回答