I am confused and not clear whether the block of code is correct or not

[rasmidas]

Hi,
Please check whether the following block of code is correct or not.

      items[index] = 0; // Add an end of list marker
 
      // Add the data to the blob
 
      if (blob_ptr != 0)
      {
         long additional_space = 0;
 
         char** item_ptr = items;
 
         // Calculate the space taken by the new items
 
         while (*item_ptr != 0)
         {
            additional_space += strlen(*item_ptr);
 
            item_ptr++;
         }
 
         // Increase the size of the blob for the new items
 
         blob_ptr = realloc(blob_ptr, blob_size + additional_space + 1);
 
         item_ptr = items;
 
         // Loop through the items and add them to the blob
 
         while (*item_ptr != 0)
         {
            strcat((char*) blob_ptr, *item_ptr);
 
            delete *item_ptr;
 
            item_ptr++;
         }
 
         // Set the new size of the blob
 
         blob_size = blob_size + additional_space;
      }
 

Please help me. Thanks in advance.

[svlsr2000]

Hi,
Please check whether the following block of code is correct or not.


items[index] = 0; // Add an end of list marker

// Add the data to the blob

if (blob_ptr != 0)
{
long additional_spac e = 0;

char** item_ptr = items;

// Calculate the space taken by the new items

while (*item_ptr != 0)
{
additional_spac e += strlen(*item_pt r);

item_ptr++;
}

// Increase the size of the blob for the new items

blob_ptr = realloc(blob_pt r, blob_size + additional_spac e + 1);

item_ptr = items;

// Loop through the items and add them to the blob

while (*item_ptr != 0)
{
strcat((char*) blob_ptr, *item_ptr);

delete *item_ptr;

item_ptr++;
}

// Set the new size of the blob

blob_size = blob_size + additional_spac e;
}


Please help me. Thanks in advance.

You are using pointer to pointer,
char** item_ptr = items;
This is a good idea when u want to use item by item. Use pointer to array.

[weaknessforcats]

To the OP: Please use code tags. Read the Reply Guildelines Box to the right of the message window.