Merge pull request #1646 from ldrumm/cleanup/unecessary-ternary

kbenzie · web-flow · commit a76254e4a9d0 · 2024-05-22T13:58:14.000+01:00
[NFC] Remove redundant ternary operation
diff --git a/source/adapters/opencl/usm.cpp b/source/adapters/opencl/usm.cpp
@@ -68,8 +68,7 @@ usmDescToCLMemProperties(const ur_base_desc_t *Desc,
       return UR_RESULT_ERROR_INVALID_VALUE;
     }
 
-    Next = Next->pNext ? static_cast<const ur_base_desc_t *>(Next->pNext)
-                       : nullptr;
+    Next = static_cast<const ur_base_desc_t *>(Next->pNext);
   } while (Next);
 
   if (AllocFlags) {

Original file line number	Diff line number	Diff line change
`@@ -68,8 +68,7 @@ usmDescToCLMemProperties(const ur_base_desc_t *Desc,`
`68`	`68`	`return UR_RESULT_ERROR_INVALID_VALUE;`
`69`	`69`	`}`
`70`	`70`
`71`		`- Next = Next->pNext ? static_cast<const ur_base_desc_t *>(Next->pNext)`
`72`		`- : nullptr;`
	`71`	`+ Next = static_cast<const ur_base_desc_t *>(Next->pNext);`
`73`	`72`	`} while (Next);`
`74`	`73`
`75`	`74`	`if (AllocFlags) {`