[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug #44221] Spurious warning about non-option arguments is spurious
From: |
James Youngman |
Subject: |
[bug #44221] Spurious warning about non-option arguments is spurious |
Date: |
Sun, 22 Feb 2015 17:03:49 +0000 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/39.0.2171.65 Safari/537.36 |
Update of bug #44221 (project findutils):
Severity: 3 - Normal => 2 - Minor
Status: None => Wont Fix
Open/Closed: Open => Closed
_______________________________________________________
Follow-up Comment #8:
Dale's most recent comment captures precisely the reason why I added this
warning (see
http://git.savannah.gnu.org/cgit/findutils.git/commit/?id=d4ed83819c9ced4a2453473851456177d6bb4857).
The initial bug report does limit the scope of the complaint to command-lines
in which all tests are joined by AND conditions. In those cases the warning
is, perhaps, useless. However, it's quite complex to implement an exception
for that case accurately, since the options aren't propagated into the
resulting expression tree. Limiting the cases in which the warning is
issued would likely require either leaving a placeholder for each global
option or perhaps parsing the command line an extra time.
I don't believe the additional complexity of either approach is justified.
For that reason, and despite the fact that the complaint does have merit, I'm
going to close this bug as working-as-intended (for which the nearest status
code in the Savannah UI is "Won't Fix").
However, I'm willing for this not to be the end of the discussion. If you
think that "find" can be improved in this respect and at a very small
complexity cost, please feel free to contribute (see README.hacking) a patch
demonstrating your point.
_______________________________________________________
Reply to this item at:
<http://savannah.gnu.org/bugs/?44221>
_______________________________________________
Message sent via/by Savannah
http://savannah.gnu.org/
- [bug #44221] Spurious warning about non-option arguments is spurious, anonymous, 2015/02/10
- [bug #44221] Spurious warning about non-option arguments is spurious, Kamil Dudka, 2015/02/10
- [bug #44221] Spurious warning about non-option arguments is spurious, Andrew McGill, 2015/02/12
- [bug #44221] Spurious warning about non-option arguments is spurious, Kamil Dudka, 2015/02/12
- [bug #44221] Spurious warning about non-option arguments is spurious, Andrew McGill, 2015/02/12
- [bug #44221] Spurious warning about non-option arguments is spurious, Dale Worley, 2015/02/15
- [bug #44221] Spurious warning about non-option arguments is spurious, Andrew McGill, 2015/02/16
- [bug #44221] Spurious warning about non-option arguments is spurious, Dale Worley, 2015/02/16
- [bug #44221] Spurious warning about non-option arguments is spurious,
James Youngman <=
- Re: [bug #44221] Spurious warning about non-option arguments is spurious, Paul E Condon, 2015/02/24