Thanks Roland, I’ll review your feedback and look a bit further.
For the record, I’m trying to build this for the Qt environment and the source is v2.1.0rc0-615-gf38bd00 from master Thanks and regards…Paul From: [email protected] [mailto:[email protected]] On Behalf Of Roland Knall Sent: 27 November 2015 09:41 To: Developer support list for Wireshark <[email protected]> Subject: Re: [Wireshark-dev] plugin_if menu problems 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 On Fri, Nov 27, 2015 at 8:27 AM, Paul Offord <[email protected]<mailto:[email protected]>> wrote: I can answer part of the problem myself. Adding #define WS_BUILD_DLL to the head of the file stops the dllimport issue. Best regards…Paul From: Paul Offord Sent: 26 November 2015 22:37 To: '[email protected]<mailto:[email protected]>' <[email protected]<mailto:[email protected]>> Subject: plugin_if menu problems Hi, I am experimenting with the plugin_if menu functions. I’m creating the code in VS2013 and I have two stubborn errors that I can’t seem to resolve. The code is: #include <plugin_if.h> extern int proto_tmsvc; void menu_setup() { ext_menu_t * ext_menu, *os_menu = NULL; ext_menu = ext_menubar_register_menu( proto_tmsvc, "TestMenu", TRUE); ext_menubar_add_entry(ext_menu, "Jump 55", "This button causes Wireshark to jump to packet 55", menu_cb, "55"); } WS_DLL_PUBLIC_DEF void menu_cb(gpointer user_data) { } I get two errors: IntelliSense: argument of type "void (*)(gpointer user_data)" is incompatible with parameter of type "ext_menubar_action_cb" – referring to the menu_cb parameter in the call to ext_menubar_add_entry IntelliSense: a function declared 'dllimport' may not be defined – referring to the menu_cb function definition Any advice much appreciated. Thanks and regards…Paul ______________________________________________________________________ This message contains confidential information and is intended only for the individual named. If you are not the named addressee you should not disseminate, distribute or copy this e-mail. Please notify the sender immediately by e-mail if you have received this e-mail by mistake and delete this e-mail from your system. Any views or opinions expressed are solely those of the author and do not necessarily represent those of Advance Seven Ltd. E-mail transmission cannot be guaranteed to be secure or error-free as information could be intercepted, corrupted, lost, destroyed, arrive late or incomplete, or contain viruses. The sender therefore does not accept liability for any errors or omissions in the contents of this message, which arise as a result of e-mail transmission. Advance Seven Ltd. Registered in England & Wales numbered 2373877 at Endeavour House, Coopers End Lane, Stansted, Essex CM24 1SJ ______________________________________________________________________ This email has been scanned by the Symantec Email Security.cloud service. For more information please visit http://www.symanteccloud.com ______________________________________________________________________ ___________________________________________________________________________ Sent via: Wireshark-dev mailing list <[email protected]<mailto:[email protected]>> Archives: https://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev mailto:[email protected]<mailto:[email protected]>?subject=unsubscribe ______________________________________________________________________ This message contains confidential information and is intended only for the individual named. If you are not the named addressee you should not disseminate, distribute or copy this e-mail. Please notify the sender immediately by e-mail if you have received this e-mail by mistake and delete this e-mail from your system. Any views or opinions expressed are solely those of the author and do not necessarily represent those of Advance Seven Ltd. E-mail transmission cannot be guaranteed to be secure or error-free as information could be intercepted, corrupted, lost, destroyed, arrive late or incomplete, or contain viruses. The sender therefore does not accept liability for any errors or omissions in the contents of this message, which arise as a result of e-mail transmission. Advance Seven Ltd. Registered in England & Wales numbered 2373877 at Endeavour House, Coopers End Lane, Stansted, Essex CM24 1SJ ______________________________________________________________________ This email has been scanned by the Symantec Email Security.cloud service. For more information please visit http://www.symanteccloud.com ______________________________________________________________________
___________________________________________________________________________ Sent via: Wireshark-dev mailing list <[email protected]> Archives: https://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev mailto:[email protected]?subject=unsubscribe
