Hi
I do not have anything standing in front of my callbacks, therefore I would remove WS_DLL_PUBLIC_DEF completely. It is not necessary in your case.
Secondly, if you are using the latest git source of Wireshark or the 2.0 source, the correct prototype for the callback would be:
void menu_cb(ext_menubar_gui_type gui_type, gpointer gui_data, gpointer user_data _U_)
In which gui_type is defined as GTK or Qt to allow different callback handling for each interface (which is needed if you are opening a dialog window for example).
My code for opening the Qt Dialog is as follows:
void GuiHandler::executeDialog(QDialog * dialog)
{
bool hasGuiApp = (qobject_cast<QApplication*>(QCoreApplication::instance())!=0);
if ( ! hasGuiApp )
{
/* Necessity for creating the correct app context */
int argc = 1;
char * argv = (char *) "Test";
/* In Gtk there is no application context, must be created and displayed */
QApplication app(argc, &argv);
dialog->show();
app.exec();
}
else
{
/* With Wireshark Qt, an application context already exists, therefore just
* displaying the dialog */
dialog->exec();
}
}
hasGuiApp searches for an existing QApplication object, which may only be instated once for each process, and will already exist for WS 2.0, otherwise it creates a QApplication context and opens the dialog. If the application exists, it simply executes the dialog.
Hope this helps, regards,
Roland
