The bell is dead

git-svn-id: svn://svn.lyx.org/lyx/lyx-devel/trunk@1344 a592a061-630c-0410-9148-cb99ea01b6c8

ChangeLog

@@ -1,3 +1,9 @@
+2001-01-15  Jean-Marc Lasgouttes  <Jean-Marc.Lasgouttes@inria.fr>
+
+	* INSTALL (Problems): add tip about using --with-included-string
+	on solaris 2.6 with gcc 2.95.2 (ld does not like long symbol
+	names). 
+
 2001-01-12  Lars Gullik Bjønnes  <larsbj@lyx.org>
 
 	* ChangeLog: store old and begin new

INSTALL

@@ -317,7 +317,8 @@ notify us.
 
   o On Tru64 Unix, you may have to compile with
     --with-included-string to work around a Tru64 linker limitation
-    (the STL string template creates names which may be too long).
+    (the STL string template creates names which may be too long). We
+    also had reports that it helps with gcc 2.95.2 on solaris 2.6. 
 
   o On Tru64 Unix with cxx, you may have a compilation error in
     lyx_main.C if you have GNU gettext installed. This is due to a bug

src/BufferView2.C

@@ -644,11 +644,9 @@ void BufferView::gotoInset(std::vector<Inset::Code> const & codes,
 				if (!text->GotoNextInset(this, codes, contents)) {
 					text->cursor = tmp;
 					owner()->getMiniBuffer()->Set(_("No more insets"));
-					LyXBell();
 				}
 			} else {
 				owner()->getMiniBuffer()->Set(_("No more insets"));
-				LyXBell();
 			}
 	}
 	update(BufferView::SELECT|BufferView::FITCUR);

src/ChangeLog

@@ -1,3 +1,10 @@
+2000-01-17  John Levon  <moz@compsoc.man.ac.uk>
+
+	* BufferView2.C:
+	* lyx_gui_misc.h:
+	* lyxfr1.C:
+	* lyxfunc.C: kill LyXBell.
+
 2001-01-17  Dekel Tsur  <dekelts@tau.ac.il>
 
 	* WorkArea.C (work_area_handler): Decrease keyboard purge thershold.

src/insets/ChangeLog

@@ -0,0 +1,3 @@
+2001-01-17  John Levon  <moz@compsoc.man.ac.uk>
+
+	* insettext.C: kill LyXBell.

src/insets/insettext.C

@@ -783,7 +783,6 @@ InsetText::LocalDispatch(BufferView * bv,
 	// Normal chars
     case LFUN_UNKNOWN_ACTION:
 	if (bv->buffer()->isReadonly()) {
-	    LyXBell();
 //	    setErrorMessage(N_("Document is read only"));
 	    break;
 	}

src/lyx_gui_misc.h

@@ -78,11 +78,4 @@ void WarnReadonly(string const & file);
 /// Get the dpi setting of the current screen
 float getScreenDPI();
 
-/// rings the audio bell.
-inline
-void LyXBell() {
-	// if (audio()) ON/OFF switch yet to be implemented
-	//fl_ringbell(20);
-}
-
 #endif

src/lyxfr1.C

@@ -187,7 +187,6 @@ void LyXFindReplace::SearchReplaceAllCB()
 		}
 	} while (SearchCB(true));
 	if (replace_count == 0) {
-		LyXBell();	
 		bv->owner()->getMiniBuffer()->Set(
 			_("String not found!"));
 	} else {
@@ -235,7 +234,6 @@ bool LyXFindReplace::SearchCB(bool fForward)
 		bv->owner()->getMiniBuffer()->Set(_("Found."));
 		result = true;
 	} else {
-		LyXBell();
 		bv->owner()->getMiniBuffer()->Set(_("String not found!"));
 		result = false;
 	}

src/lyxfunc.C

@@ -252,7 +252,6 @@ int LyXFunc::processKeySym(KeySym keysym, unsigned int state)
 	if (action == -1) {
 		if (keyseq.length < -1) { // unknown key sequence...
 			string buf;
-			LyXBell();
 			keyseq.print(buf);
 			owner->getMiniBuffer()->Set(_("Unknown sequence:"), buf);
 			return 0;
@@ -711,8 +710,7 @@ string const LyXFunc::Dispatch(int ac,
 			// set the new selection 
 			// SetSelectionOverLenChars(owner->view()->currentBuffer()->text, iLenSelected);
 			owner->view()->toggleSelection(false);
-		} else 
+		}
-			LyXBell();	
 	 
 		// REMOVED : if (owner->view()->getWorkArea()->focus)
 		owner->view()->showCursor();
@@ -2915,7 +2913,6 @@ string const LyXFunc::Dispatch(int ac,
 		string lyx_name;
 		string const x11_name = split(argument, lyx_name, ' ');
 		if (lyx_name.empty() || x11_name.empty()) {
-			LyXBell();
 			setErrorMessage(N_("Syntax: set-color <lyx_name>"
 						" <x11_name>"));
 			break;
@@ -2926,7 +2923,6 @@ string const LyXFunc::Dispatch(int ac,
 			static string const err2 (
 				N_("\" failed - color is undefined "
 				   "or may not be redefined"));
-			LyXBell();
 			setErrorMessage(_(err1) + lyx_name + _(err2));
 			break;
 		}
@@ -2938,13 +2934,11 @@ string const LyXFunc::Dispatch(int ac,
 	case LFUN_UNKNOWN_ACTION:
 	{
 		if (!owner->buffer()) {
-			LyXBell();
 			setErrorMessage(N_("No document open"));
 			break;
 		}
 
 		if (owner->buffer()->isReadonly()) {
-			LyXBell();
 			setErrorMessage(N_("Document is read only"));
 			break;
 		}
@@ -2999,7 +2993,6 @@ string const LyXFunc::Dispatch(int ac,
 			// why is an "Unknown action" with empty
 			// argument even dispatched in the first
 			// place? I`ll probably change that. (Lgb)
-			LyXBell();
 			setErrorMessage(N_("Unknown action"));
 		}
 		break;