Qt 5:无法声明采用右值参考的信号

时间:2015-06-16 17:05:09

标签: c++ qt c++11 signals rvalue-reference

可能我错过了什么,但我找不到任何信号不能接受右值参考的信息。

所以,我有一个具有以下信号声明的类:

signals:
   void messageDecoded(HTDataMsg &&msg);

当我尝试编译它时,我遇到了错误:

moc_htcodec.cpp: In static member function ‘static void HTCodec::qt_static_metacall(QObject*, QMetaObject::Call, int, void**)’:
moc_htcodec.cpp:71:77: error: cannot bind ‘HTDataMsg’ lvalue to ‘HTDataMsg&&’
         case 0: _t->messageDecoded((*reinterpret_cast< HTDataMsg(*)>(_a[1]))); break;
                                                                             ^
In file included from moc_htcodec.cpp:9:0:
../hterm_core/htcodec/htcodec.h:59:9: error:   initializing argument 1 of ‘void HTCodec::messageDecoded(HTDataMsg&&)’
    void messageDecoded(HTDataMsg &&msg);
         ^
make: *** [moc_htcodec.o] Error 1

生成的moc文件中的代码确实是错误的:

void HTCodec::qt_static_metacall(QObject *_o, QMetaObject::Call _c, int _id, void **_a)
{
    if (_c == QMetaObject::InvokeMetaMethod) {
        HTCodec *_t = static_cast<HTCodec *>(_o);
        switch (_id) {
        case 0: _t->messageDecoded((*reinterpret_cast< HTDataMsg(*)>(_a[1]))); break;
        default: ;
        }
    } else if (_c == QMetaObject::IndexOfMethod) {
        int *result = reinterpret_cast<int *>(_a[0]);
        void **func = reinterpret_cast<void **>(_a[1]);
        {
            typedef void (HTCodec::*_t)(HTDataMsg && );
            if (*reinterpret_cast<_t *>(func) == static_cast<_t>(&HTCodec::messageDecoded)) {
                *result = 0;
            }
        }
    }
}

预计会出现这种情况吗?信号采用右值参考是否合法?

如果我将HTDataMsg &&msg更改为const HTDataMsg &msg,那么当然可行。

2 个答案:

答案 0 :(得分:4)

  

信号采用右值参考是否合法?

是。它们采用右值引用是没有意义的,因为接收器的数量是非负整数。 Rvalue引用只对零或一个接收器的特殊情况有意义,即使这样,它也需要一个仅限C ++ 11的Qt版本。

如果您认为这样的优化对于常见数据类型有意义(通过基准测量和支持断言!),它可以在Qr 5.7之后实现,因为它需要平台支持C ++ 11。这需要moc以及图书馆本身的支持。

答案 1 :(得分:0)

如果需要将内存移动用于一个信号且只有一个插槽的情况,则可以通过简单引用而不是'const'传递对象,然后在插槽内使用移动语义。 这是一些简单的示例:

class PcapPacket {
    public:
        PcapPacket();
        std::vector<uint8_t>&& getData() {
          return std::move(_vData);
        }
    private:
        std::vector<uint8_t> _vData;
    };

    signals:
        void packet(PcapPacket& pcapPacket);


 // Register types for signals
 qRegisterMetaType<PcapPacket>("PcapPacket&");

    connect(pcapReader, &PcapReader::packet, controlItem, &SensorControlItem::processPcapPacket);
 //   connect(_pcapReader, SIGNAL(packet(PcapPacket&)), controlItem, SLOT(processPcapPacket(PcapPacket&)));

    void SensorControlItem::processPcapPacket(PcapPacket& packet) {
      _sensor->processPcapPacket(packet.getData());
    }

    void sensor::processPcapPacket(std::vector<uint8_t>&& data) {
      // Do here with data what ever you want. It's yours without memory copy
    }