Comment.

[lyx.git] / src / Bullet.cpp

diff --git a/src/Bullet.cpp b/src/Bullet.cpp
index 5c0f91741991387208e4ee88454c91b94171ba45..da946dcbdce4e9ed6391f16ae87622cebfcc2075 100644 (file)
--- a/src/Bullet.cpp
+++ b/src/Bullet.cpp
@@ -20,9 +20,9 @@
 
 #include <boost/assert.hpp>
 
-namespace lyx {
+using namespace std;
 
-using std::string;
+namespace lyx {
 
 
 /** The four LaTeX itemize environment default bullets
@@ -41,15 +41,12 @@ Bullet const ITEMIZE_DEFAULTS[4] = { Bullet(0, 8),//"\\(\\bullet\\)"
 Bullet::Bullet(int f, int c, int s)
        : font(f), character(c), size(s), user_text(0)
 {
-       if (f < MIN || f >= FONTMAX) {
+       if (f < MIN || f >= FONTMAX)
                font = MIN;
-       }
-       if (c < MIN || c >= CHARMAX) {
+       if (c < MIN || c >= CHARMAX)
                character = MIN;
-       }
-       if (s < MIN || s >= SIZEMAX) {
+       if (s < MIN || s >= SIZEMAX)
                size = MIN;
-       }
        generateText();
        testInvariant();
 }
@@ -65,11 +62,10 @@ Bullet::Bullet(docstring const & t)
 
 void Bullet::setCharacter(int c)
 {
-       if (c < MIN || c >= CHARMAX) {
+       if (c < MIN || c >= CHARMAX)
                character = MIN;
-       } else {
+       else
                character = c;
-       }
        user_text = 0;
        testInvariant();
 }
@@ -77,11 +73,10 @@ void Bullet::setCharacter(int c)
 
 void Bullet::setFont(int f)
 {
-       if (f < MIN || f >= FONTMAX) {
+       if (f < MIN || f >= FONTMAX)
                font = MIN;
-       } else {
+       else
                font = f;
-       }
        user_text = 0;
        testInvariant();
 }
@@ -89,11 +84,10 @@ void Bullet::setFont(int f)
 
 void Bullet::setSize(int s)
 {
-       if (s < MIN || s >= SIZEMAX) {
+       if (s < MIN || s >= SIZEMAX)
                size = MIN;
-       } else {
+       else
                size = s;
-       }
        user_text = 0;
        testInvariant();
 }
@@ -141,9 +135,8 @@ Bullet & Bullet::operator=(Bullet const & b)
 
 docstring const & Bullet::getText() const
 {
-       if (user_text == 0) {
+       if (user_text == 0)
                generateText();
-       }
        return text;
 }
 
@@ -154,12 +147,10 @@ bool operator==(const Bullet & b1, const Bullet & b2)
 
        if (b1.user_text && b2.user_text) {
                /* both have valid text */
-               if (b1.text == b2.text) {
+               if (b1.text == b2.text)
                        result = true;
-               }
-       } else if (((b1.character == b2.character) &&
-                         (b1.font == b2.font)) &&
-                        (b1.size == b2.size)) {
+       } else if (b1.character == b2.character && b1.font == b2.font &&
+                        b1.size == b2.size) {
                result = true;
        }
        return result;
@@ -185,9 +176,8 @@ void Bullet::generateText() const
 
        if ((font >= 0) && (character >= 0)) {
                text = bulletEntry(font, character);
-               if (size >= 0) {
+               if (size >= 0)
                        text = bulletSize(size) + text;
-               }
                user_text = -1;
                // text is now defined and doesn't need to be recalculated
                // unless font/character or text is modified