[Paraview-developers] Find Data

[David Thompson]

Hi all (but especially Alan & Utkarsh),

I'm fiddling with the Find Data dialog and am curious if the intended behavior is to allow multiple open dialogs? Because clicking on the "Find Data" toolbar button or "Edit->Find Data..." menu item opens a new (different) dialog each time. The code that does it (Qt/ApplicationComponents/pqDataQueryReaction.cxx's showQueryDialog()) also does some weird Qt event loop stuff which appears to increase the call stack depth for no real reason.

If the behavior is not intentional, I'd like to change things to keep track of a singleton. Even if it is intentional, it seems to me that we should not call QEventLoop::exec() from within showQueryDialog(). Do you agree, or is the extra 20 stack frames not worth changing things?

	Thanks,
	David