jcs
/detritus
/amendments
/66
util: Reduce data section where we can
jcs made amendment 66 about 1 year ago
--- util.c Mon Nov 18 21:28:35 2024
+++ util.c Fri Dec 27 22:17:47 2024
@@ -846,7 +846,7 @@ about(char *program_name)
{
VersRecHndl vers;
char vers_s[255];
- char short_vers[255] = { 0 };
+ char short_vers[255];
short vlen, n;
if ((vers = (VersRecHndl)GetResource('vers', 1))) {
@@ -1215,9 +1215,10 @@ stat(char *path, struct stat *sb)
short
FStat(Str255 path, struct stat *sb)
{
- CInfoPBRec catblock = { 0 };
+ CInfoPBRec catblock;
short ret;
+ memset(&catblock, 0, sizeof(catblock));
catblock.hFileInfo.ioNamePtr = path;
ret = PBGetCatInfo(&catblock, 0);
if (ret != noErr)
@@ -1348,7 +1349,9 @@ copy_file_contents(short source_ref, short dest_ref)
return error;
buf = xmalloc(1024);
-
+ if (buf == NULL)
+ return -1;
+
while (source_size > 0) {
count = 1024;
if (count > source_size)
@@ -1472,43 +1475,107 @@ GetSystemSubfolder(OSType folder, bool create, short *
char *
gestalt_machine_type(void)
{
- short error;
long resp;
- error = Gestalt(gestaltMachineType, &resp);
- if (error)
+ if (Gestalt(gestaltMachineType, &resp) != 0)
return NULL;
+
switch (resp) {
case gestaltClassic:
- return "Macintosh 128K";
+ return "128K";
case gestaltMacXL:
- return "Macintosh XL";
+ return "XL";
case gestaltMac512KE:
- return "Macintosh 512Ke";
+ return "512Ke";
case gestaltMacPlus:
- return "Macintosh Plus";
+ return "Plus";
case gestaltMacSE:
- return "Macintosh SE";
+ return "SE";
case gestaltMacII:
- return "Macintosh II";
+ return "II";
case gestaltMacIIx:
- return "Macintosh IIx";
+ return "IIx";
case gestaltMacIIcx:
- return "Macintosh IIcx";
+ return "IIcx";
case gestaltMacSE030:
- return "Macintosh SE/30";
+ return "SE/30";
case gestaltPortable:
- return "Macintosh Portable";
+ return "Portable";
case gestaltMacIIci:
- return "Macintosh IIci";
+ return "IIci";
case gestaltMacIIfx:
- return "Macintosh IIfx";
+ return "IIfx";
case gestaltMacClassic:
- return "Macintosh Classic";
+ return "Classic";
case gestaltMacIIsi:
- return "Macintosh IIsi";
+ return "IIsi";
case gestaltMacLC:
- return "Macintosh LC";
+ return "LC";
+
+ case 20:
+ return "Quadra 900";
+ case 21:
+ return "PowerBook 170";
+ case 22:
+ return "Quadra 700";
+ case 23:
+ return "Classic II";
+ case 24:
+ return "PowerBook 100";
+ case 25:
+ return "PowerBook 140";
+ case 26:
+ return "Quadra 950";
+ case 27:
+ return "LC III";
+ case 29:
+ return "PowerBook Duo 210";
+ case 30:
+ return "Centris 650";
+ case 32:
+ return "PowerBook Duo 230";
+ case 33:
+ return "PowerBook 180";
+ case 34:
+ return "PowerBook 160";
+ case 35:
+ return "Quadra 800";
+ case 36:
+ return "Quadra 650";
+ case 37:
+ return "LC II";
+ case 38:
+ return "PowerBook Duo 250";
+ case 44:
+ return "IIvi";
+ case 45:
+ return "Performa 600";
+ case 48:
+ return "IIvx";
+ case 49:
+ return "Color Classic";
+ case 50:
+ return "PowerBook 165c";
+ case 52:
+ return "Centris 610";
+ case 53:
+ return "Quadra 610";
+ case 54:
+ return "PowerBook 145";
+ case 56:
+ return "LC 520";
+ case 71:
+ return "PowerBook 180c";
+ case 84:
+ return "PowerBook 165";
+ case 102:
+ return "PowerBook Duo 280";
+ case 103:
+ return "PowerBook Duo 280c";
+ case 115:
+ return "PowerBook 150";
+ case 122:
+ return "PowerBook 190";
}
return NULL;