Package: amule
Version: 2.2.0~20080309-1
Severity: normal
Tags: patch

Please could you remove the "do you really want to exit" query?

  - It is IMO an annoying, and mostly useless nag (session state is
    preserved, so it makes no harm to exit by mistake).

  - It's inconsistent with what all the other applications do.

  - It integrated badly with the desktop (at least in GNOME), as it triggers
    the "this application is not responding" dialog very often.

Thanks

-- System Information:
Debian Release: lenny/sid
  APT prefers unstable
  APT policy: (500, 'unstable')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.18-6-amd64 (SMP w/2 CPU cores)
Locale: LANG=ca_AD.UTF-8, LC_CTYPE=ca_AD.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/bash
--- src/amuleDlg.cpp~	2008-03-04 12:23:32.000000000 +0100
+++ src/amuleDlg.cpp	2008-04-17 14:38:12.000000000 +0200
@@ -846,15 +846,6 @@
 
 void CamuleDlg::OnClose(wxCloseEvent& evt)
 {
-	// This will be here till the core close is != app close
-	if (evt.CanVeto() && thePrefs::IsConfirmExitEnabled() ) {
-		if (wxNO == wxMessageBox(wxString(_("Do you really want to exit aMule?")),
-				wxString(_("Exit confirmation")), wxYES_NO, this)) {
-			evt.Veto();
-			return;
-		}
-	}
-	
 	theApp->ShutDown(evt);
 }
 

Reply via email to