List:Commits« Previous MessageNext Message »
From:Rafal Somla Date:May 2 2011 10:24am
Subject:Re: bzr push into mysql-trunk branch (rafal.somla:3345 to 3346) Bug#11766631
Bug#11879051
View as plain text  
Hi again,

On 29/04/2011 21:56, Davi Arnaut wrote:
> On 4/29/11 4:45 PM, Rafal Somla wrote:
>> + if (m_buf)
>> + free(m_buf);
>
> free() handles NULL pointers just fine.

Correct, but above code has 2 slight advantages:

1. It might be more efficient than calling free() in case m_buf is NULL.

2. It will always work, even if on some exotic platform/configuration, 
free() does not handle NULL pointers.

Rafal
Thread
bzr push into mysql-trunk branch (rafal.somla:3345 to 3346) Bug#11766631Bug#11879051Rafal Somla29 Apr
  • Re: bzr push into mysql-trunk branch (rafal.somla:3345 to 3346) Bug#11766631Bug#11879051Davi Arnaut29 Apr
    • Re: bzr push into mysql-trunk branch (rafal.somla:3345 to 3346) Bug#11766631Bug#11879051Rafal Somla2 May
      • Re: bzr push into mysql-trunk branch (rafal.somla:3345 to 3346) Bug#11766631Bug#11879051Davi Arnaut2 May