]> git.lyx.org Git - features.git/commitdiff
remove assert
authorAndré Pönitz <poenitz@gmx.net>
Sat, 16 Jul 2005 09:40:16 +0000 (09:40 +0000)
committerAndré Pönitz <poenitz@gmx.net>
Sat, 16 Jul 2005 09:40:16 +0000 (09:40 +0000)
git-svn-id: svn://svn.lyx.org/lyx/lyx-devel/trunk@10235 a592a061-630c-0410-9148-cb99ea01b6c8

src/ChangeLog
src/paragraph.h

index aa6f7ff96882b5d32ae1c74cc285125ca8c5dc4d..d34ed386b5d85f33f0a9c343c88661271ee94449 100644 (file)
@@ -1,3 +1,10 @@
+
+2005-07-16  André Pönitz  <poenitz@gmx.net>
+
+       * paragraph.h (getChar): remove asserts as the same kind of security
+  can be achieved by using a suitable standard library implementation
+  and the function was in the critical path (25.9% in profiler)
+
 2005-07-16  Juergen Vigna <jug@lyx.org>
 
        * cursor.C (bruteFind2):
@@ -17,7 +24,7 @@
        * lyxfunc.C (dispatch): save and restore the cursor after a
        textclass switch.
 
-2005-07-15    <lgb@tandberg.net>
+2005-07-15  Lars Gullik Bjønnes <lgb@tandberg.net>
 
        * text.C (backspace): bug 806 for empty keepempty paragraphs we
        change the layout to default before letting DEPM do its thing.
index 14c9cc156eaecb16d30a4320f999e2faf81c44df..b16ac0d5b0c85b34501722bc61a1182aafcd2f90 100644 (file)
@@ -26,8 +26,6 @@
 
 #include "support/types.h"
 
-#include <boost/assert.hpp>
-
 #include <string>
 #include <utility>
 
@@ -288,12 +286,7 @@ public:
        std::pair<lyx::pos_type, lyx::pos_type> getFontSpan(lyx::pos_type pos) const;
        ///
        /// this is a bottleneck.
-       value_type getChar(lyx::pos_type pos) const
-       {
-               BOOST_ASSERT(pos >= 0);
-               BOOST_ASSERT(pos < int(text_.size()));
-               return text_[pos];
-       }
+       value_type getChar(lyx::pos_type pos) const { return text_[pos]; }
        /// Get the char, but mirror all bracket characters if it is right-to-left
        value_type getUChar(BufferParams const &, lyx::pos_type pos) const;
        /// The position must already exist.