-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
stty: fix negated operands #6798
base: main
Are you sure you want to change the base?
stty: fix negated operands #6798
Conversation
GNU testsuite comparison:
|
d184170
to
71b99a3
Compare
GNU testsuite comparison:
|
71b99a3
to
204062e
Compare
GNU testsuite comparison:
|
caf4b74
to
50daa44
Compare
GNU testsuite comparison:
|
0feb930
to
97868ee
Compare
97868ee
to
1c59495
Compare
GNU testsuite comparison:
|
1c59495
to
7a391e5
Compare
GNU testsuite comparison:
|
1 similar comment
GNU testsuite comparison:
|
@@ -0,0 +1,240 @@ | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if it is generated at build time, does it need to be in tree ?
and if the answer is yes, could you please add a comment on top of this file to explain what it is, thanks
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@andrewliebenow ping ? :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ping? :)
cffaef0
to
6e66be7
Compare
GNU testsuite comparison:
|
Also: use correct file descriptor (stdin) to look up tty Also: fix handling of end of options delimiter ("--") Also: use `StdoutLock` for printing
6e66be7
to
85c9c94
Compare
GNU testsuite comparison:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice PR! I'd suggest splitting this up a bit to make it easier for review. I'd also prefer trying an implementation without phf
first, so that we can make a properly benchmarked decision on whether phf
is worth it in this case. Adding build scripts and generated files is usually a bit of a hassle and hard for incremental compilation.
(I'm not really involved with the project much anymore and mostly chiming in as the author of stty. The other maintainers' feedback is much more important)
Also: use correct file descriptor (stdin) to look up tty
Also: fix handling of end of options delimiter ("--")
Also: use
StdoutLock
for printing