Make Insert Symbol Dialog modeless under WIN32

CVS: ----------------------------------------------------------------------
CVS: Enter Log.  Lines beginning with `CVS:' are removed automatically
CVS:
CVS: Committing in .
CVS:
CVS: Modified Files:
CVS:    af/xap/beos/xap_BeOSDlg_Insert_Symbol.h
CVS:    af/xap/mac/xap_MacDlg_Insert_Symbol.h
CVS:    af/xap/qnx/xap_QNXDlg_Insert_Symbol.h
CVS:  af/xap/unix/xap_UnixDlg_Insert_Symbol.h
CVS:    af/xap/win/xap_Win32Dlg_Insert_Symbol.cpp
CVS:    af/xap/win/xap_Win32Dlg_Insert_Symbol.h af/xap/xp/xap_App.cpp
CVS:    af/xap/xp/xap_App.h af/xap/xp/xap_Dialog.cpp
CVS:    af/xap/xp/xap_Dialog.h af/xap/xp/xap_Dlg_Insert_Symbol.cpp
CVS:    af/xap/xp/xap_Dlg_Insert_Symbol.h wp/ap/xp/ap_EditMethods.cpp
CVS:    wp/main/beos/paragraph.rsrc
CVS: ----------------------------------------------------------------------

Martin,

        Why is the function UT_Bool XAP_Dialog_Modeless::isRunning(void) not a
virtual function.
        Storing a widget pointer in XAP_App is not cross-platform.

        Making isRunning virtual means that calling isRunning will call the
platform specific code to determine if the dialog is running or not.

        EG.

        UT_Bool XAP_Win32Dialog_Insert_Symbol::isRunning(void)
        {
                return m_hDlg != NULL;
        }

        or

        UT_Bool XAP_UnixDialog_Insert_Symbol::isRunning(void)
        {
                return mainWindow != NULL;
        }


Bruce.




Reply via email to