Remove unneeded check in CFRelease and CFRetain.

AZero13 · DTeachs · commit 4374dd79ce64 · 2022-01-25T12:16:41.000-05:00
If they were NULL, the program would just exit so this check is not needed
diff --git a/CoreFoundation/Base.subproj/CFRuntime.c b/CoreFoundation/Base.subproj/CFRuntime.c
@@ -771,7 +771,7 @@ CFTypeRef _CFNonObjCRetain(CFTypeRef cf) {
 
 CFTypeRef CFRetain(CFTypeRef cf) {
     if (NULL == cf) { CRSetCrashLogMessage("*** CFRetain() called with NULL ***"); HALT; }
-    if (cf) __CFGenericAssertIsCF(cf);
+    __CFGenericAssertIsCF(cf);
     return _CFRetain(cf, false);
 }
 
@@ -789,7 +789,7 @@ void _CFNonObjCRelease(CFTypeRef cf) {
 
 void CFRelease(CFTypeRef cf) {
     if (NULL == cf) { CRSetCrashLogMessage("*** CFRelease() called with NULL ***"); HALT; }
-    if (cf) __CFGenericAssertIsCF(cf);
+    __CFGenericAssertIsCF(cf);
     _CFRelease(cf);
 }
 

Original file line number	Diff line number	Diff line change
`@@ -771,7 +771,7 @@ CFTypeRef _CFNonObjCRetain(CFTypeRef cf) {`
`771`	`771`
`772`	`772`	`CFTypeRef CFRetain(CFTypeRef cf) {`
`773`	`773`	`if (NULL == cf) { CRSetCrashLogMessage("* CFRetain() called with NULL *"); HALT; }`
`774`		`- if (cf) __CFGenericAssertIsCF(cf);`
	`774`	`+ __CFGenericAssertIsCF(cf);`
`775`	`775`	`return _CFRetain(cf, false);`
`776`	`776`	`}`
`777`	`777`
`@@ -789,7 +789,7 @@ void _CFNonObjCRelease(CFTypeRef cf) {`
`789`	`789`
`790`	`790`	`void CFRelease(CFTypeRef cf) {`
`791`	`791`	`if (NULL == cf) { CRSetCrashLogMessage("* CFRelease() called with NULL *"); HALT; }`
`792`		`- if (cf) __CFGenericAssertIsCF(cf);`
	`792`	`+ __CFGenericAssertIsCF(cf);`
`793`	`793`	`_CFRelease(cf);`
`794`	`794`	`}`
`795`	`795`