Re: Question about Malloc

[Jan-Benedict Glaw <jbglaw@lug-owl.de>]

[-- Attachment #1: Type: text/plain, Size: 1072 bytes --]

On Tue, 2004-10-19 09:41:49 -0300, Matías Aguirre <matiasaguirre@gmail.com>
wrote in message <a3c4234104101905413e127d3e@mail.gmail.com>:
> It's not safer to make:
> 
> Recpointer r;
> r=(Recpointer)malloc(Recpointer);

malloc is void, while r isn't. So you don't need the cast here. However,
notice that your malloc call is actually wrong: the argument isn't a
number (or sizeof(something)), but only a type.

General rule of thumb:

	The "output" of malloc should be assigned to a variable whose
	type is a pointer to whatever you give to sizeof(xxx) as the
	malloc argument.

or in short:

	One star more on the left than on the right.

Of course, if you play with types, this doesn't apply any longer...

MfG, JBG

-- 
Jan-Benedict Glaw       jbglaw@lug-owl.de    . +49-172-7608481             _ O _
"Eine Freie Meinung in  einem Freien Kopf    | Gegen Zensur | Gegen Krieg  _ _ O
 fuer einen Freien Staat voll Freier Bürger" | im Internet! |   im Irak!   O O O
ret = do_actions((curr | FREE_SPEECH) & ~(NEW_COPYRIGHT_LAW | DRM | TCPA));

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 189 bytes --]