<!DOCTYPE html>
<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
</head>
<body>
<p><br>
</p>
<div class="moz-cite-prefix">On 7/31/23 6:26 AM, Ruan Jinjie wrote:<br>
</div>
<blockquote type="cite"
cite="mid:20230731132610.2675314-1-ruanjinjie@huawei.com">
<pre class="moz-quote-pre" wrap="">Ther are many ternary operators, the true or false judgement
of which is unnecessary in C language semantics.
Signed-off-by: Ruan Jinjie <a class="moz-txt-link-rfc2396E" href="mailto:ruanjinjie@huawei.com"><ruanjinjie@huawei.com></a>
---
</pre>
</blockquote>
<span style="white-space: pre-wrap">
</span><br>
snip<br>
<blockquote type="cite"
cite="mid:20230731132610.2675314-1-ruanjinjie@huawei.com">
<pre class="moz-quote-pre" wrap="">
data->registry_data.avfs_support =
- hwmgr->feature_mask & PP_AVFS_MASK ? true : false;
+ hwmgr->feature_mask & PP_AVFS_MASK;
data->registry_data.led_dpm_enabled = 1;</pre>
</blockquote>
<p>These are not equivalent, consider 0xffff & 0x1000 != 1<br>
</p>
<p>Tom</p>
<br>
</body>
</html>