<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Mar 31, 2021 at 11:24 AM 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 Wed, Mar 31, 2021 at 10:21 AM Ryan Long <<a href="mailto:ryan.long@oarcorp.com" target="_blank">ryan.long@oarcorp.com</a>> wrote:<br>
><br>
> I'm assuming you accidentally left the (void) in after that #endif, but that's the approach that Joel and I did first.  After Joel saw it implemented, he said it made the code look too messy to him because of the if statements.<br>
><br>
Right. The thing is that we aren't looking to make this code clean,<br>
we're optimizing for easier updates from the upstream. Generally, it<br>
is easier to undo and redo small changes than large changes.<br></blockquote><div><br></div><div>If you are OK with an ugly #if (void) #endif, then I'm OK with it.</div><div><br></div><div>It is  ugly either way.</div><div><br></div><div>Make the void ifdef'ed. Sorry about the pendulum swinging.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> -----Original Message-----<br>
> From: Gedare Bloom <<a href="mailto:gedare@rtems.org" target="_blank">gedare@rtems.org</a>><br>
> Sent: Wednesday, March 31, 2021 11:17 AM<br>
> To: Ryan Long <<a href="mailto:ryan.long@oarcorp.com" target="_blank">ryan.long@oarcorp.com</a>><br>
> Cc: <a href="mailto:devel@rtems.org" target="_blank">devel@rtems.org</a><br>
> Subject: Re: [PATCH 2/3] main_cp.c: Ignore return value from stat()<br>
><br>
> On Wed, Mar 31, 2021 at 9:16 AM Ryan Long <<a href="mailto:ryan.long@oarcorp.com" target="_blank">ryan.long@oarcorp.com</a>> wrote:<br>
> ><br>
> > CID 26051: Unchecked return value from library in main_cp().<br>
> ><br>
> > Closes #4365<br>
> > ---<br>
> >  cpukit/libmisc/shell/main_cp.c | 7 +++++++<br>
> >  1 file changed, 7 insertions(+)<br>
> ><br>
> > diff --git a/cpukit/libmisc/shell/main_cp.c<br>
> > b/cpukit/libmisc/shell/main_cp.c index cddbc95..bb31a35 100644<br>
> > --- a/cpukit/libmisc/shell/main_cp.c<br>
> > +++ b/cpukit/libmisc/shell/main_cp.c<br>
> > @@ -254,10 +254,17 @@ main_cp(rtems_shell_cp_globals* cp_globals, int argc, char *argv[])<br>
> >                  * the initial mkdir().<br>
> >                  */<br>
> >                 if (r == -1) {<br>
> > +                       #ifdef __rtems__<br>
> > +                       if (Rflag && (Lflag || Hflag))<br>
> > +                               (void) stat(*argv, &tmp_stat);<br>
> > +                       else<br>
> > +                               (void) lstat(*argv, &tmp_stat);<br>
> > +                       #else<br>
> >                         if (Rflag && (Lflag || Hflag))<br>
> >                                 stat(*argv, &tmp_stat);<br>
> >                         else<br>
> >                                 lstat(*argv, &tmp_stat);<br>
> > +                       #endif<br>
> ><br>
><br>
> Would it be better to keep each change localized? at first I didn't really care for this approach that was taken in patch 1, but compared to copy-pasting multiple lines of code, it might be simpler to deal with fine-grained modifications, like this:<br>
><br>
>                          if (Rflag && (Lflag || Hflag))<br>
> +                               #ifdef __rtems__<br>
> +                               (void)<br>
> +                               #endif<br>
>                                  (void) stat(*argv, &tmp_stat);<br>
><br>
> >                         if (S_ISDIR(tmp_stat.st_mode) && Rflag)<br>
> >                                 type = DIR_TO_DNE;<br>
> > --<br>
> > 1.8.3.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>
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></div>