[Groff-commit] groff src/roff/troff/node.cpp ChangeLog

[Werner LEMBERG]

CVSROOT:        /cvsroot/groff
Module name:    groff
Changes by:     Werner LEMBERG <wl>     10/12/15 05:53:51

Modified files:
        src/roff/troff : node.cpp 
        .              : ChangeLog 

Log message:
        Use enum to increase readability.
        
        * src/roff/troff/node.cpp (break_char_type): New enum.
        (break_char_node::add_self, node::add_char): Use it.

CVSWeb URLs:
http://cvs.savannah.gnu.org/viewcvs/groff/src/roff/troff/node.cpp?cvsroot=groff&r1=1.33&r2=1.34
http://cvs.savannah.gnu.org/viewcvs/groff/ChangeLog?cvsroot=groff&r1=1.1254&r2=1.1255

Patches:
Index: src/roff/troff/node.cpp
===================================================================
RCS file: /cvsroot/groff/groff/src/roff/troff/node.cpp,v
retrieving revision 1.33
retrieving revision 1.34
diff -u -b -r1.33 -r1.34
--- src/roff/troff/node.cpp     28 Jun 2010 18:06:30 -0000      1.33
+++ src/roff/troff/node.cpp     15 Dec 2010 05:53:51 -0000      1.34
@@ -2765,6 +2765,12 @@
   return n->character_type();
 }
 
+enum break_char_type {
+  CAN_BREAK_BEFORE = 0x01,
+  CAN_BREAK_AFTER = 0x02,
+  IGNORE_HCODES = 0x04
+};
+
 class break_char_node : public node {
   node *ch;
   char break_code;
@@ -2846,16 +2852,16 @@
 node *break_char_node::add_self(node *n, hyphen_list **p)
 {
   assert((*p)->hyphenation_code == 0);
-  if (break_code & 1) {
-    if ((*p)->breakable || break_code & 4) {
+  if (break_code & CAN_BREAK_BEFORE) {
+    if ((*p)->breakable || break_code & IGNORE_HCODES) {
       n = new space_node(H0, col, n);
       n->freeze_space();
     }
   }
   next = n;
   n = this;
-  if (break_code & 2) {
-    if ((*p)->breakable || break_code & 4) {
+  if (break_code & CAN_BREAK_AFTER) {
+    if ((*p)->breakable || break_code & IGNORE_HCODES) {
       n = new space_node(H0, col, n);
       n->freeze_space();
     }
@@ -5082,11 +5088,11 @@
   }
   int break_code = 0;
   if (ci->can_break_before())
-    break_code = 1;
+    break_code = CAN_BREAK_BEFORE;
   if (ci->can_break_after())
-    break_code |= 2;
+    break_code |= CAN_BREAK_AFTER;
   if (ci->ignore_hcodes())
-    break_code |= 4;
+    break_code |= IGNORE_HCODES;
   if (break_code) {
     node *next1 = res->next;
     res->next = 0;

Index: ChangeLog
===================================================================
RCS file: /cvsroot/groff/groff/ChangeLog,v
retrieving revision 1.1254
retrieving revision 1.1255
diff -u -b -r1.1254 -r1.1255
--- ChangeLog   13 Dec 2010 17:43:07 -0000      1.1254
+++ ChangeLog   15 Dec 2010 05:53:51 -0000      1.1255
@@ -1,3 +1,10 @@
+2010-12-15  Werner LEMBERG  <address@hidden>
+
+       Use enum to increase readability.
+
+       * src/roff/troff/node.cpp (break_char_type): New enum.
+       (break_char_node::add_self, node::add_char): Use it.
+
 2010-12-13  Daiki Ueno  <address@hidden>
            Werner LEMBERG  <address@hidden>