'void *(...)(void *)'类型的参数与'void *(*)(void *)不匹配

时间:2014-06-19 15:41:48

标签: c++ multithreading void gnuradio

我目前正在GNU Radio上开发一个集团,我想使用一个线程。这个线程是从UDP套接字获取数据,所以我可以在我的GNU Radio bloc中使用它。 "一般工作"功能是完成所有信号和数据处理的功能。

主源文件的组织方式如下:

namespace gr {
    namespace adsb {

    out::sptr
    out::make()
    {
        return gnuradio::get_initial_sptr
        (new out_impl());
    }

    /*
     * UDP thread
     */
    void *task_UdpRx (void *arg)
    {
        while(true)
        {
            printf("Task UdpRx\n\r");
            usleep(500*1000);
        }
       pthread_exit(NULL);
    }

    /*
    * The private constructor
    */
    out_impl::out_impl()
        : gr::block("out",
                gr::io_signature::make(1, 1, sizeof(int)),
                gr::io_signature::make(1, 1, sizeof(char)))
    {
        pthread_t Thread_UdpRx;

        //Thread init
        if(pthread_create(&Thread_UdpRx, NULL, task_UdpRx, NULL))
        {
            err("Pthread error");
        }
        else
        {
            printf("UDP thread initialization completed\n\r");
        }
    }

    /*
    * Our virtual destructor.
    */
    out_impl::~out_impl()
    {
    }

    void out_impl::forecast (int noutput_items, gr_vector_int &ninput_items_required)
    {
        ninput_items_required[0] = noutput_items;
    }

    int out_impl::general_work (int noutput_items, gr_vector_int &ninput_items, gr_vector_const_void_star &input_items, gr_vector_void_star &output_items)
    {
        const int *in = (const int *) input_items[0];
        char *out = (char *) output_items[0];

        // Do <+signal processing+>
        for(int i = 0; i < noutput_items; i++)
        {
            printf("General work\n\r");
        }/* for < noutput_items */

        // Tell runtime system how many input items we consumed on
        // each input stream.
        consume_each (noutput_items);

        // Tell runtime system how many output items we produced.
        return noutput_items;
    } /* general work */

    } /* namespace adsb */
} /* namespace gr */`

我得到的问题是,当我尝试编译时,我收到此错误:

In constructor ‘gr::adsb::out_impl::out_impl()’:
error: argument of type ‘void* (gr::adsb::out_impl::)(void*)’ does not match ‘void* (*)(void*)’

此错误指的是行,它涉及task_UdpRx:

if(pthread_create(&Thread_UdpRx, NULL, task_UdpRx, NULL))

有人有任何想法吗?

如果需要,请不要犹豫,询问更多细节。我显示的代码是我能做的最短的代码,以便您尽可能地了解它。

谢谢!

1 个答案:

答案 0 :(得分:2)

您无法将指向成员函数的指针传递给pthread_create()

你需要一个免费的功能:

void *thread_start(void *object)
{
    gr::adsb::out_impl *object = reinterpret_cast<gr::adsb::out_impl *>(object);
    object.task_UpdRx(0);
    return 0;
}

或左右,您需要将此函数传递给pthread_create()并传递this指针作为数据参数:

if (pthread_create(&Thread_UdpRx, NULL, thread_start, this))

我保留误用reinterpret_cast<>()的权利 - 替换正确的演员表示法。我对在构造函数中启动线程有所保留,但这是应该工作的方案的大纲。


2014-06-25上的代码更新后

代码仍然不是MCVE。我无法复制并编译它。丢失了许多标头。未定义/声明类型outout_impl。可能会遗漏一些GNU Radio继承内容。

这是我的MCVE版本。它并不完美,因为它不会重现您的问题。它使用GCC 4.9.0在Ubuntu 12.04 LTS衍生产品上完全编译。它假设您有一个标题<err.h>,用于定义函数err()。我不确定破坏者是否是MCVE所必需的(并且删除它会消除另外5行左右)。

来源

#include <cstdio>
#include <pthread.h>
#include <unistd.h>
#include <err.h>

namespace gr
{
    namespace adsb
    {
        class out_impl
        {
        public:
            out_impl();
            virtual ~out_impl();
        };

        void *task_UdpRx(void *arg)
        {
            while (true)
            {
                printf("Task UdpRx %p\n\r", arg);
                usleep(500*1000);
            }
            pthread_exit(NULL);
        }

        out_impl::out_impl()
        {
            pthread_t Thread_UdpRx;

            if (pthread_create(&Thread_UdpRx, NULL, task_UdpRx, NULL))
                err(1, "Pthread error");
            else
                printf("UDP thread initialization completed\n\r");
        }

        out_impl::~out_impl()
        {
        }

    }
}

汇编

$ g++ --version
g++ (GCC) 4.9.0
Copyright (C) 2014 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

$ make gr.o
g++ -g -O3 -std=c++11 -Wall -Wextra -Werror -c gr.cpp
$ nm -C -g gr.o
0000000000000010 T gr::adsb::task_UdpRx(void*)
0000000000000050 T gr::adsb::out_impl::out_impl()
0000000000000050 T gr::adsb::out_impl::out_impl()
0000000000000040 T gr::adsb::out_impl::~out_impl()
0000000000000000 T gr::adsb::out_impl::~out_impl()
0000000000000000 T gr::adsb::out_impl::~out_impl()
0000000000000000 V typeinfo for gr::adsb::out_impl
0000000000000000 V typeinfo name for gr::adsb::out_impl
                 U vtable for __cxxabiv1::__class_type_info
0000000000000000 V vtable for gr::adsb::out_impl
                 U operator delete(void*)
                 U err
                 U printf
                 U pthread_create
                 U usleep
$

后续步骤

这个Q&amp; A有几种方法可以从这里开始:

  1. 问题被放弃或关闭。
  2. 您将获取MCVE代码并添加内容,直到您收到原始编译错误。
  3. 您将获取非编译代码并移除内容,直到您进入MCVE,您可以从中删除任何内容而不删除错误。优选地,代码不应引用常规Linux安装中找不到的任何标头。如果必须,你需要确定它们可以从哪里获得 - 是的,如果必须的话,我可能会找到GNU Radio,但我不需要这样做。