From: DavMac AT iname DOT com (Davin McCall)
Newsgroups: comp.os.msdos.djgpp
Subject: Re: help: void pointers
Date: Sun, 08 Aug 1999 05:06:26 GMT
Organization: Monash Uni
Lines: 37
Distribution: world
Message-ID: <37ad0e94.18907885@newsserver.cc.monash.edu.au>
References: <8UQq3.6777$K%6 DOT 145927 AT news1 DOT rdc2 DOT on DOT home DOT com>
NNTP-Posting-Host: damcc5.halls.monash.edu.au
X-Trace: towncrier.cc.monash.edu.au 934088726 1115 130.194.198.138 (8 Aug 1999 05:05:26 GMT)
X-Complaints-To: abuse AT monash DOT edu DOT au
NNTP-Posting-Date: 8 Aug 1999 05:05:26 GMT
X-Newsreader: Forte Free Agent 1.1/32.230
To: djgpp AT delorie DOT com
DJ-Gateway: from newsgroup comp.os.msdos.djgpp
Reply-To: djgpp AT delorie DOT com

On Sat, 07 Aug 1999 07:04:36 GMT, "YUBS" <jdebever1 AT home DOT com> wrote:

>void pcx_init(void *image, char type)
>{
>    switch (type)
>        {
>            case 's': {
>                            if (!(((pcx_image_ptr)(image->buffer)) = (char
>* )malloc (image->width* image->height)))
>                         }
>        }
>}

Should be: 		     if(!(((pcx_image_ptr)image)->buffer) =
(char *)malloc(((pcx_image_ptr)image)->width *
((pcx_image_ptr)image)->height)) { printf( "error!\n" ); exit(1); }

or something similar. Better still:

void pcx_init(void *image, char type)
{
    switch (type)
        {
            case 's': {
		pcx_image_ptr tempptr = (pcx_image_ptr)image;
                if (!(tempptr->buffer = (char *)malloc (tempptr->width
* tempptr->height))) {
			print( "error!\n" ); exit(1);
		 }
            }
        }
}

Davin.
__________________________________________________________
       *** davmac - sharkin'!! davmac AT iname DOT com ***
my programming page: http://yoyo.cc.monash.edu.au/~davmac/