🚀 go-pugleaf

Alfred wrote:

>         char *buf = (char *) calloc( Size, sizeof(char) );
>         File->Read(buf,Size);
>         Form1->Memo->Lines->Add(buf);

What about the string terminator? Also:Why use calloc()?

You have two choices. Either make the buffer one char longer than
needed and ensure that the final char is '\0' or just use the optional
length argument for the AnsiString ctor:

char * buf=new char[ Size ];
File->Read( buf, Size );
Form1->Memo->Lines->Add( AnsiString( buf, Size ) );
delete[] buf;

////////////

BTW:The above code is not exception safe in that any exceptions thrown
(quite probable for file I/O) will leak the allocated memory by
bypassing the 'delete' (and your 'free()'). This would be better:

  std::vector<char> buf( Size );
  File->Read( &( buf[ 0 ] ), Size );
  Form1->Memo->Lines->Add( AnsiString( buf, Size ) );

..now you don't have to worry about deallocating the buffer because
it's automatic when 'buf' is popped from the stack.

-- 
Andrue Cope [TeamB]
[Bicester, Uk]
http://info.borland.com/newsgroups/guide.html