<table><tr><td style="">mpyne created this revision.<br />Restricted Application added a project: Plasma.<br />Restricted Application added a subscriber: plasma-devel.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D6658" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Something I noticed while doing the musl libc fixes:</p>

<ol class="remarkup-list">
<li class="remarkup-list-item">A GCC warning is generated for <tt style="background: #ebebeb; font-size: 13px;">in_devs</tt>.  It turns out the warning was referring to a Coverity entry (but the ID was wrong, it's 253228, not 3228), the Coverity entry noted that the <tt style="background: #ebebeb; font-size: 13px;">in_devs</tt> code is dead code.  And so it is.  Whatever it was supposed to do before (warn about RAID entries using another RAID device as a sub-device?), it doesn't do it now --> removed.</li>
</ol>

<ol class="remarkup-list" start="2">
<li class="remarkup-list-item">A boolean check for whether the raid array was active or not was miscoded (and therefore was always true).  The compiler warning for this was actually what led to me looking through this code, but Coverity also flagged it as well.  From inspection, the check actually is necessary since the "level" data we're extracting here is only present if the array is active, so I fixed by removing the errorneous condition.</li>
</ol></div></div><br /><div><strong>TEST PLAN</strong><div><p>Builds fine on glibc and musl libc.</p>

<p>However since I don't have a RAID array I have no good way of testing the change itself, though it should at least not break anything worse than it's already broken.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R106 KSysguard</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D6658" rel="noreferrer">https://phabricator.kde.org/D6658</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>ksysguardd/Linux/softraid.c</div></div></div><br /><div><strong>To: </strong>mpyne, Plasma<br /><strong>Cc: </strong>plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas<br /></div>