<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Sep 12, 2024 at 11:35 PM Rob Landley <<a href="mailto:rob@landley.net">rob@landley.net</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 9/10/24 17:16, enh via Toybox wrote:<br>
> cp -i was showing the _source_ rather than the destination, while -v<br>
> was only showing the source rather than the source _and_ the destination.<br>
<br>
In theory flags is separate because it gets locally modified,</blockquote><div><br></div><div>oops, i didn't spot that ... i thought it was one of those "multiple toys in the same file with almost-but-not-quite-the-same flags" cases. but since there was already a FLAG() in the same function, i just assumed it was okay in this context.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> but at this point<br>
it's only doing that for FLAG_r so you're right all the other tests can just use<br>
the FLAG() macros. In fact having a separate extra r flag is probably the<br>
correct thing to do here. (I think there were more local flag modifications<br>
before, but they got cleaned up at some point...)<br>
<br>
Applied, I'll try to do a follow-up patch this evening...<br></blockquote><div><br></div><div>thanks!</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Rob<br>
</blockquote></div></div>