On Tue, 21 Dec 2004 00:11:20 -0800 Kaffe CVS <[EMAIL PROTECTED]> wrote:
Hi, > @@ -66,7 +66,7 @@ > > #if defined(HANDLE_MANIFEST_CLASSPATH) > static int isEntryInClasspath(const char*); > -static uint8* getManifestMainAttribute(jarFile*, char*); > +static uint8* getManifestMainAttribute(jarFile*, const char*); > static void handleManifestClassPath (classpathEntry *); > #endif > > @@ -134,9 +134,8 @@ > case CP_DIR: > case CP_ZIPFILE: > class = newClass(); > - if (class == 0) { > + if (class == NULL) { > postOutOfMemory(einfo); > - KFREE(hand.base); > return (NULL); > } > > @@ -144,14 +143,13 @@ > class->centry = centry; > class = readClass(class, &hand, NULL, einfo); > > - if (hand.base != 0) { > + if (hand.base != NULL) { > #if defined(KAFFE_STATS) > if (hand.type == CP_ZIPFILE) { > addToCounter(&jarmem, "vmmem-jar files", 1, > -(jlong)GCSIZEOF(hand.base)); > } > #endif > - KFREE(hand.base); > } > return (class); I think the removal of these two KFREE calls will leak memory, won't it? Regards, Helmer _______________________________________________ kaffe mailing list [EMAIL PROTECTED] http://kaffe.org/cgi-bin/mailman/listinfo/kaffe