0024129: Eliminate remaining compiler warnings in MSVC++ 2008 32 bit with warning level 4

List of resolved warnings:

c4063: Aspect.cdl; TObj_Application.cxx; PCDM.cdl; OpenGl_Workspace_5.cxx;
c4100: XDEDRAW.cxx; ViewerTest_ViewerCommands.cxx; TopOpeBRep_FacesFiller_1.cxx; TopClass_Classifier3d.gxx; TDataStd_ReferenceArray.cxx; QABugs_19.cxx; IntPatch_ImpImpIntersection_5.gxx; HLRTopoBRep_DSFiller.cxx; HLRBRep_Data.lxx; DPrsStd_AISPresentationCommands.cxx; BRepMesh_Delaun.cxx;
c4127: BOPTools_AlgoTools2D.cxx; MAT_Mat.gxx; GeomFill_SweepSectionGenerator.cxx; BRepMesh_Delaun.cxx;
c4189: IntCurveSurface_Inter.gxx; IGESToBRep_BRepEntity.cxx; BRepMesh_Delaun.cxx; BRepAlgo_Loop.cxx; IntStart_SearchOnBoundaries_1.gxx;
c4190: Plugin_Macro.hxx;
c4389: Visual3d_View.cxx; TopOpeBRep/TopOpeBRep_vpr.cxx; TDataStd_BooleanArray.cxx; IntPatch_ALine.cxx;
c4701: BRepAlgo.cxx;
c4702: MNaming_NamingRetrievalDriver_1.cxx; MNaming_NamingRetrievalDriver_2.cxx; BRepClass3d_SolidExplorer.cxx;
c4706: TestTopOpe_BOOP.cxx;

Additional fixes:
- old-style declarations of C functions in IGES and STEP parsers
- clean-up of debug code, fixes for building in Debug mode
- in BRepFill_TrimShellCorner.cxx, lines 878-9, wrong use of assignment instead of comparison fixed
- fix for Plugin_Macro reverted; warning 4190 disabled instead
- in IntPatch_ALine.cxx, line 520, wrong comparison of boolean with index fixed
- in InterfaceGraphic_Visual3d.hxx, field IsCustomMatrix made boolean
- in TopOpeBRepBuild_ShapeSet, obsolete (unused) methods removed

src/IGESFile/analiges.c

@@ -23,7 +23,7 @@
 
 void iges_newparam(int typarg,int longval, char *parval);
 void iges_newpart(int numsec);
-void iges_curpart(int numsec,int dnum);
+void iges_curpart(int dnum);
 void iges_addparam(int longval, char* parval);
 
 #define ArgVide 0
@@ -55,8 +55,7 @@ static int bases[] =
 /*      Utilitaire decodant un nombre en format fixe dans une ligne
 	Il part de "depuis" inclus (debut ligne = 0) et prend "tant" caracteres
 	Valeur lue en retour de fonction   */
-static int IGES_decode(ligne,depuis,tant)
-char* ligne; int depuis, tant;
+static int IGES_decode (char* ligne, int depuis, int tant)
 {
   int val = 0; int i;
   int depart = depuis+tant-1;
@@ -71,8 +70,7 @@ char* ligne; int depuis, tant;
 }
 
 /*   Recopie d'une chaine de caracteres de longueur fixe (close par \0)  */
-void IGES_copstr(ligne,depuis,tant,dans)
-char *ligne; int depuis, tant; char* dans;
+void IGES_copstr(char *ligne, int depuis, int tant, char* dans)
 {
   int i;
   for (i = 0; i < tant; i ++) { dans[i] = ligne[depuis+i]; }
@@ -80,8 +78,7 @@ char *ligne; int depuis, tant; char* dans;
 }
 
 /*                   Analyse section D                */
-void iges_Dsect (Dstat,numsec,ligne)
-int *Dstat,numsec; char* ligne;
+void iges_Dsect (int *Dstat, int numsec, char* ligne)
 {
   struct dirpart *curp;
   if (*Dstat == 0) {
@@ -125,15 +122,16 @@ int *Dstat,numsec; char* ligne;
 /*     Lecture section P : preanalyse
        Extraction du numero D et troncature a 64 caracteres  */
 
-void iges_Psect(Pstat,numsec,ligne)
-int *Pstat; int numsec; char ligne[80];
+void iges_Psect (int numsec, char ligne[80])
 {
   int dnum;
   dnum = atoi(&ligne[65]);
   ligne[64] = '\0';
-  iges_curpart(numsec,dnum);
+  iges_curpart(dnum);
 #ifdef VERIFPRINT
   printf("Entite P:%d ->D:%d,soit %s\n",numsec,dnum,ligne);
+#else
+  (void)numsec; // just to avoid warning
 #endif
 }
 
@@ -154,8 +152,7 @@ static int typarg;
 /*  +  definitions des types de parametres en tete  */
 
 
-void iges_param(Pstat,ligne,c_separ,c_fin,lonlin)
-int *Pstat; char c_separ,c_fin, *ligne; int lonlin;
+void iges_param (int *Pstat, char *ligne, char c_separ, char c_fin, int lonlin)
 {
   int i,i0,j; char param[80]; char unpar;
   if (*Pstat == 0) reste  = 0;

src/IGESFile/igesread.c

@@ -34,7 +34,7 @@ int  iges_lire (FILE* lefic, int *numsec, char ligne[100], int modefnes);
 void iges_newparam(int typarg,int longval, char *parval);
 void iges_param(int *Pstat,char *ligne,char c_separ,char c_fin,int lonlin);
 void iges_Dsect (int *Dstat,int numsec,char* ligne);
-void iges_Psect(int *Pstat,int numsec,char ligne[80]);
+void iges_Psect(int numsec,char ligne[80]);
 
 
 
@@ -56,8 +56,7 @@ void iges_Psect(int *Pstat,int numsec,char ligne[80]);
 static  char sects [] = " SGDPT ";
 
 
-int igesread(nomfic,lesect,modefnes)
-char* nomfic; int lesect[6]; int modefnes;
+int igesread (char* nomfic, int lesect[6], int modefnes)
 {
   /* MGE 16/06/98 */
 
@@ -113,7 +112,7 @@ char* nomfic; int lesect[6]; int modefnes;
     }
     if (i == 3) iges_Dsect(&Dstat,numsec,ligne);    /* Directory  (Dsect) */
     if (i == 4) {                                   /* Parametres (Psect) */
-      iges_Psect(&Pstat,numsec,ligne);
+      iges_Psect(numsec,ligne);
       for (;;) {
 	iges_param(&Pstat,ligne,c_separ,c_fin,64);
 	if (Pstat != 2) break;

src/IGESFile/igesread.h

@@ -56,7 +56,7 @@ extern "C" {
   void iges_newparam(int typarg,int longval, char *parval);
   void iges_param(int *Pstat,char *ligne,char c_separ,char c_fin,int lonlin);
   void iges_Dsect (int *Dstat,int numsec,char* ligne);
-  void iges_Psect(int *Pstat,int numsec,char ligne[80]);
+  void iges_Psect(int numsec,char ligne[80]);
 
   /* MGE 20/07/98 */
   void IGESFile_Check2 (int mode,char * code, int num, char * str);

src/IGESFile/structiges.c

@@ -92,8 +92,7 @@ static struct dirpage *curpage;
 /*    Utilitaire : Reservation de caracteres
       Remplace suite de mini-malloc par gestion de page   */
 
-char* iges_newchar(newtext,lentext)
-char* newtext; int lentext;
+char* iges_newchar (char* newtext, int lentext)
 {
   int i, lnt;
   if ((lnt = onecarpage->used) > Maxcar-lentext-1) {  /* allouer nouvelle page */
@@ -143,8 +142,7 @@ void iges_setglobal()
 
 /*   Definition et Selection d'un nouveau dirpart   */
 
-void iges_newpart(numsec)
-int numsec;
+void iges_newpart(int numsec)
 {
   if (curpage->used >= Maxparts) {
     struct dirpage* newpage;
@@ -163,8 +161,7 @@ int numsec;
 
 /*   Selection du dirpart dnum, correspond a numsec en Psect   */
 
-void iges_curpart(numsec,dnum)
-int numsec,dnum;
+void iges_curpart (int dnum)
 {
   if (curp == NULL) return;
   if (dnum == curp->numpart) return;
@@ -198,8 +195,7 @@ int numsec,dnum;
 /*   (manque la gestion d'un Hollerith sur plusieurs lignes)   */
 
 /*   longval : longueur de parval, incluant le zero final   */
-void iges_newparam(typarg,longval,parval)
-int typarg,longval; char *parval;
+void iges_newparam (int typarg, int longval, char *parval)
 {
   char *newval;
   if (curlist == NULL) return;      /*  non defini : abandon  */
@@ -224,8 +220,7 @@ int typarg,longval; char *parval;
 }
 
 /*     Complement du parametre courant (cf Hollerith sur +ieurs lignes)    */
-void iges_addparam(longval,parval)
-int longval; char* parval;
+void iges_addparam (int longval, char* parval)
 {
   char *newval, *oldval; int i,long0;
   if (longval <= 0) return;
@@ -243,8 +238,7 @@ int longval; char* parval;
 /*               Relecture : Initialiation              */
 /*  entites relues par suite de lirpart + {lirparam}
     lirparam initiaux : pour relire le demarrage (start section)   */
-void iges_stats(nbpart, nbparam)
-int* nbpart; int* nbparam;
+void iges_stats (int* nbpart, int* nbparam)
 {
   curpage  = firstpage; curnumpart = 0;
   curlist  = starts;
@@ -253,13 +247,10 @@ int* nbpart; int* nbparam;
   *nbparam = nbparams;
 }
 
-/*               Lecture d'une part : retour = n0 section, 0 si fin         */
-int iges_lirpart
-  (tabval, res1,res2,nom,num, nbparam)
-
-int* *tabval;        /* tableau recepteur des entiers (reserver 17 valeurs) */
-char* *res1; char* *res2; char* *nom; char* *num;  /* char : transmis a part */
-int *nbparam;
+/*      Lecture d'une part : retour = n0 section, 0 si fin         */
+/* \par tabval tableau recepteur des entiers (reserver 17 valeurs) */
+/* \par res1 res2 nom num char : transmis a part */
+int iges_lirpart (int* *tabval, char* *res1, char* *res2, char* *nom, char* *num, int *nbparam)
 {
   if (curpage == NULL) return 0;
   curp = &(curpage->parts[curnumpart]);
@@ -283,8 +274,7 @@ void iges_nextpart()
 }
 
 /*               Lecture parametre + passage au suivant                   */
-int iges_lirparam(typarg,parval)    /* renvoie 0 si fin de liste, 1 sinon */
-int *typarg; char* *parval;
+int iges_lirparam (int *typarg, char* *parval)    /* renvoie 0 si fin de liste, 1 sinon */
 {
   if (curparam == NULL) return 0;
   *typarg = curparam->typarg;
@@ -295,8 +285,7 @@ int *typarg; char* *parval;
 
 /*               Fin pour ce fichier : liberer la place                  */
 /*    mode = 0 : tout; 1 : parametres; 2 : caracteres  */
-void iges_finfile(mode)
-int mode;
+void iges_finfile (int mode)
 {
   struct dirpage* oldpage;
   if (mode == 0 || mode == 2) {  free (starts);  free (header);  }