Re: pfctl: Simplify host()

2018-07-29 Thread Alexandr Nedvedicky
Hello, On Sun, Jul 29, 2018 at 12:35:22PM +0200, Klemens Nanni wrote: > This gets rid of the `cont' flag and squashes the code a bit. `host_*()' > are pretty self explanatory so I zapped the comments as well. > > Regress tests pass, no issues in production use. > > Feedback? OK? reads OK to

Re: Make systat(1) more human readable

2018-07-29 Thread Martijn van Duren
Just got some positive user feedback off-list. Any takers? On 02/11/18 13:48, Martijn van Duren wrote: > Some documentation nits found by jmc. > > On 02/11/18 12:31, Martijn van Duren wrote: >> Hello tech@, >> >> Since my brain is incapable of quickly processing numbers larger than a >> million (

[diff] acme-client - clean up main.c

2018-07-29 Thread Ross L Richardson
[This diff is very similar to one I sent previously off-list to a few.] Just some style(9) and simple cleanup: - order getopt string and switch cases - add space between "if" and "(" - wrap a long line - return rather than exit() from main() - move chngdir == NULL test to where it belongs Ques

Re: [bugfix] xterm(1) needs "cpath" pledge(2)

2018-07-29 Thread lists
Sun, 29 Jul 2018 15:46:20 +0300 (EEST) Leonid Bobrov > Lauri, Matthieu, thank you for saving me the trouble having to read and > understand the code of this big fat hippopotamus. > > So, that means somehow I actually accidentally pressed annoying touchpad > and was lucky enough to click "Print-Al

Re: paste(1): use getline instead of fgetln

2018-07-29 Thread Ingo Schwarze
Hi, Lauri Tirkkonen wrote on Tue, Jul 17, 2018 at 09:14:29PM +0300: > In the same vein as my previous diff for join(1), > make paste(1) use getline instead of fgetln. I think it's correct, but i tweaked it a bit for simplicity. In sequential(), the double loop with double getline(), both with e

Re: remove uaudio_id_name()

2018-07-29 Thread Klemens Nanni
On Sun, Jul 29, 2018 at 09:31:51PM +0800, Michael W. Bombardieri wrote: > The function uaudio_id_name() was used once for formatting a string > and its return value was used to format a second string. By rolling > the prepended letter "i" into the outer snprintf() the function can > be removed enti

remove uaudio_id_name()

2018-07-29 Thread Michael W. Bombardieri
Hello, The function uaudio_id_name() was used once for formatting a string and its return value was used to format a second string. By rolling the prepended letter "i" into the outer snprintf() the function can be removed entirely. Does this look correct? - Michael Index: uaudio.c =

Re: stop using WSMOUSE_TYPE_ELANTECH in other drivers (xenocara)

2018-07-29 Thread Ulf Brosziewski
It's still possible to use synaptics as alternative to ws. The new type would break such a configuration for ubcmtp, iatp, and hidmt. I think as long as it doesn't take too much efforts, we should keep synaptics up to date and add the diff below to this one. However, I don't insist on that, so

Re: stop using WSMOUSE_TYPE_ELANTECH in other drivers

2018-07-29 Thread Ulf Brosziewski
ok bru@ (but please see my comments on the second diff) On 07/27/2018 09:59 PM, joshua stein wrote: > Back when touchpad drivers were using the synaptics Xorg driver, > they had to pretend to be Elantech devices in order to get > particular packet processing. > > Since Ulf switched us to wstpad

Re: [bugfix] xterm(1) needs "cpath" pledge(2)

2018-07-29 Thread Leonid Bobrov
Lauri, Matthieu, thank you for saving me the trouble having to read and understand the code of this big fat hippopotamus. So, that means somehow I actually accidentally pressed annoying touchpad and was lucky enough to click "Print-All Immediately" without noticing that... I regret that I've sent

Re: [bugfix] xterm(1) needs "cpath" pledge(2)

2018-07-29 Thread Theo de Raadt
Matthieu Herrb wrote: > > but to decide, we should know *what* triggered this behaviour. > > Hi, > > After digging a bit, there is at least the 'Print All Immediatly' > function from button 1 menu that will trigger the creation of a file > and violate the pledge. > > see xtermPrintImmediately

pfctl: Simplify host()

2018-07-29 Thread Klemens Nanni
This gets rid of the `cont' flag and squashes the code a bit. `host_*()' are pretty self explanatory so I zapped the comments as well. Regress tests pass, no issues in production use. Feedback? OK? Index: pfctl_parser.c === RCS file

Re: [bugfix] xterm(1) needs "cpath" pledge(2)

2018-07-29 Thread Sebastien Marie
On Sun, Jul 29, 2018 at 08:43:22AM +0200, Matthieu Herrb wrote: > On Sun, Jul 29, 2018 at 07:28:19AM +0200, Sebastien Marie wrote: > > > > but to decide, we should know *what* triggered this behaviour. > > Hi, > > After digging a bit, there is at least the 'Print All Immediatly' > function from