<div dir="ltr">Okay I will understand the programmer's intentions from now on first. </div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Mar 2, 2020 at 9:10 PM Gedare Bloom <<a href="mailto:gedare@rtems.org">gedare@rtems.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Sun, Mar 1, 2020 at 11:32 AM Joel Sherrill <<a href="mailto:joel@rtems.org" target="_blank">joel@rtems.org</a>> wrote:<br>
><br>
><br>
><br>
> On Sun, Mar 1, 2020, 12:03 AM suyash singh <<a href="mailto:suyashsingh234@gmail.com" target="_blank">suyashsingh234@gmail.com</a>> wrote:<br>
>><br>
>> I don't know. There are checks for other things in the function when it return other than successful.<br>
><br>
><br>
> I don't know this code but count starts at 0. If nothing is processed, I would expect the count to be 0 after the loop.<br>
><br>
> I would tend to change the <= to ==. If no data was processed, that's the condition. The code is just slightly off.<br>
<br>
The question is intent. This is part of grlib driver, I'm not sure<br>
what its interface requires from 'write', there are two cases, either<br>
write of 0 bytes is RTEMS_UNSUCCESSFUL or any write is RTEMS_SUCCESSFUL.<br>
<br>
I suspect a write of 0 bytes is not supposed to indicate<br>
RTEMS_SUCCESSFUL. So changing >= to > might be correct.<br>
<br>
>><br>
>><br>
>> Since it was never going to return "RTEMS_UNSATISFIED" as the "if" would always evaluate to true I removed the unnecessary comparison<br>
>><br>
It is important to try to understand the programmer's intended<br>
behavior when dealing with these kinds of dead code issues. Maybe it<br>
is a bug for it to always evaluate to true? Clearly, someone thought<br>
at some point there would be a reason that it could be false.<br>
<br>
>> On Sat, Feb 29, 2020 at 2:52 AM Peter Dufault <<a href="mailto:dufault@hda.com" target="_blank">dufault@hda.com</a>> wrote:<br>
>>><br>
>>> And regardless of the value of count it is successful?<br>
>>><br>
>>> > On Feb 28, 2020, at 12:17 , suyash singh <<a href="mailto:suyashsingh234@gmail.com" target="_blank">suyashsingh234@gmail.com</a>> wrote:<br>
>>> ><br>
>>> > count is unsigned int and will always be >=0.<br>
>>> ><br>
>>> > On Fri, Feb 28, 2020 at 10:42 PM suyash singh <<a href="mailto:suyashsingh234@gmail.com" target="_blank">suyashsingh234@gmail.com</a>> wrote:<br>
>>> > ---<br>
>>> >  bsps/shared/grlib/1553/b1553brm.c | 6 ++----<br>
>>> >  1 file changed, 2 insertions(+), 4 deletions(-)<br>
>>> ><br>
>>> > diff --git a/bsps/shared/grlib/1553/b1553brm.c b/bsps/shared/grlib/1553/b1553brm.c<br>
>>> > index 57ef70126b..4041423541 100644<br>
>>> > --- a/bsps/shared/grlib/1553/b1553brm.c<br>
>>> > +++ b/bsps/shared/grlib/1553/b1553brm.c<br>
>>> > @@ -982,10 +982,8 @@ static rtems_device_driver brm_write(rtems_device_major_number major, rtems_devi<br>
>>> ><br>
>>> >         rw_args->bytes_moved = count;<br>
>>> ><br>
>>> > -       if (count >= 0) {<br>
>>> > -               return RTEMS_SUCCESSFUL;<br>
>>> > -       }<br>
>>> > -       return RTEMS_UNSATISFIED;<br>
>>> > +       return RTEMS_SUCCESSFUL;<br>
>>> > +<br>
>>> >  }<br>
>>> ><br>
>>> >  static rtems_device_driver brm_control(rtems_device_major_number major, rtems_device_minor_number minor, void *arg)<br>
>>> > --<br>
>>> > 2.17.1<br>
>>> ><br>
>>> > _______________________________________________<br>
>>> > devel mailing list<br>
>>> > <a href="mailto:devel@rtems.org" target="_blank">devel@rtems.org</a><br>
>>> > <a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
>>><br>
>>> Peter<br>
>>> -----------------<br>
>>> Peter Dufault<br>
>>> HD Associates, Inc.      Software and System Engineering<br>
>>><br>
>>> This email is delivered through the public internet using protocols subject to interception and tampering.<br>
>>><br>
>> _______________________________________________<br>
>> devel mailing list<br>
>> <a href="mailto:devel@rtems.org" target="_blank">devel@rtems.org</a><br>
>> <a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
><br>
> _______________________________________________<br>
> devel mailing list<br>
> <a href="mailto:devel@rtems.org" target="_blank">devel@rtems.org</a><br>
> <a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
</blockquote></div>