clipdel: add -q to suppress output#260
Conversation
This seems like a valid and universal solution to me. What's the problem with this? |
|
Redirection works, and that's a fair point, but native support for quietness is common in similar utilities for better ergonomics. It keeps scripts cleaner and I believe it's a low-maintenance addition that makes the tool more intuitive. |
|
Hi! Just checking on this PR as it's been a while. Happy to make any change if needed. |
|
Apologies for not getting back sooner. My opinion is basically the same as @N-R-K I guess. I don't feel strongly about it either way, but I also think there's the question to ask of:
This is a pretty weak argument since I doubt clipmenu as a suite of tools will grow that much, or the functionality either, but using So I guess I weakly prefer to leave it as is. |
|
I get your point and understand it makes sense to leave it as is. Thanks for your feedback! |
While working on an script involving clipmenu I ran into it printing all the pastes to be deleted to stdout. This was a major inconvenience as I was trying to delete sensitive information following a list of regexs.
This PR adds a -q (quiet) flag to allow the output to be suppressed natively without the need for workarounds (as redirecting output to /dev/null).
Changes:
added -q flag to clipdel.c
updated clipdel.1 man page
Added integration tests in tests/x_integration_tests to verify the quiet behavior