why does the memory increase step by step?

Code:

---

for ( int i = 0; i < 100; ++i )
{
    char **p = evaluation();//assign values to **p
}

---

why does memory increase?
though defination of **p is local.( **p is local variable )

You keep declaring a new **p. Stop that! :p

That's not the problem. It's deallocated when the scope ends.

It's probably in the evaluation function.

Quote:

---

Originally Posted by Adak

You keep declaring a new **p. Stop that! :p

---

LOL!

Quote:

---

Originally Posted by cyberfish

That's not the problem. It's deallocated when the scope ends.

It's probably in the evaluation function.

---

Code:

---

evalue function:
usint** csMartrix(char **p,usint Punish,int *e)
{
    usint equalPunish = 10;           
    usint gapPunish  = 5;
    uint psNumber = caculateCombNum(e[1]);
    uint m = 0;
    usint *........ = (usint **) calloc (e[0],sizeof(usint*));
    usint preLineScore[psNumber];
    usint *st = (usint *) calloc (psNumber,sizeof(usint));
    for( uint pi = 0; pi < psNumber; ++pi) { preLineScore[pi] = 0; st[pi] = 0; } 
    ........ = st;
    for( uint i = 0; i < e[0]; ++i )
    {                 
          m = 0;
          usint *s = (usint *) calloc (psNumber,sizeof(usint));//if ( i%1000 == 0 ) printf("%d\n",i);printf("%d %d\n",psNumber,psNumber*sizeof(usint));
          int n = i+1;
          if ( s == NULL ) { printf("malloc memory error.\n");exit(0); }
          //printf("%d\n",n);
          ss[n] = s ; //*(ss+n) = s;
                             
          for ( uint j = 0; j < e[1]-1; ++j)
          {
              char a = p[i][j];//char a = *(*(p+i)+j);           
              for ( uint k = j+1; k < e[1]; ++k)
              {
                  char b = p[i][k];                                 
                  if ( a == '-' || a == '?' )
                  {
                      s[m] = preLineScore[m]+gapPunish;
                      preLineScore[m] = s[m];//printf("%d %d\n",i,m);//*(s+m) =  gapPunish;s[i][m] = 0;                     
                  }else if ( b == '-' || a == '?' )
                  {
                      s[m] = preLineScore[m]+gapPunish;
                      preLineScore[m] = s[m];
                  }else if ( a == b )
                  {
                      s[m] = preLineScore[m]+equalPunish;
                      preLineScore[m] = s[m];             
                  }else
                  {
                      s[m] = preLineScore[m]-Punish;
                      preLineScore[m] = s[m];     
                  }                 
                  ++m;
              }
             
          }
               
    }
    return ss;   
}

---

You are allocating memory and not freeing it?

he likes memory leaks

Quote:

---

Originally Posted by cyberfish

You are allocating memory and not freeing it?

---

I do free it

Code:

---

/* free memory of usint** */
void freeUS(usint** p,int n)
{
        if ( p[0][0] != '\0' )
        {
                for ( int i = 0; i < n; ++i )
                {
                        if ( p[i][0] != '\0' ) free(p[i]);
                }
                //delete [] p;
                free(p);
        } 
}

---

But not all of it

Code:

---

    usint * ss = (usint **) calloc (e[0],sizeof(usint*));
    usint preLineScore[psNumber];
    usint *st = (usint *) calloc (psNumber,sizeof(usint));
    for( uint pi = 0; pi < psNumber; ++pi) { preLineScore[pi] = 0; st[pi] = 0; } 
    * ss = st;
    for( uint i = 0; i < e[0]; ++i )
    {                 
          m = 0;
          usint *s = (usint *) calloc (psNumber,sizeof(usint));//if ( i%1000 == 0 ) printf("%d\n",i);printf("%d %d\n",psNumber,psNumber*sizeof(usint));
          int n = i+1;
          if ( s == NULL ) { printf("malloc memory error.\n");exit(0); }
          //printf("%d\n",n);
          ss[n] = s ; //*(ss+n) = s;

---

The red alloc is lost with the blue assignment.

I'm assuming the profanity filter has been at your code.

And that's not even called from your code.