njh / twolame

MPEG Audio Layer 2 (MP2) encoder
http://www.twolame.org/
GNU Lesser General Public License v2.1
59 stars 34 forks source link

Cleaning operation #64

Closed eblanca closed 6 years ago

eblanca commented 6 years ago

In libtwolame/mem.h I see the macro (useful, indeed) TWOLAME_FREE which is defined as follows: if(ptr!=NULL) { free(ptr); ptr=NULL; } Now, what's the point in setting to NULL a parameter passed as value? The caller will never see the change. I don't get this.

eblanca commented 6 years ago

Oh, got it! It's a macro, not a function. Sorry.