]> git.lyx.org Git - features.git/blobdiff - src/mathed/math_defs.h
make doc++ able to generate the source documentation for lyx
[features.git] / src / mathed / math_defs.h
index cd464b5e235f0e5f30a656cd1c4bfca6bfff6ac3..bebf604dafddadf50199ebf0953b803004ef3767 100644 (file)
@@ -8,7 +8,7 @@
  *
  *  Dependencies: Xlib
  *
- *  Copyright: (c) 1996, 1997 Alejandro Aguilar Sierra
+ *  Copyright: 1996, 1997 Alejandro Aguilar Sierra
  *
  *   Version: 0.8beta, Mathed & Lyx project.
  *
  * 
  */
 
-#ifndef __MATH_DEFS__
-#define __MATH_DEFS__
+#ifndef MATH_DEFS
+#define MATH_DEFS
 
 #ifdef __GNUG__
 #pragma interface
 #endif
 
-#include <stdio.h>
+#include <iosfwd>
+
+#include "LString.h"
+#include "debug.h"
+
 #include "array.h"
 
+class Painter;
+
 ///
-#define MATH_ALIGN_LEFT    1
-///
-#define MATH_ALIGN_RIGHT   2
-///
-#define MATH_ALIGN_BOTTOM  4
-///
-#define MATH_ALIGN_TOP     8
+enum math_align {
+       ///
+       MATH_ALIGN_LEFT = 1,
+       ///
+       MATH_ALIGN_RIGHT = 2,
+       ///
+       MATH_ALIGN_BOTTOM = 4,
+       ///
+       MATH_ALIGN_TOP = 8
+};
 ///
 #define MATH_COLSEP 8
 ///
 /// Standard Math Sizes (Math mode styles)
 enum MathedStyles {
        ///
-   LM_ST_DISPLAY=0,
-   ///
-   LM_ST_TEXT,
-   ///
-   LM_ST_SCRIPT,
-   ///
-   LM_ST_SCRIPTSCRIPT
+       LM_ST_DISPLAY = 0,
+       ///
+       LM_ST_TEXT,
+       ///
+       LM_ST_SCRIPT,
+       ///
+       LM_ST_SCRIPTSCRIPT
 };
 
 
 /// Standard LaTeX Math Environments
 enum MathedEnvironment {
        ///
-   LM_EN_INTEXT=0,
-   ///
-   LM_EN_DISPLAY,
-   ///
-   LM_EN_EQUATION,
-   ///
-   LM_EN_EQNARRAYAST,
-   ///
-   LM_EN_EQNARRAY,
-   ///
-   LM_EN_ARRAY
+       LM_EN_INTEXT = 0,
+       ///
+       LM_EN_DISPLAY,
+       ///
+       LM_EN_EQUATION,
+       ///
+       LM_EN_EQNARRAYAST,
+       ///
+       LM_EN_EQNARRAY,
+       ///
+       LM_EN_ARRAY
 };
 
 
 /** The restrictions of a standard LaTeX math paragraph
   allows to get a small number of text codes (<30) */
 enum MathedTextCodes  {
-   /// This must be >= 0
-   LM_TC_MIN = 0,
-   /// Open and Close group
-   LM_TC_OPEN,
-   ///
-   LM_TC_CLOSE,
-   /// Tabulator
-   LM_TC_TAB,
-   /// New line
-   LM_TC_CR,
-   /// Math Inset
-   LM_TC_INSET,
-   /// Super and sub scripts
-   LM_TC_UP,
-   ///
-   LM_TC_DOWN,
-   /// Editable Math Inset
-   LM_TC_ACTIVE_INSET,
-   /// Editable Text Inset
-   LM_TC_TEXT_INSET,
-   ///
-   LM_FONT_BEGIN,
-   /// Internal code for constants
-   LM_TC_CONST,
-   /// Internal code for variables
-   LM_TC_VAR,
-   ///
-   LM_TC_RM,
-   ///
-   LM_TC_CAL,
-   ///
-   LM_TC_BF,
-   ///
-   LM_TC_SF,
-   ///
-   LM_TC_TT,
-   ///
-   LM_TC_IT,
-   ///
-   LM_TC_TEXTRM,
-   /// Math mode TeX characters ",;:{}"
-   LM_TC_TEX,
-   /// Special characters "{}&#_%"
-   LM_TC_SPECIAL,
-   /// Internal code for operators
-   LM_TC_BOP,
-   /// Internal code for symbols
-   LM_TC_SYMB,
-   ///
-   LM_TC_BOPS,
-   ///
-   LM_TC_BSYM,
-   ///
-   LM_FONT_END,
-   
-   /// This must be < 32 
-   LM_TC_MAX
+       /// This must be >= 0
+       LM_TC_MIN = 0,
+       /// Open and Close group
+       LM_TC_OPEN,
+       ///
+       LM_TC_CLOSE,
+       /// Tabulator
+       LM_TC_TAB,
+       /// New line
+       LM_TC_CR,
+       /// Math Inset
+       LM_TC_INSET,
+       /// Super and sub scripts
+       LM_TC_UP,
+       ///
+       LM_TC_DOWN,
+       /// Editable Math Inset
+       LM_TC_ACTIVE_INSET,
+       /// Editable Text Inset
+       LM_TC_TEXT_INSET,
+       ///
+       LM_FONT_BEGIN,
+       /// Internal code for constants
+       LM_TC_CONST,
+       /// Internal code for variables
+       LM_TC_VAR,
+       ///
+       LM_TC_RM,
+       ///
+       LM_TC_CAL,
+       ///
+       LM_TC_BF,
+       ///
+       LM_TC_SF,
+       ///
+       LM_TC_TT,
+       ///
+       LM_TC_IT,
+       ///
+       LM_TC_TEXTRM,
+       /// Math mode TeX characters ",;:{}"
+       LM_TC_TEX,
+       /// Special characters "{}&#_%"
+       LM_TC_SPECIAL,
+       /// Internal code for operators
+       LM_TC_BOP,
+       /// Internal code for symbols
+       LM_TC_SYMB,
+       ///
+       LM_TC_BOPS,
+       ///
+       LM_TC_BSYM,
+       ///
+       LM_FONT_END,
+       
+       /// This must be < 32 
+       LM_TC_MAX
 };
 
+///
+std::ostream & operator<<(std::ostream &, MathedTextCodes mtc);
+
 ///
 #define LM_TC_NORMAL LM_TC_VAR
  
        
 /// Types of lyx-math insets 
 enum MathedInsetTypes  {
-   ///
-   LM_OT_MIN = 0,
-   /// A simple paragraph
-   LM_OT_PAR,
-   /// A simple numbered paragraph
-   LM_OT_PARN,
-   /// A multiline paragraph
-   LM_OT_MPAR,
-   /// A multiline numbered paragraph
-   LM_OT_MPARN,
-   /// An array
-   LM_OT_MATRIX,
-   /// A big operator
-   LM_OT_BIGOP,
-   /// A LaTeX macro
-   LM_OT_UNDEF,
-   ///
-   LM_OT_FUNC,
-   ///
-   LM_OT_FUNCLIM,
-   ///
-   LM_OT_SCRIPT,
-   ///
-   LM_OT_SPACE,
-   ///
-   LM_OT_DOTS,
-   /// A fraction
-   LM_OT_FRAC,
-   ///
-   LM_OT_ATOP,
-   ///
-   LM_OT_STACKREL,
-   /// A radical
-   LM_OT_SQRT,
-   /// A delimiter
-   LM_OT_DELIM,
-   /// A decoration
-   LM_OT_DECO,
-   /// An accent
-   LM_OT_ACCENT,
-   ///
-   LM_OT_MACRO,
-   ///
-   LM_OT_MACRO_ARG,
-   ///
-   LM_OT_MAX
+       ///
+       LM_OT_MIN = 0,
+       /// A simple paragraph
+       LM_OT_PAR,
+       /// A simple numbered paragraph
+       LM_OT_PARN,
+       /// A multiline paragraph
+       LM_OT_MPAR,
+       /// A multiline numbered paragraph
+       LM_OT_MPARN,
+       /// An array
+       LM_OT_MATRIX,
+       /// A big operator
+       LM_OT_BIGOP,
+       /// A LaTeX macro
+       LM_OT_UNDEF,
+       ///
+       LM_OT_FUNC,
+       ///
+       LM_OT_FUNCLIM,
+       ///
+       LM_OT_SCRIPT,
+       ///
+       LM_OT_SPACE,
+       ///
+       LM_OT_DOTS,
+       /// A fraction
+       LM_OT_FRAC,
+       ///
+       LM_OT_ATOP,
+       ///
+       LM_OT_STACKREL,
+       /// A radical
+       LM_OT_SQRT,
+       /// A delimiter
+       LM_OT_DELIM,
+       /// A decoration
+       LM_OT_DECO,
+       /// An accent
+       LM_OT_ACCENT,
+       ///
+       LM_OT_MACRO,
+       ///
+       LM_OT_MACRO_ARG,
+       ///
+       LM_OT_MAX
 };
 
 ///
 enum MathedBinaryTypes {
        ///
-    LMB_NONE = 0,
-    ///
-    LMB_RELATION,
-    ///
-    LMB_OPERATOR,
-    ///
-    LMB_BOP = (LMB_RELATION | LMB_OPERATOR)
+       LMB_NONE = 0,
+       ///
+       LMB_RELATION,
+       ///
+       LMB_OPERATOR,
+       ///
+       LMB_BOP = (LMB_RELATION | LMB_OPERATOR)
 };
 
-class LString;
-class MathedInset;
 class MathParInset;
 
-
 /** Abstract base class for all math objects.
     A math insets is for use of the math editor only, it isn't a
     general LyX inset. It's used to represent all the math objects.
@@ -212,25 +221,24 @@ class MathParInset;
 class MathedInset  {
  public: 
     /// A math inset has a name (usually its LaTeX name), type and font-size
-    MathedInset(char const *nm, short ot, short st);
+    MathedInset(char const * nm, short ot, short st);
     ///
-    MathedInset(MathedInset*);
+    explicit
+    MathedInset(MathedInset *);
     ///
-    virtual ~MathedInset() { };
-    
-    /// Draw the object 
-    virtual void Draw(int x, int baseline)=0;
-    
-    /// Write LaTeX and Lyx code
-    virtual void Write(FILE *file)=0;
+    virtual ~MathedInset() {}
+
+    /// Draw the object
+    virtual void draw(Painter &, int x, int baseline) = 0;     
+
     /// Write LaTeX and Lyx code
-    virtual void Write(LString &file)=0;
-   
+    virtual void Write(std::ostream &, bool fragile) = 0;
+
     /// Reproduces itself
-    virtual MathedInset *Clone()=0;
+    virtual MathedInset * Clone() = 0;
    
     /// Compute the size of the object
-    virtual void Metrics()=0; 
+    virtual void Metrics() = 0; 
     /// 
     virtual int Ascent() const { return ascent; }
     ///
@@ -243,10 +251,10 @@ class MathedInset  {
     ///
     virtual bool GetLimits() const { return false; }
     ///
-    virtual void SetLimits(bool) { }   
+    virtual void SetLimits(bool) {}   
    
     ///
-    char const *GetName() const { return name; }
+    char const * GetName() const { return name; }
     ///
     short GetType() const { return objtype; }
     ///
@@ -258,13 +266,10 @@ class MathedInset  {
     ///
     virtual void  SetStyle(short st) { size = st; } // Metrics();
     ///
-    virtual void  SetName(char const* n) { name = n; }
-    /// 
-    void setDrawable(long unsigned int d) { pm = d; }
+    virtual void  SetName(char const * n) { name = n; }
  protected:
     ///
-    char const *name;
+    char const * name;
     ///
     short objtype;
     ///
@@ -275,17 +280,17 @@ class MathedInset  {
     int descent;
     ///
     short size;
-    /// This works while only one process can draw unless
-    /// the process have their own data
-    static long unsigned int pm;
     /// Default metrics
-    static int df_asc, df_des, df_width;
-
+    static int df_asc;
+    ///
+    static int df_des;
+    ///
+    static int df_width;
     /// In a near future maybe we use a better fonts renderer than X
-    void drawStr(short, int, int, int, byte*, int);
-       ///
+    void drawStr(Painter &, short, int, int, int, byte const *, int);
+    ///
     friend class MathedCursor;
-       ///
+    ///
     friend void mathed_init_fonts();
 };
 
@@ -294,19 +299,19 @@ struct MathedRowSt;
 
 /// Paragraph permissions
 enum MathedParFlag {
-    LMPF_BASIC = 0,
-    /// If false can use a non-standard size
-    LMPF_FIXED_SIZE = 1,
-    /// If true can insert newlines 
-    LMPF_ALLOW_CR  = 2,
-    /// If true can use tabs
-    LMPF_ALLOW_TAB = 4,
-    /// If true can insert new columns
-    LMPF_ALLOW_NEW_COL = 8,
-    /// Smaller than current size (frac)
-    LMPF_SMALLER = 16,
-    /// Script size (subscript, stackrel)
-    LMPF_SCRIPT = 32
+       LMPF_BASIC = 0,
+       /// If false can use a non-standard size
+       LMPF_FIXED_SIZE = 1,
+       /// If true can insert newlines 
+       LMPF_ALLOW_CR  = 2,
+       /// If true can use tabs
+       LMPF_ALLOW_TAB = 4,
+       /// If true can insert new columns
+       LMPF_ALLOW_NEW_COL = 8,
+       /// Smaller than current size (frac)
+       LMPF_SMALLER = 16,
+       /// Script size (subscript, stackrel)
+       LMPF_SCRIPT = 32
 };
 
 
@@ -314,21 +319,22 @@ enum MathedParFlag {
 class MathParInset: public MathedInset  {
  public: 
     ///
-    MathParInset(short st=LM_ST_TEXT, char const *nm=NULL, short ot=LM_OT_MIN);
+    MathParInset(short st = LM_ST_TEXT, char const * nm = 0,
+                short ot = LM_OT_MIN);
     ///
-    MathParInset(MathParInset*);
+    explicit
+    MathParInset(MathParInset *);
     ///
     virtual ~MathParInset();
     ///
-    virtual MathedInset *Clone();   
+    virtual MathedInset * Clone();
 
     /// Draw the object on a drawable
-    virtual void Draw(int x, int baseline);
-   
-    /// Write LaTeX code
-    virtual void Write(FILE *file);
+    virtual void draw(Painter &, int x, int baseline);
+
     /// Write LaTeX code
-    virtual void Write(LString &file);
+    virtual void Write(std::ostream &, bool fragile);
+
     ///
     virtual void Metrics();
     ///
@@ -340,17 +346,17 @@ class MathParInset: public MathedInset  {
     virtual LyxArrayBase * GetData() { return array; }
 
     /// Paragraph position
-    virtual void GetXY(int&, int&) const;
+    virtual void GetXY(int &, int &) const;
     ///
-    virtual void setXY(int x, int y) { xo =x;  yo = y; }
+    virtual void setXY(int x, int y) { xo = x;  yo = y; }
     ///
-    virtual void SetFocus(int,int) { };
+    virtual void SetFocus(int, int) {}
     ///
     virtual bool Inside(int, int);   
    
     // Tab stuff used by Matrix.
     ///
-    virtual void SetAlign(char, char const*) { };
+    virtual void SetAlign(char, char const *) {}
 //    ///
 //    virtual int GetTabPos() { return 0; }
 //    ///
@@ -367,7 +373,7 @@ class MathParInset: public MathedInset  {
 
     // Vertical switching
     ///
-    virtual bool setArgumentIdx(int i) { return (i==0); }
+    virtual bool setArgumentIdx(int i) { return (i == 0); }
     ///
     virtual bool setNextArgIdx() { return false; }
     ///
@@ -375,61 +381,69 @@ class MathParInset: public MathedInset  {
     ///
     virtual int  getMaxArgumentIdx() { return 0; }
 //    ///
-//    virtual void SetLabel(char const*) { }
+//    virtual void SetLabel(char const *) {}
     ///
     virtual void SetStyle(short);
-       ///
-    virtual MathedRowSt *getRowSt() const { return 0; }
-       ///
-    virtual void setRowSt(MathedRowSt*) { }
-       ///
-    virtual bool Permit(short f) { return (bool)(f & flag); }
+    ///
+    virtual MathedRowSt * getRowSt() const { return 0; }
+    ///
+    virtual void setRowSt(MathedRowSt *) {}
+    ///
+    virtual bool Permit(short f) { return bool(f & flag); }
     
  protected:
     /// Paragraph data is stored here
-    LyxArrayBase *array;
+    LyxArrayBase * array;
     /// Cursor start position
-    int xo, yo;
+    int xo;
+    ///
+    int yo;
     /// 
     short flag;
 
  private:
     ///
     virtual void setFlag(MathedParFlag f) { flag |= f; }
-       ///
+    ///
     friend class InsetFormula;
-       ///
+    ///
     friend class MathedXIter;
-       ///
+    ///
     friend class MathedCursor;
-       ///
-    friend LyxArrayBase *mathed_parse(unsigned flags = 0, LyxArrayBase*a=NULL, MathParInset**p=NULL);
+    ///
+    friend LyxArrayBase * mathed_parse(unsigned flags = 0,
+                                      LyxArrayBase * a = 0,
+                                      MathParInset ** p = 0);
 };
 
 
-/* The physical structure of a row and aditional information is stored here.
+/** The physical structure of a row and aditional information is stored here.
     It allows to manage the extra info independently of the paragraph data.  
     Only used for multiline paragraphs.
  */
 struct MathedRowSt {    
-    /// 
+    ///
+    explicit
     MathedRowSt(int n) {
-           w = new int[n+1]; // this leaks
-       next = 0;
-       label = 0;
-       numbered = true;
+           w = new int[n + 1]; // this leaks
+           asc = desc = y = 0;
+           for (int i = 0 ; i < n + 1 ; ++i)
+             w[i] = 0;
+           next = 0;
+           label = 0;
+           numbered = true;
     }
     ///
     ~MathedRowSt() {
-       delete[] w;
-       if (label) delete[] label;
+           delete[] w;
+           delete[] label;
     }
     /// Should be const but...
-    MathedRowSt* getNext() const  { return next; }
+    MathedRowSt * getNext() const  { return next; }
     /// ...we couldn't use this.
-    void setNext(MathedRowSt* n) { next = n; }
+    void setNext(MathedRowSt * n) { next = n; }
     ///
-    char const* getLabel() const { return label; }
+    char const * getLabel() const { return label; }
     ///
     bool isNumbered() const { return numbered; }
     ///
@@ -437,7 +451,7 @@ struct MathedRowSt {
     ///
     int  getTab(int i) { return w[i]; }
     /// 
-    void setLabel(char* l) { label = l; }
+    void setLabel(char * l) { label = l; }
     ///
     void setNumbered(bool nf) { numbered = nf; }
     ///
@@ -447,16 +461,16 @@ struct MathedRowSt {
     /// Vericals 
     int asc, desc, y;
     /// widths 
-    int *w;
+    int * w;
     /// 
-    char *label;
+    char * label;
     ///
     bool numbered;
     ///
-    MathedRowSt *next;
-       ///
+    MathedRowSt * next;
+    ///
     friend class MathMatrixInset;
-       ///
+    ///
     friend class MathedXIter;
 };
 
@@ -468,27 +482,27 @@ struct MathedRowSt {
 class MathMatrixInset: public MathParInset {
  public: 
     ///
-    MathMatrixInset(int m=1, int n=1, short st=LM_ST_TEXT);
+    explicit
+    MathMatrixInset(int m = 1, int n = 1, short st = LM_ST_TEXT);
     ///
-    MathMatrixInset(MathMatrixInset*);
+    explicit
+    MathMatrixInset(MathMatrixInset *);
     ///
-    MathedInset *Clone();
+    MathedInset * Clone();
     ///
     virtual ~MathMatrixInset();
     ///
-    void Draw(int, int);
-    ///
-    void Write(FILE *file);
+    void draw(Painter &, int, int);
     ///
-    void Write(LString &file);
+    void Write(std::ostream &, bool fragile);
     ///
     void Metrics();
     ///
     void SetData(LyxArrayBase *);
     ///
-    void SetAlign(char, char const*);
+    void SetAlign(char, char const *);
     ///
-    char *GetAlign(char* vv) {
+    char * GetAlign(char * vv) {
        *vv = v_align;
        return h_align;
     }
@@ -502,21 +516,23 @@ class MathMatrixInset: public MathParInset {
     virtual bool isMatrix() { return true; }
 
     /// Use this to manage the extra information independently of paragraph
-    MathedRowSt *getRowSt() const { return row; }
-       ///
-    void setRowSt(MathedRowSt* r) { row = r; }
+    MathedRowSt * getRowSt() const { return row; }
+    ///
+    void setRowSt(MathedRowSt * r) { row = r; }
     
  protected:
     ///  Number of columns & rows
-    int nc, nr;
+    int nc;
+    ///
+    int nr;
     /// tab sizes
-    int *ws;   
+    int * ws;   
     /// 
     char v_align; // add approp. signedness
     ///
-    char* h_align;
+    char * h_align;
     /// Vertical structure
-    MathedRowSt *row;
+    MathedRowSt * row;
 
 };
 
@@ -524,13 +540,14 @@ class MathMatrixInset: public MathParInset {
 
 /*************************  Prototypes  **********************************/
 /// 
-LyxArrayBase *mathed_parse(unsigned flags, LyxArrayBase *data, MathParInset **mt);
-///
-void mathed_write(MathParInset*, FILE *, int*, char fragile, char const* label=NULL);
+LyxArrayBase * mathed_parse(unsigned flags, LyxArrayBase * data,
+                           MathParInset ** mt);
 ///
-void mathed_write(MathParInset*, LString&, int*, char fragile, char const* label=NULL);
+void mathed_write(MathParInset *, std::ostream &, int *, bool fragile,
+                 char const * label = 0);
+
 ///
-void mathed_parser_file(FILE*, int);
+void mathed_parser_file(std::istream &, int);
 ///
 int mathed_parser_lineno();
 ///
@@ -539,51 +556,92 @@ int MathedLookupBOP(short);
 /************************ Inline functions ********************************/
 
 ///
-#define MathIsInset(x)  (LM_TC_INSET<=(x) && (x)<=LM_TC_ACTIVE_INSET)
-///
-#define MathIsFont(x)  (LM_TC_CONST<=(x) && (x)<=LM_TC_BSYM)
+inline
+bool MathIsInset(short x)
+{
+       return LM_TC_INSET <= x && x <= LM_TC_ACTIVE_INSET;
+}
+
 ///
-#define MathIsAlphaFont(x)  (LM_TC_VAR<=(x) && (x)<=LM_TC_TEXTRM)
+inline
+bool MathIsFont(short x)
+{
+       return LM_TC_CONST <= x && x <= LM_TC_BSYM;
+}
+
 ///
-#define MathIsActive(x)  (LM_TC_INSET<(x) && (x)<=LM_TC_ACTIVE_INSET) 
+inline
+bool MathIsAlphaFont(short x)
+{
+       return LM_TC_VAR <= x && x <= LM_TC_TEXTRM;
+}
+
 ///
-#define MathIsUp(x)    ((x)==LM_TC_UP) 
+inline
+bool MathIsActive(short x)
+{
+       return LM_TC_INSET < x && x <= LM_TC_ACTIVE_INSET;
+}
+
 ///
-#define MathIsDown(x)  ((x)==LM_TC_DOWN)
+inline
+bool MathIsUp(short x)
+{
+       return x == LM_TC_UP;
+}
+
 ///
-#define MathIsScript(x)  ((x)==LM_TC_DOWN || (x)==LM_TC_UP)  
+inline
+bool MathIsDown(short x)
+{
+       return x == LM_TC_DOWN;
+}
+
 ///
-#define MathIsBOPS(x)    (MathedLookupBOP(x)>LMB_NONE)
+inline
+bool MathIsScript(short x)
+{
+       return x == LM_TC_DOWN || x == LM_TC_UP;
+}
 
+///
+inline
+bool MathIsBOPS(short x)
+{
+       return MathedLookupBOP(x) > LMB_NONE;
+}
 
 ///
-inline bool MathIsBinary(short x)
+inline
+bool MathIsBinary(short x)
 {
-    return (x==LM_TC_BOP || x==LM_TC_BOPS);
+    return x == LM_TC_BOP || x == LM_TC_BOPS;
 }
 
 ///
-inline bool MathIsSymbol(short x) {
-    return (LM_TC_SYMB<=x && x<=LM_TC_BSYM);
+inline
+bool MathIsSymbol(short x) {
+    return LM_TC_SYMB <= x && x <= LM_TC_BSYM;
 }
      
 
-inline 
-MathedInset::MathedInset(char const *nm, short ot, short st):
+inline
+MathedInset::MathedInset(char const * nm, short ot, short st):
   name(nm), objtype(ot), size(st) 
 {
    width = ascent = descent = 0;
 }
 
+
 inline
 bool MathParInset::Inside(int x, int y) 
 {
-  return (x>=xo && x<=xo+width && y<=yo+descent && y>=yo-ascent);
+  return (x >= xo && x <= xo + width && y <= yo + descent && y >= yo - ascent);
 }
 
 
 inline
-void MathParInset::GetXY(int& x, int& y) const
+void MathParInset::GetXY(int & x, int & y) const
 {
    x = xo; y = yo;
 }
@@ -592,7 +650,7 @@ void MathParInset::GetXY(int& x, int& y) const
 inline
 void MathParInset::UserSetSize(short sz)
 {
-   if (sz>=0) {
+   if (sz >= 0) {
        size = sz;      
        flag = flag & ~LMPF_FIXED_SIZE;
    }
@@ -604,13 +662,12 @@ void MathParInset::SetStyle(short sz)
 {
     if (Permit(LMPF_FIXED_SIZE)) {
        if (Permit(LMPF_SCRIPT)) 
-         sz = (sz<LM_ST_SCRIPT) ? LM_ST_SCRIPT: LM_ST_SCRIPTSCRIPT;
+         sz = (sz < LM_ST_SCRIPT) ? LM_ST_SCRIPT: LM_ST_SCRIPTSCRIPT;
        if (Permit(LMPF_SMALLER) && sz < LM_ST_SCRIPTSCRIPT) {
-           sz++;
+           ++sz;
        } 
        MathedInset::SetStyle(sz);
     }
 }
 
 #endif
-