Changeset 667 for CMT


Ignore:
Timestamp:
Jan 31, 2014, 4:16:22 PM (10 years ago)
Author:
rybkin
Message:

See C.L. 524

Location:
CMT/HEAD
Files:
13 edited

Legend:

Unmodified
Added
Removed

  • CMT/HEAD/ChangeLog

    r665 r667  
     12014-01-31    <rybkin@lal.in2p3.fr> 524
     2
     3        * source/cmt_use.cxx: Add default clause to switch statements to avoid
     4        compilation warning about enumeration value(s) not being handled on
     5        Mac OS X 10.8.5
     6        * source/cmt_deps_builder.cxx: idem
     7        * source/cmt_generators.cxx: idem
     8        * source/cmt_script.cxx: idem
     9        * source/cmt_project.cxx: idem
     10        * source/cmt_pattern.cxx: idem
     11        * source/cmt_lock.cxx: idem
     12        * source/cmt_vcs.cxx: idem
     13        * source/cmt_generator.cxx: idem
     14        * source/cmt_symbol.cxx: idem
     15        * source/cmt_parser.cxx: idem,
     16        In class Cmt function do_broadcast, if depth option value is less than -1,
     17        use depth value of -1 and issue warning
     18        * source/cmt_constituent.cxx: Use cmt_string and string literal as first
     19        and second arguments respectively of != operator to avoid compilation
     20        warning about comparison being unspecified on Mac OS X 10.8.5
     21       
    1222014-01-25    <rybkin@lal.in2p3.fr> 523
    223

  • CMT/HEAD/source/cmt_constituent.cxx

    r663 r667  
    408408  parameters.clear ();
    409409
    410   if (has_target_tag && (Document != type || "cmt_actions" != group->name ()))
     410  if (has_target_tag && (Document != type || group->name () != "cmt_actions"))
    411411    {
    412412      const Tag* tag = Tag::find ("target_" + name);
    … …  
    454454//----------------------------------------------------------
    455455Constituent* Constituent::add_for_action (const Symbol& symbol)
    456 //Constituent* Constituent::add_for_action (const cmt_string& name)
    457456{
    458457  Constituent* constituent (0);
    … …  
    464463
    465464  cmt_string value = symbol.build_macro_value ();
    466   if ( "" == value && !target_tag_used) return constituent;
     465  if ( 0 == value.size () && !target_tag_used) return constituent;
    467466
    468467  constituent = add (Document, symbol.name, "cmt_action_runner");

  • CMT/HEAD/source/cmt_deps_builder.cxx

    r663 r667  
    713713          state = at_start;
    714714          break;
     715        default:
     716          break;
    715717        }
    716718      line_number++;

  • CMT/HEAD/source/cmt_generator.cxx

    r663 r667  
    11081108                  stamp += "mgr";
    11091109                  break;
     1110                default:
     1111                  break;
    11101112                }
    11111113              stamp += CmtSystem::file_separator ();
    … …  
    11531155                  prefix = "lib_";
    11541156                  break;
     1157                default:
     1158                  break;
    11551159                }
    11561160              cmt_string macro_name (prefix + constituent.name + "_cppflags");
    … …  
    11751179            case Library:
    11761180              prefix = "lib_";
     1181              break;
     1182            default:
    11771183              break;
    11781184            }

  • CMT/HEAD/source/cmt_generators.cxx

    r663 r667  
    493493      is_application = false;
    494494      m_TITLE = "Library";
     495      break;
     496    default:
    495497      break;
    496498    }
    … …  
    25512553              temp += use->prefix;
    25522554              temp += "ROOT)";
    2553               temp += CmtSystem::file_separator ();
    25542555              switch (use->style)
    25552556                {
    25562557                case cmt_style:
     2558                  temp += CmtSystem::file_separator ();
    25572559                  temp += "cmt";
    25582560                  break;
    25592561                case mgr_style:
     2562                  temp += CmtSystem::file_separator ();
    25602563                  temp += "mgr";
     2564                  break;
     2565                default:
    25612566                  break;
    25622567                }
    … …  
    32963301              stamp += "mgr";
    32973302              break;
     3303            default:
     3304              break;
    32983305            }
    32993306          stamp += CmtSystem::file_separator ();
    … …  
    33533360                  stamp += CmtSystem::file_separator ();
    33543361                  stamp += "mgr";
     3362                  break;
     3363                default:
    33553364                  break;
    33563365                }

  • CMT/HEAD/source/cmt_lock.cxx

    r663 r667  
    9797      //      cerr << "Package not locked" << endl;
    9898      return (s);
     99    default:
     100      break;
    99101    }
    100102

  • CMT/HEAD/source/cmt_parser.cxx

    r663 r667  
    19011901                  msg = "Cannot read `depth' value `";
    19021902                  msg += depth_str;
    1903                   msg += cmt_string ("'");
     1903                  msg += "'";
    19041904                  if (errno)
    1905                     msg += cmt_string (strerror (errno));
     1905                    msg += strerror (errno);
    19061906                  CmtError::set (CmtError::system_error, msg);
    19071907                  return;
    … …  
    19101910                  msg = "Invalid `depth' value `";
    19111911                  msg += depth_str;
    1912                   msg += cmt_string ("'");
     1912                  msg += "'";
    19131913                  CmtError::set (CmtError::syntax_error, msg);
    19141914                  return;
    … …  
    19171917                  if (depth_value < -1)
    19181918                    {
    1919                       char num[32]; sprintf (num, "%lu", -depth_value);
     1919                      /*
     1920                      char num[32]; sprintf (num, "%d", -depth_value);
    19201921                      msg = "Invalid `depth' value `-";
    19211922                      msg += num;
    … …  
    19231924                      CmtError::set (CmtError::syntax_error, msg);
    19241925                      return;
     1926                      */
     1927                      depth = -1;
     1928                      char num[4]; sprintf (num, "%d", depth);
     1929                      msg = "Using minimum `depth' value `";
     1930                      msg += num;
     1931                      msg += "'";
     1932                      CmtMessage::warning (msg);
    19251933                    }
    1926                   depth = depth_value;
     1934                  else
     1935                    {
     1936                      depth = depth_value;
     1937                    }
    19271938                  //cerr << "depth: " << depth << endl;
    19281939                  break;
    … …  
    40104021      //CmtSystem::cd ("mgr");
    40114022      break;
     4023    default:
     4024      break;
    40124025    }
    40134026
    … …  
    48224835    case Xml :
    48234836      cout << "</setup>" << endl;
     4837      break;
     4838    default:
    48244839      break;
    48254840    }
    … …  
    55535568        case with_version_directory:
    55545569          cout << "with_version_directory";
     5570          break;
     5571        default:
    55555572          break;
    55565573        }
    … …  
    81368153        }
    81378154      break;
     8155    default:
     8156      break;
    81388157    }
    81398158
    … …  
    81728191              cout << "set " << prefix << "CONFIG=" << endl;
    81738192              break;
     8193            default:
     8194              break;
    81748195            }
    81758196        }
    … …  
    81878208    case Bat :
    81888209      cout << "set CMTEXTRATAGS=" << endl;
     8210      break;
     8211    default:
    81898212      break;
    81908213    }
    … …  
    83558378        }
    83568379       
     8380      break;
     8381    default:
    83578382      break;
    83588383    }
    … …  
    1000610031            prefix = "lib_";
    1000710032            break;
     10033          default:
     10034            break;
    1000810035          }
    1000910036       

  • CMT/HEAD/source/cmt_pattern.cxx

    r663 r667  
    10201020
    10211021          break;
     1022        default:
     1023          break;
    10221024        }
    10231025

  • CMT/HEAD/source/cmt_project.cxx

    r663 r667  
    12941294      cout << "<projects>";
    12951295      break;
     1296    default:
     1297      break;
    12961298    }
    12971299
    … …  
    13081310    case Xml :
    13091311      cout << "</projects>" << endl;
     1312      break;
     1313    default:
    13101314      break;
    13111315    }
    … …  
    28472851      cout << "<clients>";
    28482852      break;
     2853    default:
     2854      break;
    28492855    }
    28502856  for (i = 0; i < m_parents.size (); i++)
    … …  
    28722878      cout << "</clients>";
    28732879      break;
     2880    default:
     2881      break;
    28742882    }
    28752883
    … …  
    28782886    case Xml :
    28792887      cout << "<uses>";
     2888      break;
     2889    default:
    28802890      break;
    28812891    }

  • CMT/HEAD/source/cmt_script.cxx

    r400 r667  
    211211      cout << ". ";
    212212      break;
     213    default:
     214      break;
    213215    }
    214216

  • CMT/HEAD/source/cmt_symbol.cxx

    r665 r667  
    808808      type = SymbolCleanupScript;
    809809      break;
     810    default:
     811      break;
    810812    }
    811813
    … …  
    850852                    s1 = "alias";
    851853                    break;
     854                  default:
     855                    break;
    852856                  }
    853857
    … …  
    874878                  case SymbolAlias:
    875879                    s2 = "alias";
     880                    break;
     881                  default:
    876882                    break;
    877883                  }
    … …  
    936942    case SymbolAction:
    937943      symbol.builder = &Action;
     944      break;
     945    default:
    938946      break;
    939947    }
    … …  
    18081816        CmtMessage::info ("Replace " + name + " with " + _name);
    18091817      break;
     1818    default:
     1819      break;
    18101820    }
    18111821
    … …  
    18281838          result = 1;
    18291839          break;
     1840        default:
     1841          break;
    18301842        }
    18311843      break;
    … …  
    18401852            cout << "alias " << name << " >/dev/null 2>&1 && unalias " << name;
    18411853            result = 1;
     1854            break;
     1855          default:
    18421856            break;
    18431857        }
    … …  
    18651879          cout << "set " << _name << "=" << CmtSystem::quote (temp, " \t");
    18661880          result = 1;
     1881          break;
     1882        default:
    18671883          break;
    18681884        }
    … …  
    18951911          result = 1;
    18961912          break;
    1897         }
     1913        default:
     1914          break;
     1915        }
     1916      break;
     1917    default:
    18981918      break;
    18991919    }
    … …  
    19791999                   << "<value>" << temp << "</value></variable>";
    19802000              result = 1;
     2001              break;
     2002            default:
    19812003              break;
    19822004          }
    … …  
    20072029                   << "<value>" << temp << "</value></alias>";
    20082030              result = 1;
     2031              break;
     2032            default:
    20092033              break;
    20102034          }
    … …  
    20482072                  out << "<script>" << name << "</script>";
    20492073                  result = 1;
     2074                  break;
     2075                default:
    20502076                  break;
    20512077              }
    … …  
    24522478            temp = new_value;
    24532479           
     2480            break;
     2481           
     2482          default:
    24542483            break;
    24552484        }
    … …  
    29082937             
    29092938            break;
     2939          default:
     2940             
     2941            break;
    29102942        }
    29112943
    … …  
    30603092                temp.replace_all (";;", ";");
    30613093              }
     3094             
     3095            break;
     3096          default :
    30623097             
    30633098            break;
    … …  
    33443379              }
    33453380            break;
     3381          default :
     3382            break;
    33463383        }
    33473384    }
    … …  
    35633600        if (value.text == "") return;
    35643601        break;
     3602      default:
     3603        break;
    35653604    }
    35663605
    … …  
    36393678        cout << " " << define_text << " action " << symbol.name << " as ";
    36403679        first_definition = false;
     3680        break;
     3681      default :
    36413682        break;
    36423683    }
    … …  
    36963737        //case CommandAction :
    36973738        if (value.text.size () == 0) return result;
     3739        break;
     3740      default :
    36983741        break;
    36993742    }
    … …  
    38953938
    38963939        break;
     3940      default :
     3941        break;
    38973942    }
    38983943
    … …  
    40914136        break;
    40924137        */
     4138      default :
     4139        break;
    40934140    }
    40944141

  • CMT/HEAD/source/cmt_use.cxx

    r663 r667  
    550550          Cmt::print_xml_prolog ("uses", out);
    551551          out << "<uses>";
     552          break;
     553        default:
    552554          break;
    553555        }
    … …  
    713715          out << "</uses>" << endl;
    714716          break;
     717        default:
     718          break;
    715719        }
    716720      /*
    … …  
    824828            while (path_alias[path_alias.size() - 1] == '\\') path_alias.erase (path_alias.size() - 1);
    825829            state = finished;
     830            break;
     831          default:
    826832            break;
    827833          }

  • CMT/HEAD/source/cmt_vcs.cxx

    r663 r667  
    649649            path = s;
    650650          state = no_need;
     651          break;
     652        default:
    651653          break;
    652654        }
    … …  
    33763378          state = no_need;
    33773379          break;
     3380        default:
     3381          break;
    33783382        }
    33793383    }
    … …  
    34743478          state = no_need;
    34753479          break;
     3480        default:
     3481          break;
    34763482        }
    34773483    }
    … …  
    35393545          tag = s;
    35403546          state = no_need;
     3547          break;
     3548        default:
    35413549          break;
    35423550        }
    … …  
    36353643          tag = s;
    36363644          state = no_need;
     3645          break;
     3646        default:
    36373647          break;
    36383648        }
Note: See TracChangeset for help on using the changeset viewer.