Changeset 13790


Ignore:
Timestamp:
10/22/12 13:36:40 (12 years ago)
Author:
Mathieu Morlighem
Message:

CHG: some issues reported by cppcheck and now use C++ casting operator

Location:
issm/trunk-jpl/src/c/Container
Files:
3 edited

Legend:

Unmodified
Added
Removed

  • issm/trunk-jpl/src/c/Container/Elements.cpp

    r13780 r13790  
    4646        for ( object=objects.begin() ; object < objects.end(); object++ ){
    4747
    48                 element=(Element*)(*object);
     48                element=dynamic_cast<Element*>((*object));
    4949                element->Configure(elements,loads,nodes,materials,parameters);
    5050
    … …  
    5858        //Process results to be output in the correct units
    5959        for(int i=0;i<this->Size();i++){
    60                 Element* element=(Element*)this->GetObjectByOffset(i);
     60                Element* element=dynamic_cast<Element*>(this->GetObjectByOffset(i));
    6161                element->ProcessResultsUnits();
    6262        }
    … …  
    6767
    6868        for (int i=0;i<this->Size();i++){
    69                 Element* element=(Element*)this->GetObjectByOffset(i);
     69                Element* element=dynamic_cast<Element*>(this->GetObjectByOffset(i));
    7070                element->DeleteResults();
    7171        }
    … …  
    116116        for(i=0;i<this->Size();i++){
    117117
    118                 Element* element=(Element*)this->GetObjectByOffset(i);
     118                Element* element=dynamic_cast<Element*>(this->GetObjectByOffset(i));
    119119                element->PatchSize(&element_numrows,&element_numvertices,&element_numnodes);
    120120
    … …  
    141141        count=0;
    142142        for(i=0;i<this->Size();i++){
    143                 Element* element=(Element*)this->GetObjectByOffset(i);
     143                Element* element=dynamic_cast<Element*>(this->GetObjectByOffset(i));
    144144                element->PatchFill(&count,patch);
    145145        }
    … …  
    156156        for ( object=objects.begin() ; object < objects.end(); object++ ){
    157157
    158                 element=(Element*)(*object);
     158                element=dynamic_cast<Element*>((*object));
    159159                element->SetCurrentConfiguration(elements,loads,nodes,materials,parameters);
    160160
    … …  
    208208                if(my_rank==minrank){
    209209                        if(this->Size()==0) _error_("Cannot write results because there is no element??");
    210                         Element* element=(Element*)this->GetObjectByOffset(0);
     210                        Element* element=dynamic_cast<Element*>(this->GetObjectByOffset(0));
    211211                        element->ListResultsInfo(&resultsenums,&resultssizes,&resultstimes,&resultssteps,&numberofresults);
    212212                }
    … …  
    240240
    241241                        for(int j=0;j<this->Size();j++){
    242                                 Element* element=(Element*)this->GetObjectByOffset(j);
     242                                Element* element=dynamic_cast<Element*>(this->GetObjectByOffset(j));
    243243                                element->GetVectorFromResults(vector,i,resultsenums[i],resultssizes[i]);
    244244                        }
    … …  
    314314
    315315        for(int i=0;i<this->Size();i++){
    316                 Element* element=(Element*)this->GetObjectByOffset(i);
     316                Element* element=dynamic_cast<Element*>(this->GetObjectByOffset(i));
    317317                element->InputDuplicate(input_enum,output_enum);
    318318        }

  • issm/trunk-jpl/src/c/Container/Nodes.cpp

    r13622 r13790  
    4646        for ( object=objects.begin() ; object < objects.end(); object++ ){
    4747
    48                 node=(Node*)(*object);
     48                node=dynamic_cast<Node*>(*object);
    4949                node->Configure(nodes,vertices);
    5050
    … …  
    7676        /*Go through objects, and distribute dofs locally, from 0 to numberofdofsperobject*/
    7777        for (i=0;i<this->Size();i++){
    78                 Node* node=(Node*)this->GetObjectByOffset(i);
     78                Node* node=dynamic_cast<Node*>(this->GetObjectByOffset(i));
    7979
    8080                /*Check that this node corresponds to our analysis currently being carried out: */
    … …  
    105105        for (i=0;i<this->Size();i++){
    106106                /*Check that this node corresponds to our analysis currently being carried out: */
    107                 Node* node=(Node*)this->GetObjectByOffset(i);
     107                Node* node=dynamic_cast<Node*>(this->GetObjectByOffset(i));
    108108                if (node->InAnalysis(analysis_type)){
    109109                        node->OffsetDofs(dofcount,setenum);
    … …  
    122122
    123123        for (i=0;i<this->Size();i++){
    124                 Node* node=(Node*)this->GetObjectByOffset(i);
     124                Node* node=dynamic_cast<Node*>(this->GetObjectByOffset(i));
    125125                if (node->InAnalysis(analysis_type)){
    126126                        node->ShowTrueDofs(truedofs,maxdofspernode,setenum);//give maxdofspernode, column size, so that nodes can index into truedofs
    … …  
    136136        /* Now every cpu knows the true dofs of everyone else that is not a clone*/
    137137        for (i=0;i<this->Size();i++){
    138                 Node* node=(Node*)this->GetObjectByOffset(i);
     138                Node* node=dynamic_cast<Node*>(this->GetObjectByOffset(i));
    139139                if (node->InAnalysis(analysis_type)){
    140140                        node->UpdateCloneDofs(alltruedofs,maxdofspernode,setenum);
    … …  
    185185        for(i=0;i<this->Size();i++){
    186186
    187                 Node* node=(Node*)this->GetObjectByOffset(i);
     187                Node* node=dynamic_cast<Node*>(this->GetObjectByOffset(i));
    188188
    189189                /*Check that this node corresponds to our analysis currently being carried out: */
    … …  
    212212        for(i=0;i<this->Size();i++){
    213213
    214                 Node* node=(Node*)this->GetObjectByOffset(i);
     214                Node* node=dynamic_cast<Node*>(this->GetObjectByOffset(i));
    215215
    216216                /*Check that this node corresponds to our analysis currently being carried out: */
    … …  
    242242        for(i=0;i<this->Size();i++){
    243243
    244                 Node* node=(Node*)this->GetObjectByOffset(i);
     244                Node* node=dynamic_cast<Node*>(this->GetObjectByOffset(i));
    245245
    246246                /*Check that this node corresponds to our analysis currently being carried out: */
    … …  
    277277        /*Now go through all nodes, and get how many dofs they own, unless they are clone nodes: */
    278278        for(i=0;i<this->Size();i++){
    279                 Node* node=(Node*)this->GetObjectByOffset(i);
     279                Node* node=dynamic_cast<Node*>(this->GetObjectByOffset(i));
    280280
    281281                /*Ok, this object is a node, ask it to plug values into partition: */
    … …  
    304304        for(i=0;i<this->Size();i++){
    305305
    306                 Node* node=(Node*)this->GetObjectByOffset(i);
     306                Node* node=dynamic_cast<Node*>(this->GetObjectByOffset(i));
    307307
    308308                /*Check that this node corresponds to our analysis currently being carried out: */
    … …  
    344344        for(int i=0;i<this->Size();i++){
    345345
    346                 Node* node=(Node*)this->GetObjectByOffset(i);
     346                Node* node=dynamic_cast<Node*>(this->GetObjectByOffset(i));
    347347
    348348                /*Check that this node corresponds to our analysis currently being carried out: */
    … …  
    363363        for ( object=objects.begin() ; object < objects.end(); object++ ){
    364364
    365                 node=(Node*)(*object);
     365                node=dynamic_cast<Node*>(*object);
    366366                node->SetCurrentConfiguration(nodes,vertices);
    367367

  • issm/trunk-jpl/src/c/Container/Options.h

    r13780 r13790  
    9999        else{
    100100                /*Make a copy*/
    101                 char* outstring=NULL;
    102                 int   stringsize;
    103 
    104                 stringsize=strlen(default_value)+1;
    105                 outstring=xNew<char>(stringsize);
     101                int   stringsize=strlen(default_value)+1;
     102                char* outstring=xNew<char>(stringsize);
    106103                xMemCpy<char>(outstring,default_value,stringsize);
    107104                *pvalue=outstring;
Note: See TracChangeset for help on using the changeset viewer.