Re: [hackers] [st][PATCH] base64_digits: reduce scope, implicit zero, +1 size

2022-03-18 Thread Hiltjo Posthuma
On Fri, Mar 18, 2022 at 05:03:34PM +0600, NRK wrote: > the array is not accessed outside of base64dec() so it makes sense to > limit it's scope to the related function. the static-storage duration of > the array is kept intact. > > this also removes unnecessary explicit zeroing from the start and

Re: [hackers] [st][PATCH] rm unnecessary explicit zeroing

2022-03-18 Thread Hiltjo Posthuma
On Fri, Mar 18, 2022 at 02:11:27PM +0600, NRK wrote: > On Fri, Mar 18, 2022 at 12:40:00AM +0100, Roberto E. Vargas Caballero wrote: > > Uh, I didn't realize about it, I just saw that having 255 entries was > > wrong ^^!!!. > > I think the best approach is to split the commit in two and

Re: [hackers] Re: [dwm][PATCH v3] manage: For isfloating/oldstate check/set, ensure trans client actually exists

2022-03-13 Thread Hiltjo Posthuma
On Fri, Mar 11, 2022 at 11:25:18AM -0500, Miles Alan wrote: > On Fri, Mar 11, 2022, at 9:11 AM, sir fish wrote: > > Miles Alan wrote: > >> On Mon, Feb 21, 2022, at 1:10 AM, m...@milesalan.com wrote: > >> > From: Miles Alan > >> > > >> > In certain instances trans may be set to a window that

Re: [hackers] [dwm][PATCH] fix mem leak in cleanup()

2022-03-13 Thread Hiltjo Posthuma
On Fri, Mar 11, 2022 at 08:40:05PM +0600, NRK wrote: > maybe leak isn't the best word, given that the object lives for the > entire duration of the program's lifetime. > > however, all elements of scheme are free-ed, can't think of any reason > why scheme itself should be an exception. > --- >

Re: [hackers] [st][patch] Made underlines and strikethroughs respect `chscale`.

2022-03-13 Thread Hiltjo Posthuma
On Sat, Mar 12, 2022 at 04:26:14PM +, Zacchary Dempsey-Plante wrote: > On Saturday, March 12th, 2022 at 16:19, Hiltjo Posthuma > wrote: > > > On Sat, Mar 12, 2022 at 03:24:57PM +, Zacchary Dempsey-Plante wrote: > > > > > During my submission of the char

Re: [hackers] [st][patch] Made underlines and strikethroughs respect `chscale`.

2022-03-12 Thread Hiltjo Posthuma
On Sat, Mar 12, 2022 at 03:24:57PM +, Zacchary Dempsey-Plante wrote: > During my submission of the charoffsets customisation patch, I found that the > underline and strikethrough rendering didn't take `chscale` into account. > This meant that when using `chscale < 1.0`, underlines were not

Re: [hackers] Re: [dwm][PATCH v3] manage: For isfloating/oldstate check/set, ensure trans client actually exists

2022-03-08 Thread Hiltjo Posthuma
On Tue, Mar 08, 2022 at 12:56:27PM -0500, Miles Alan wrote: > On Mon, Feb 21, 2022, at 1:10 AM, m...@milesalan.com wrote: > > From: Miles Alan > > > > In certain instances trans may be set to a window that doesn't actually > > map to a client via wintoclient; in this case it doesn't make sense >

Re: [hackers] [dmenu][PATCH] Remove warning for int comparison as bool

2022-03-01 Thread Hiltjo Posthuma
On Fri, Feb 25, 2022 at 11:07:49AM +0530, Prathu Baronia wrote: > - Compare the result of the macro with 0 instead of treating as bool to remove > the following warning. > > dmenu.c: In function ‘setup’: > dmenu.c:24:30: warning: ‘*’ in boolean context, suggest ‘&&’ instead >

Re: [hackers] st][PATCH - proper escape sequence for CTRL+HOME

2022-03-01 Thread Hiltjo Posthuma
On Tue, Mar 01, 2022 at 07:49:21AM -0800, Jeremy wrote: > On 02/28/22 09:27PM, Dave Blanchard wrote: > > On that note, regrettably it will be necessary for me to fork this project, > > if for no other reason than to properly comment it, so that its > > functionality can be understood and easily

Re: [hackers] st][PATCH - proper escape sequence for CTRL+HOME

2022-03-01 Thread Hiltjo Posthuma
On Mon, Feb 28, 2022 at 09:27:22PM -0600, Dave Blanchard wrote: > Hello all, > Hi David, > This patch for 'st' causes CTRL+HOME to send the ANSI sequence \033[J and > \033[1;5H , which signals the user program to scroll to the top of the > document, same as in Xterm. > > I have absolutely

Re: [hackers] [dwm][PATCH] Added support for RTL languages (Farsi, Arabic and Hebrew using the FriBiDi library)

2022-02-19 Thread Hiltjo Posthuma
On Sat, Feb 19, 2022 at 07:49:28AM +0330, Mahdi Mirzade wrote: > From: mahdymirzade > > --- > config.mk | 8 ++-- > dwm.c | 45 - > 2 files changed, 42 insertions(+), 11 deletions(-) > > diff --git a/config.mk b/config.mk > index

Re: [hackers] [st][PATCH] Delay redrawals on palette changes

2022-02-18 Thread Hiltjo Posthuma
On Thu, Feb 17, 2022 at 04:00:47PM +0200, Santtu Lakkala wrote: > Build on auto-sync and only mark window dirty on palette changes and let > the event handler do the actual draw. > --- > st.c | 8 > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/st.c b/st.c > index

Re: [hackers] [dwm][PATCH] Use proper conversion specifier and don't assume int == 32bits

2022-02-16 Thread Hiltjo Posthuma
On Wed, Feb 16, 2022 at 01:07:37PM +0100, Laslo Hunhold wrote: > On Wed, 16 Feb 2022 17:46:47 +0600 > NRK wrote: > > Dear NRK, > > > Attached two small patches, one fixing the conversion specifier to > > `%u` for unsigned int and another one not for not assuming int == > > 32bits. > > > >

Re: [hackers] [dmenu] follow-up fix: add -D_GNU_SOURCE for strcasestr for some systems || Hiltjo Posthuma

2022-02-07 Thread Hiltjo Posthuma
On Mon, Feb 07, 2022 at 02:31:24PM +0100, Quentin Rameau wrote: > Hola, > > Here are my two cents, > > > On Mon, Feb 07, 2022 at 10:41:58AM +0100, Laslo Hunhold wrote: > > > On Mon, 7 Feb 2022 10:36:46 +0100 (CET) > > > g...@suckless.org wrote: > > > > > > Dear Hiltjo, > > > > > > >

Re: [hackers] [dmenu] follow-up fix: add -D_GNU_SOURCE for strcasestr for some systems || Hiltjo Posthuma

2022-02-07 Thread Hiltjo Posthuma
On Mon, Feb 07, 2022 at 10:41:58AM +0100, Laslo Hunhold wrote: > On Mon, 7 Feb 2022 10:36:46 +0100 (CET) > g...@suckless.org wrote: > > Dear Hiltjo, > > > follow-up fix: add -D_GNU_SOURCE for strcasestr for some systems > > wouldn't it be better to avoid GNU-extensions in code? > > With

Re: [hackers] [st][PATCH] Fix mousereport

2022-01-10 Thread Hiltjo Posthuma
On Sat, Jan 08, 2022 at 11:40:34AM -0800, robert wrote: > This patch replaces the previous one I sent. > > The following changes are made in this patch: > - Fix tracking of pressed buttons. Previously, pressing two buttons and >then releasing one would make st think no buttons are pressed,

Re: [hackers] Re: [st][PATCH] Fix MODE_MOUSEMOTION reporting

2022-01-08 Thread Hiltjo Posthuma
On Fri, Jan 07, 2022 at 09:16:15PM -0800, Robert Russell wrote: > I realized I misinterpreted something I read about how DECSET 1002 (i.e., > MODE_MOUSEMOTION) should behave, so my patch is wrong for a different reason. > The motion event should encode one of the (possibly many) pressed buttons. I

Re: [hackers] [st][PATCH] Fix overtyping wide characters.

2021-12-30 Thread Hiltjo Posthuma
On Wed, Dec 29, 2021 at 09:07:17AM -0800, jamin wrote: > Overtyping the first half of a wide character with the > second half of a wide character results in display garbage. > This is because the trailing dummy is not cleaned up. > > i.e. ATTR_WIDE, ATTR_WDUMMY, ATTR_WDUMMY > > Here is a short

Re: [hackers] [PATCH st v2] Fix null pointer access in strhandle

2021-12-27 Thread Hiltjo Posthuma
On Mon, Dec 27, 2021 at 11:41:42AM +0100, suckl...@jochen.sprickerhof.de wrote: > From: Jochen Sprickerhof > > According to the spec the argument is optional for 104, so p can be > NULL as can be tested with printf '\x1b]104\x07'. This is a regression > of 8e31030. > --- > st.c | 4 ++-- > 1

Re: [hackers] Re: [dmenu][patch] Improve speed of drw_text when provided with large strings

2021-12-26 Thread Hiltjo Posthuma
On Mon, Aug 09, 2021 at 06:35:57PM +0200, Hiltjo Posthuma wrote: > On Sat, Aug 07, 2021 at 09:39:44PM -0400, Miles Alan wrote: > > On Sat, Aug 7, 2021, at 9:34 PM, m...@milesalan.com wrote: > > > - for (len = MIN(utf8strlen, sizeof(buf) - 1); len &am

Re: [hackers][st][PATCH] Add support for OSC color sequences.

2021-12-26 Thread Hiltjo Posthuma
a crash here while testing. > > Ok. > > >> +void xgetcolor(int x, unsigned char *r, unsigned char *g, unsigned char > *b); > > > xgetcolor should be below xstrdup and as a separate line. > > Will do. > > Thanks for reviewing the patch :). I have attached

Re: [hackers] [dwm][PATCH] drawbar: Don't expend effort drawing bar if it is occluded

2021-12-19 Thread Hiltjo Posthuma
On Sat, Dec 18, 2021 at 04:58:23PM +, Chris Down wrote: > I noticed that a non-trivial amount of dwm's work on my machine was from > drw_text, which seemed weird, because I have the bar disabled and we > only use drw_text as part of bar drawing. > > Looking more closely, I realised that while

Re: [hackers] [dwm][patch] gaplessgrid focus left/right

2021-12-17 Thread Hiltjo Posthuma
You send it to the wrong mailinglist. If its a wiki patch you can upload it, see the wiki guidelines. Thanks, On Fri, Dec 17, 2021 at 02:03:15AM +0200, שלומי אקנין wrote: > From 76d72e24117a5626827cfb0ef49515ec7f7dd4fe Mon Sep 17 00:00:00 2001 > From: shlomi-aknin > Date: Thu, 16 Dec 2021

Re: [hackers][st][PATCH] Add support for OSC color sequences.

2021-12-11 Thread Hiltjo Posthuma
Hi, Some comments below: On Fri, Dec 10, 2021 at 05:53:40PM -0500, Raheman Vaiya wrote: > The attached patch adds full support for the OSC 10/11/12/4 control > sequences. > These are used by programs like vim and theme.sh to query and set terminal > colours and are supported by most major

Re: [hackers] [dwm][patch] uselessgap fix

2021-11-21 Thread Hiltjo Posthuma
This doesn't below here, but on the wiki. You can push it yourself to the wiki repository as described on the wiki page. On Sun, Nov 21, 2021 at 11:42:05AM +0100, thim wrote: > The inner gap between the master and slave stack is currently double of > whatever the gappx variable is. This should

Re: [hackers] [PATCH svkbd] Fix color scheme editing on smiley with Xresources

2021-10-09 Thread Hiltjo Posthuma
On Fri, Oct 08, 2021 at 09:38:22PM +0200, Maarten van Gompel wrote: > From: Justin Torres > > Fixes a small oversight that was preventing the colors of the smiley on q > from changing with Xresources. > > Signed-off-by: Maarten van Gompel > --- > svkbd.c | 8 > 1 file changed, 4

Re: [hackers] [svkbd][PATCH] create layout-dependent object files

2021-10-01 Thread Hiltjo Posthuma
On Thu, Sep 30, 2021 at 09:36:53PM +0200, Max Schillinger wrote: > When you build svkbd with different layouts, you get one binary per > layout but all binaries are identical: > > $ make LAYOUT=mobile-plain > $ make LAYOUT=mobile-simple > $ diff -s svkbd-mobile-plain

Re: [hackers] [st][PATCH 1/1] fix a problem that the standard streams are unexpectedly closed

2021-08-24 Thread Hiltjo Posthuma
On Tue, Aug 24, 2021 at 06:25:05AM +0900, Koichi Murase wrote: > In the current implementation, the slave PTY (assigned to the variable > `s') is always closed after duplicating it to file descriptors of > standard streams (0, 1, and 2). However, when the allocated slave PTY > `s' is already one

Re: [hackers] [dwm][PATCH] Fix wrong computation in drw_text

2021-08-19 Thread Hiltjo Posthuma
On Thu, Aug 19, 2021 at 05:30:14AM +0200, Viktor Grigorov wrote: > This results in 1-character elements being unrendered. > I was wondering why two of my surfraw elvi were blank, they just were getting > drawn. > > Aug 16, 2021, 20:17 by suckl...@jochen.sprickerhof.de: > > > From: Jochen

Re: [hackers] [dmenu][PATCH] turn -b into a toggle

2021-08-16 Thread Hiltjo Posthuma
On Sun, Aug 15, 2021 at 11:44:58PM +0600, NRK wrote: > currently config.h allows users to set the value of topbar to 0. > however if one does that, there's no way for him to get a topbar again. > it makes more sense to have -b as a toggle instead. > > - NRK Hi, Thanks for the patch. I'd rather

Re: [hackers] Re: [dmenu][patch] Improve speed of drw_text when provided with large strings

2021-08-09 Thread Hiltjo Posthuma
On Sat, Aug 07, 2021 at 09:39:44PM -0400, Miles Alan wrote: > On Sat, Aug 7, 2021, at 9:34 PM, m...@milesalan.com wrote: > > - for (len = MIN(utf8strlen, sizeof(buf) - 1); len && ew > > > w; len--) > > - drw_font_getexts(usedfont, utf8str, len, , > >

Re: [hackers] [dwm][floatrules] patch bugfix

2021-08-02 Thread Hiltjo Posthuma
On Sun, Aug 01, 2021 at 10:39:27PM -0300, Natanael Rabello wrote: > The following is a new diff revision for the dwm patch floatrules. > It contains bug fixes and adds support for allowing float x,y,w,h to > be unspecified (-1); that is for floating windows that already set the > geometry hints to

Re: [hackers] [PATCH svkbd] Version bump prior to release

2021-07-30 Thread Hiltjo Posthuma
On Fri, Jul 30, 2021 at 02:07:19PM +0200, Maarten van Gompel wrote: > This is also a release request, could you tag and release svkbd 0.4? > We're planning packaging and a release of Sxmo 1.5.0 this weekend > that includes it. > > --- > Makefile | 2 +- > 1 file changed, 1 insertion(+), 1

[hackers] [patch] [sbase] printf: allow flags for the %s format string aswell

2021-07-25 Thread Hiltjo Posthuma
Hi, >From da499fb89b024e2fdbbe04652b00b1f7f1152523 Mon Sep 17 00:00:00 2001 From: Hiltjo Posthuma Date: Sun, 25 Jul 2021 14:24:41 +0200 Subject: [PATCH] printf: allow flags for the %s format string aswell This is useful for example to left-align strings and pad them with spaces. pri

Re: [hackers] [st][PATCH] Add 14th bit to XK_SWITCH_MOD bitmask

2021-07-18 Thread Hiltjo Posthuma
On Sun, Jul 18, 2021 at 12:14:00AM +0200, petarkap...@firemail.cc wrote: > From: Petar Kapriš > > The bits of uint signal in an XKeyEvent which concern the key group (keyboard > layout) are bits 13 and 14, as documented here: >

[hackers] [sbase] tar: check if reallocarray failed

2021-07-17 Thread Hiltjo Posthuma
Hi, The patch below is for sbase tar: >From 2eec3e07a5bd1ed1fa41ca02865297ab7d8b5fa8 Mon Sep 17 00:00:00 2001 From: Hiltjo Posthuma Date: Sat, 17 Jul 2021 21:03:27 +0200 Subject: [PATCH] tar: check if reallocarray failed --- tar.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) d

Re: [hackers] [PATCH] Add a configuration option for fullscreen locking

2021-07-14 Thread Hiltjo Posthuma
On Tue, Jul 13, 2021 at 08:04:19PM +0200, Hiltjo Posthuma wrote: > On Mon, Jul 12, 2021 at 11:44:16PM +0200, Quentin Rameau wrote: > > Some people are annoyed to have this new behaviour forced for some > > application which use fake fullscreen. > > --- > > config.def.h

Re: [hackers] [PATCH] Add a configuration option for fullscreen locking

2021-07-13 Thread Hiltjo Posthuma
On Mon, Jul 12, 2021 at 11:44:16PM +0200, Quentin Rameau wrote: > Some people are annoyed to have this new behaviour forced for some > application which use fake fullscreen. > --- > config.def.h | 1 + > dwm.c| 2 +- > 2 files changed, 2 insertions(+), 1 deletion(-) > > diff --git

Re: [hackers] [st][PATCH] arg.h: optimize & style

2021-07-04 Thread Hiltjo Posthuma
On Sun, Jul 04, 2021 at 12:30:27PM +0500, Nikita Zlobin wrote: > - improved code readability > - cleaned redundant variables and checks > - isolated reusable part for (E)ARGF > --- > arg.h | 65 +-- > 1 file changed, 32 insertions(+), 33

Re: [hackers] [sites][PATCH] Fixed the title of the page

2021-05-28 Thread Hiltjo Posthuma
On Thu, May 27, 2021 at 05:38:24PM +0100, The-Repo-Club wrote: > --- > tools.suckless.org/dmenu/patches/emoji-highlight/index.md | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/tools.suckless.org/dmenu/patches/emoji-highlight/index.md >

Re: [hackers] dvtm: add a comma at the end of kf22 entry in dvtm.info

2021-05-21 Thread Hiltjo Posthuma
On Fri, May 21, 2021 at 11:05:25AM -0400, Greg Reagle wrote: > On Fri, May 21, 2021, at 04:50, Hiltjo Posthuma wrote: > > dvtm is not a project on suckless.org (minor detail) ;) > > I see. I wrote to this list because of these instructions from > https://github.com/martanne/d

Re: [hackers] dvtm: add a comma at the end of kf22 entry in dvtm.info

2021-05-21 Thread Hiltjo Posthuma
On Thu, May 20, 2021 at 05:01:44PM -0400, Greg Reagle wrote: > Patch attached. Sending an email seems a *lot* easier than a pull request on > github, which would require me to clone and update (unless I am missing > something, which is not unlikely). dvtm is not a project on suckless.org

Re: [hackers] [dwm][PATCH] config.mk: added build options for FreeBSD

2021-05-12 Thread Hiltjo Posthuma
On Tue, May 11, 2021 at 09:21:32PM -0400, Steve Ward wrote: > On Mon, May 10, 2021 at 5:31 AM Christos Margiolis > wrote: > > > > Hello Steve, > > > > On Sun, May 09, 2021 at 10:31:49PM -0400, Steve Ward wrote: > > > Do these commands give the desired output on FreeBSD? > > > `pkg-config --cflags

Re: [hackers] [st][PATCH] Restore cursor when exiting alt mode.

2021-05-11 Thread Hiltjo Posthuma
On Tue, May 11, 2021 at 10:41:29AM +0200, Stein Gunnar Bakkeby wrote: > From 7fa0a12281f4b595b75e40e497919076497281e8 Mon Sep 17 00:00:00 2001 > From: bakkeby > Date: Tue, 11 May 2021 10:20:23 +0200 > Subject: [PATCH] Restore cursor when exiting alt mode. > > If the mouse cursor is changed to a

Re: [hackers] [dwm][PATCH] config.mk: added build options for FreeBSD

2021-05-09 Thread Hiltjo Posthuma
On Sun, May 09, 2021 at 08:29:56PM +0300, Christos Margiolis wrote: > `config.mk' didn't have the appropriate `X11INC', `X11LIB', > `FREETYPEINC' options for dwm to build on FreeBSD, so I added > them. > > -- > Kind regards, > Christos > From afc59913952b5c38119bbcbf1aa17fe3498c8766 Mon Sep 17

Re: [hackers] [st][patch] Mild const-correctness improvements.

2021-05-06 Thread Hiltjo Posthuma
On Thu, May 06, 2021 at 04:33:35PM +0200, Markus F.X.J. Oberhumer wrote: > Hi Laslo, > > On 2021-05-06 16:19, Laslo Hunhold wrote: > > On Thu, 6 May 2021 16:11:33 +0200 > > "Markus F.X.J. Oberhumer" wrote: > > > > Dear Markus, > > > >> this is my first post to this list, so I hope I got the

Re: [hackers] [st][PATCH] fix pointer to one before object UB

2021-05-05 Thread Hiltjo Posthuma
On Wed, May 05, 2021 at 09:34:15PM +, Guilherme Janczak wrote: > The expression "s + strlen(s) - 1" can create a pointer to one before > *s if strlen(s) is 0. > > --- > util.c | 14 +- > 1 file changed, 9 insertions(+), 5 deletions(-) > > diff --git a/util.c b/util.c > index

Re: [hackers] [svkbd][PATCH] allow neutralizing a key modifier by applying the same modifier again

2021-04-18 Thread Hiltjo Posthuma
On Sun, Apr 18, 2021 at 04:26:05PM +0200, Max Schillinger wrote: > svkbd allows you to create keys for symbols of the second (=shift) layer by > defining them with a modifier included, like: > > { "|", "|", XK_backslash, 1, XK_Shift_L }, > > This key creates a pipe symbol by sending shift +

Re: [hackers] [dwm][PATCH] Support for different tags when there is a client inside

2021-04-13 Thread Hiltjo Posthuma
On Tue, Apr 13, 2021 at 01:10:20PM +0300, Yigit Colakoglu wrote: > This patch allows the user to provide a second set of tags which are > displayed when there is a client in that tag. > > --- > config.def.h | 5 - > dwm.c| 36 +--- > 2 files changed,

Re: [hackers] [tabbed][PATCH] Remove quotes around variables in Makefile

2021-04-02 Thread Hiltjo Posthuma
On Fri, Apr 02, 2021 at 02:05:05PM +0200, Laslo Hunhold wrote: > On Fri, 2 Apr 2021 11:03:05 +0200 > Hiltjo Posthuma wrote: > > Dear Hiltjo, > > > I prefer with quotes. You can still do make PREFIX=~/.local or > > whatever. Otherwise you could use $HOME. > >

Re: [hackers] [tabbed][PATCH] Remove quotes around variables in Makefile

2021-04-02 Thread Hiltjo Posthuma
On Thu, Apr 01, 2021 at 08:53:35PM -0400, Sebastian LaVine wrote: > Previously, if you had changed your PREFIX in config.mk to something > like ~/.local, then instead of installing to /home/user/.local, a > directory called '~' would be created in the project directory. Commands > are also

Re: [hackers] [st][PATCH] Set custom environment variables in config.h

2021-04-02 Thread Hiltjo Posthuma
On Fri, Apr 02, 2021 at 07:42:24AM +, Subhaditya Nath wrote: > From 79e69338725563e1bdba32e856726e8fa5151e4c Mon Sep 17 00:00:00 2001 > From: Subhaditya Nath > Date: Thu, 1 Apr 2021 19:42:51 +0530 > Subject: [PATCH] Set custom environment variables in config.h > > This patch enables setting

[hackers] Re: [dwm][PATCH] Do not allow focus to drift from fullscreen client via focusstack()

2021-03-29 Thread Hiltjo Posthuma
On Mon, Mar 29, 2021 at 11:55:55AM +, Chris Down wrote: > Hey Hiltjo, > > Any opinion on this one? Just going over my patches and remembered I was > waiting on a reply to this from upstream. > > Thanks :-) > > Chris Hi Chris, Thanks for the patch! I think it makes sense and pushed the

Re: [hackers] [svkbd] [merge request] various patches for svkbd

2021-03-28 Thread Hiltjo Posthuma
/proycon > > GnuPG key: 0x39FE11201A31555C > XMPP: proy...@anaproy.nl Matrix: @proycon:matrix.anaproy.nl > Telegram: proycon IRC: proycon (freenode) > Mastodon: https://social.anaproy.nl/@proycon (@proy...@social.anaproy.nl) > Twitter:https://twitter.

Re: [hackers] [PATCH] [dwm] [patch] livereload-xresources

2021-03-28 Thread Hiltjo Posthuma
Wiki patches should not be sent to this list, please read: https://suckless.org/community/ The wiki is public to push to. Thanks On Sun, Mar 28, 2021 at 01:34:53PM +0300, ser...@ekmekci.me wrote: > From: Serhan Ekmekçi > > --- > .../dwm-livereload-xresources-20210328.diff | 275

Re: [hackers] [svkbd] [merge request] various patches for svkbd

2021-03-27 Thread Hiltjo Posthuma
On Sat, Mar 27, 2021 at 02:03:05PM +0100, Maarten van Gompel wrote: > On 21-03-19 09:23, Maarten van Gompel wrote: > > > Thanks for the work. > > > > > > I've reviewed the patches. In general it looks good, but I've found a few > > > issues. Can you resend them to the mailinglist, then me (and

Re: [hackers] [svkbd] [merge request] various patches for svkbd

2021-03-18 Thread Hiltjo Posthuma
On Mon, Mar 15, 2021 at 05:46:32PM +0100, Maarten van Gompel wrote: > Hi, > > We have done some considerable work on svkbd in the scope of the sxmo > project and would like to again merge these changes upstream to > suckless, like before. Before I inundate your mailing list with a series > of 22

Re: [hackers] Re: [dwm][PATCH] Proper restart function

2021-03-09 Thread Hiltjo Posthuma
On Mon, Mar 08, 2021 at 05:12:50PM +, toluschr wrote: > On Monday, March 8, 2021 2:49 PM, Hiltjo Posthuma > wrote: > > > On Mon, Mar 08, 2021 at 11:29:12AM +, Tolu2 wrote: > > > > > On Saturday, March 6, 2021 4:59 PM toluschr wrote: > > > >

Re: [hackers] Re: [dwm][PATCH] Proper restart function

2021-03-08 Thread Hiltjo Posthuma
On Mon, Mar 08, 2021 at 11:29:12AM +, Tolu2 wrote: > On Saturday, March 6, 2021 4:59 PM toluschr wrote: > > > The currently available patch reads /proc/self/exe, which fails when > > recompiling (Why else would I want to restart dwm?) > > This patch simply saves the commandline arguments on

Re: [hackers] [svkbd][PATCH] Force installing executable

2021-02-25 Thread Hiltjo Posthuma
On Thu, Feb 25, 2021 at 05:28:16PM +0100, Jochen Sprickerhof wrote: > So we don't fail if it is executed at the same time. > --- > Makefile | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/Makefile b/Makefile > index 076c8b0..e82704e 100644 > --- a/Makefile > +++

Re: [hackers] [quark] Ignore queries and fragments in URIs || Laslo Hunhold

2021-01-30 Thread Hiltjo Posthuma
On Sat, Jan 30, 2021 at 01:11:17PM +0100, g...@suckless.org wrote: > commit 319ba7083fdde836d6614c6b8b228bf3a9849e95 > Author: Laslo Hunhold > AuthorDate: Sat Jan 30 12:53:00 2021 +0100 > Commit: Laslo Hunhold > CommitDate: Sat Jan 30 13:10:32 2021 +0100 > > Ignore queries and

Re: [hackers] [lchat][PATCH] Makefile: provides an uninstall target

2021-01-29 Thread Hiltjo Posthuma
On Thu, Jan 28, 2021 at 08:39:40PM +0100, shirenn wrote: > On Thu, 2021-01-28 at 17:56 +0100, Hiltjo Posthuma wrote: > > > > It would be nice if the Makefile respects $DESTDIR also (can be a separate > > patch). > > > > Curently the destination directories

Re: [hackers] [lchat][PATCH] Makefile: provides an uninstall target

2021-01-28 Thread Hiltjo Posthuma
On Thu, Jan 28, 2021 at 04:55:58PM +0100, shirenn wrote: > Signed-off-by: shirenn > --- > Makefile | 5 - > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/Makefile b/Makefile > index ea45200..675ac76 100644 > --- a/Makefile > +++ b/Makefile > @@ -1,6 +1,6 @@ > include

Re: [hackers] [quark] Use epoll/kqueue and worker threads to handle connections || Laslo Hunhold

2021-01-17 Thread Hiltjo Posthuma
On Sun, Nov 01, 2020 at 12:34:52AM +0100, g...@suckless.org wrote: > commit dff98c0bcaef7be220c563ebaebd66f8c6704197 > Author: Laslo Hunhold > AuthorDate: Sun Nov 1 00:27:46 2020 +0100 > Commit: Laslo Hunhold > CommitDate: Sun Nov 1 00:27:46 2020 +0100 > > Use epoll/kqueue and

Re: [hackers] [st][PATCH] Use git archive to create the dist tar file

2021-01-14 Thread Hiltjo Posthuma
On Wed, Jan 13, 2021 at 09:05:41PM -0500, Steve Ward wrote: > --- > Makefile | 7 +-- > 1 file changed, 1 insertion(+), 6 deletions(-) > > diff --git a/Makefile b/Makefile > index 470ac86..7d5ab41 100644 > --- a/Makefile > +++ b/Makefile > @@ -33,12 +33,7 @@ clean: > rm -f st $(OBJ)

Re: [hackers] [st][patch] Implements OSC 11/12 for setting foreground/background colours

2021-01-10 Thread Hiltjo Posthuma
On Sun, Jan 10, 2021 at 02:25:38PM -0500, Raheman Vaiya wrote: > The attached patch adds support for the OSC 11 and OSC 12 escape > sequences. These are used by many theme scripts (e.g > https://github.com/lemnos/theme.sh) and implemented by most other > terminal emulators. This is my first patch

Re: [hackers] [dwm][patch] Print Screen (screenshots) using maim

2021-01-04 Thread Hiltjo Posthuma
On Mon, Jan 04, 2021 at 03:17:32PM +, Hritik Vijay wrote: > From c196805f9fed6e5a62cb606a417605df5648f3ad Mon Sep 17 00:00:00 2001 > From: Hritik Vijay > Date: Mon, 4 Jan 2021 20:38:37 +0530 > Subject: [PATCH] Print Screen using maim > > This patch uses maim to take screenshots with some

Re: [hackers] [st][patch] Handle DECRQM (Request Mode) escape sequences

2020-11-30 Thread Hiltjo Posthuma
On Mon, Nov 30, 2020 at 12:56:13AM +0100, François-Xavier Carton wrote: > On Sun, Nov 29, 2020 at 05:37:30PM +0100, Hiltjo Posthuma wrote: > > > > Hi Francois, > > > > First: thanks for the patch. > > > > Do you have maybe a practical use-case example w

Re: [hackers] [st][patch] Handle DECRQM (Request Mode) escape sequences

2020-11-29 Thread Hiltjo Posthuma
On Fri, Nov 27, 2020 at 01:54:36AM +0100, François-Xavier Carton wrote: > Handle DECRQM (Request Mode) escape sequences > > The DECRQM [1] escape sequence is a request to the terminal to query the state > of a mode. The terminal replies to this request with a DECRPM [2] escape > sequence,

Re: [hackers] [st][patch] st-delkey patch

2020-11-13 Thread Hiltjo Posthuma
On Thu, Nov 12, 2020 at 10:58:36PM +0100, Clyme wrote: > The delkey patch appears to be broken on the latest commit of st (4ef0cbd). > This patch fixes that. > > --- config.def.h.orig 2020-11-12 20:23:48.867954750 +0100 > +++ config.def.h2020-11-12 20:21:15.055922720 +0100 > @@ -276,7

Re: [hackers] [dmenu][PATCH] dmenu_run: run "exec program" instead of "program" in shell

2020-11-10 Thread Hiltjo Posthuma
On Tue, Nov 10, 2020 at 11:09:24PM +0100, наб wrote: > This means that for long-running programs the process tree is > /init → program instead of /init → /bin/sh → program > --- > Please keep me in CC > > dmenu_run | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git

Re: [hackers] [quark] Thoughts on CGI and authentication?

2020-10-22 Thread Hiltjo Posthuma
On Thu, Oct 22, 2020 at 08:52:27PM +0200, José Miguel Sánchez García wrote: > Hello, > > I'm currently serving my static webpage with quark. I want to add some > CGI, but the opinion here about CGI support in the server is pretty > low. I'm trying to update the basecgi patch, but the latest

Re: [hackers] [surf][PATCH] Use sigaction instead of signal for installing signal handlers

2020-10-22 Thread Hiltjo Posthuma
On Thu, Oct 22, 2020 at 12:16:05AM -0500, Nihal Jere wrote: > sigaction provides a more portable way of installing signals > --- > surf.c | 17 + > 1 file changed, 13 insertions(+), 4 deletions(-) > > diff --git a/surf.c b/surf.c > index 2b54e3c..65a5e66 100644 > --- a/surf.c >

Re: [hackers] [st][PATCH] ST: Add WM_ICON_NAME property support

2020-10-18 Thread Hiltjo Posthuma
r both, were being set to the WM_NAME > Xwindows atom. This patch allows for either both or one, to follow the way > that xterm and other terminal emulators do, so that the shell or program > can set them seperately. > > Hopefully this is clearer?? > > Cheers, > John >

Re: [hackers] Licensing status of patches

2020-09-30 Thread Hiltjo Posthuma
On Wed, Sep 30, 2020 at 11:41:03AM +0200, Laslo Hunhold wrote: > On Wed, 30 Sep 2020 11:19:43 +0200 > Hiltjo Posthuma wrote: > > Dear Hiltjo, > > > I actively search for FOSS in my life and think using software which > > is GPL-licensed is fine. > > yeah, opini

Re: [hackers] Licensing status of patches

2020-09-30 Thread Hiltjo Posthuma
On Wed, Sep 30, 2020 at 08:32:52AM +0200, Laslo Hunhold wrote: > On Tue, 29 Sep 2020 20:01:41 +0100 > Daniel Littlewood wrote: > > Dear Daniel, > > > Thanks for your reply - I appreciate that this does not have much > > practical importance. Unfortunately the simplest way for me to version > >

Re: [hackers] [dwm] [patch] tatami layout

2020-09-24 Thread Hiltjo Posthuma
On Thu, Sep 24, 2020 at 02:00:47PM +0530, Sarthak Shah wrote: > This patch adds a tatami arrangement to dwm, accessible through modkey+y Hi, Please submit this to the wiki as it's not an upstream patch and do not submit non-upstream patches to hackers@. https://suckless.org/community/ The

Re: [hackers] [st][PATCH] ST: Add WM_ICON_NAME property support

2020-09-06 Thread Hiltjo Posthuma
On Sun, Sep 06, 2020 at 05:53:41PM +1200, John Collis wrote: > Also added _NET_WM_ICON_NAME. > --- > st.c | 9 + > win.h | 1 + > x.c | 16 +++- > 3 files changed, 25 insertions(+), 1 deletion(-) > > diff --git a/st.c b/st.c > index 8cf7245..ccef9c4 100644 > --- a/st.c >

Re: [hackers] [dmenu] New release with IME support backed out?

2020-09-02 Thread Hiltjo Posthuma
On Tue, Sep 01, 2020 at 04:49:49PM +0100, Chris Down wrote: > Hey folks, > Hi, > Unlike a lot of the rest of our software, generally dmenu works pretty well > even as a distribution-provided binary package, since usually users only > want to change colours, fonts, and other stuff that can be

Re: [hackers] [dwm][patch] status2d extra tags - swap/save/restore fg/bg colors

2020-08-26 Thread Hiltjo Posthuma
On Wed, Aug 26, 2020 at 08:27:18PM +, tdu wrote: > From dfd158c8dea87a0a4dbc5b2eda7c096069d1484a Mon Sep 17 00:00:00 2001 > From: tdu > Date: Wed, 26 Aug 2020 18:50:09 +0300 > Subject: [PATCH] Add the following tags for the status2d patch: ^w^ - > Swaps bg/fg color. ^v^ - Saves the

Re: [hackers] [quark] Reduce global state by localizing the server-struct || Laslo Hunhold

2020-08-17 Thread Hiltjo Posthuma
On Mon, Aug 17, 2020 at 03:47:27AM -0700, Jeremy wrote: > Then should the functions which use the global variable, "term", in st.c > accept a Term pointer instead? > > In the case of st, I believe this may affect readability slightly, > however, it would allow for some extensibility(if dvtm, for

Re: [hackers] [quark] Begin comment in lowercase || Laslo Hunhold

2020-08-05 Thread Hiltjo Posthuma
On Wed, Aug 05, 2020 at 07:14:08PM +0200, g...@suckless.org wrote: > commit 2318a89ecd7aad5a296b657caec22beff92a4284 > Author: Laslo Hunhold > AuthorDate: Wed Aug 5 19:14:10 2020 +0200 > Commit: Laslo Hunhold > CommitDate: Wed Aug 5 19:14:10 2020 +0200 > > Begin comment in lowercase

Re: [hackers] [PATCH v2 2/2] cleaned up code style, fixed buffer overflow in layers configuration, added a printdbg function.

2020-08-03 Thread Hiltjo Posthuma
On Sun, Aug 02, 2020 at 08:57:06PM +0200, Maarten van Gompel wrote: > --- > svkbd.c | 102 ++-- > 1 file changed, 55 insertions(+), 47 deletions(-) > > diff --git a/svkbd.c b/svkbd.c > index 1ff77f9..132a52d 100644 > --- a/svkbd.c > +++

Re: [hackers] [svkbd] Xft + Layers + Overlays + Mobile

2020-08-02 Thread Hiltjo Posthuma
On Sun, Aug 02, 2020 at 03:46:05PM +0200, Maarten van Gompel wrote: > This is v2 of a big patch for svkbd. The overall intention is to make > svkbd viable keyboard on a smartphone (the pinephone in particular). > Svkbd is the keyboard of choice in the SXMO project > (https://sr.ht/~mil/Sxmo/) by

Re: [hackers] [svkbd] Xft + Layers + Overlays + Mobile

2020-08-02 Thread Hiltjo Posthuma
t only for mobile. svkbd is not specificly for sxmo/mobile, but I'd like to support this project. - Theres still a buffer overflow in SVKBD_LAYERS. You could use strdup() there. - Code-style should be similar to st, comment style should be /*, TABs should be used consistently. */ Below

Re: [hackers] setsid: add optional -f to force fork()

2020-07-25 Thread Hiltjo Posthuma
On Tue, Jul 14, 2020 at 10:15:43AM +0200, Hiltjo Posthuma wrote: > Hi, > > The below patch adds an -f flag to force fork(2)ing and creating a new > process. > > > From a75ef384c11b64732dd6a3adc9249ba6beb8a67e Mon Sep 17 00:00:00 2001 > From: Hiltjo Posthuma > Date

Re: [hackers] [PATCH 5/6] Added a dialer/numpad keyboard, added the ability to handle layouts with less keys/different layouts. Extra configurability: select layout on startup, customisable height fac

2020-07-24 Thread Hiltjo Posthuma
On Fri, Jul 24, 2020 at 09:49:59PM +0200, Maarten van Gompel wrote: > --- > README.md | 87 > config.def.h | 1 + > layout.sxmo.h | 39 +- > svkbd.c | 107 +- > 4 files changed,

Re: [hackers] [PATCH 4/6] added an extra key column (alt now on keyboard by default), added cyrillic keymap, added a toggle to enable/disable overlays, added quick toggle buttons to toggle primary two

2020-07-24 Thread Hiltjo Posthuma
On Fri, Jul 24, 2020 at 09:49:58PM +0200, Maarten van Gompel wrote: > --- > README| 63 --- > layout.sxmo.h | 168 -- > svkbd.c | 165 - > 3 files changed, 297

Re: [hackers] [PATCH 1/6] Add custom layouts. Move layouts to single layout.h file & toggle layers via memcpy w/ XK_Cancel. Clean up secondary layout positioning LRDU properly & blanking placeholders.

2020-07-24 Thread Hiltjo Posthuma
On Fri, Jul 24, 2020 at 09:49:55PM +0200, Maarten van Gompel wrote: > From: Miles Alan > > Fix SIGTERM handler - flip terminate flag in sigterm handler & cleanup > properly > > Modify run function to use select() with a timeout since X events will be > blocked otherwise and terminate wouldn't

Re: [hackers] [quark][PATCH] fix for NOT_MODIFIED being sent in non-GMT timezone

2020-07-21 Thread Hiltjo Posthuma
On Tue, Jul 21, 2020 at 07:51:56PM +0200, Laslo Hunhold wrote: > On Sun, 19 Jul 2020 09:37:31 -0700 > Jeremy wrote: > > Dear Jeremy, > > I wrote this this mail yesterday, but the mailing list had some > technical problems, so I'm resending it. > > > stat(3)'s mtime is in local time while

Re: [hackers] [quark][PATCH] fix for NOT_MODIFIED being sent in non-GMT timezone

2020-07-19 Thread Hiltjo Posthuma
On Sun, Jul 19, 2020 at 09:37:31AM -0700, Jeremy wrote: > From: Jeremy Bobbin > > stat(3)'s mtime is in local time while REQ_MOD is in GMT. > This patch translates mtime to GMT before comparing to REQ_MOD. > --- > http.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git

Re: [hackers] [slstatus][PATCH] Handle SIGUSR1 for forced refreshes

2020-07-14 Thread Hiltjo Posthuma
On Tue, Jul 14, 2020 at 02:45:14PM +0200, Mart Lubbers wrote: > Dear all, > I've submitted this patch before but maybe I did something wrong (it was > before > the confirmation mail of the subscription). If there is anything else off with > this email/patch, please let me know. > Best, > >

Re: [hackers] setsid: add optional -f to force fork()

2020-07-14 Thread Hiltjo Posthuma
ls/setsid.c#n96 > Regards, > Mattias Andrée > > > On Tue, 14 Jul 2020 10:15:43 +0200 > Hiltjo Posthuma wrote: > > > Hi, > > > > The below patch adds an -f flag to force fork(2)ing and creating a new > > process. > > > > > > F

[hackers] setsid: add optional -f to force fork()

2020-07-14 Thread Hiltjo Posthuma
Hi, The below patch adds an -f flag to force fork(2)ing and creating a new process. >From a75ef384c11b64732dd6a3adc9249ba6beb8a67e Mon Sep 17 00:00:00 2001 From: Hiltjo Posthuma Date: Tue, 14 Jul 2020 10:11:43 +0200 Subject: [PATCH] setsid: add optional -f to force fork() --- setsid.1

Re: [hackers] [ii][PATCH] ewritestr: Remove duplicate shift of offset

2020-07-05 Thread Hiltjo Posthuma
On Sun, Jul 05, 2020 at 07:39:19PM +, miz...@protonmail.com wrote: > --- > ii.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/ii.c b/ii.c > index 7ca3ee8..426fcff 100644 > --- a/ii.c > +++ b/ii.c > @@ -102,7 +102,6 @@ ewritestr(int fd, const char *s) > for (off = 0; off <

Re: [hackers] [st][PATCH] x.c: modify xsettitle to work with musl libc

2020-07-03 Thread Hiltjo Posthuma
On Wed, Jul 01, 2020 at 09:29:25PM +0200, joris.k...@gmail.com wrote: > From: Joris Klaasse Bos > > Before this commit, running st on a system with musl libc would > result in a segmentation fault. This is because musl implements > locales differently, which means the function >

Re: [hackers] [dwm][PATCH] Fix memory leaks in drw

2020-06-11 Thread Hiltjo Posthuma
On Thu, Jun 11, 2020 at 03:28:32PM +0200, Alex Flierl wrote: > The function drw_fontset_free in drw.c was never called. > --- > drw.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drw.c b/drw.c > index 8fd1ca4..4cdbcbe 100644 > --- a/drw.c > +++ b/drw.c > @@ -95,6 +95,7 @@

Re: [hackers] [dwm][patch] Make EWMH windows float

2020-06-11 Thread Hiltjo Posthuma
On Sun, Jun 07, 2020 at 08:36:57PM +0200, Jakub Leszczak wrote: > Hi, > > What is the status of this patch? I do not see it being added to the > mainline, nor any new messages for some time. > > Br, > Jakub Leszczak > Hi Jakub, Can you send the last version one more time? The last sent

Re: [hackers] [tabbed][PATCH] Fix clients BadDrawable at cleanup with multiple tabs

2020-06-10 Thread Hiltjo Posthuma
On Thu, Jun 04, 2020 at 04:01:17PM +0500, Nikita Zlobin wrote: > unmanage() after killclient() causes at least one BadDrawable from client. > For urxvtd it can crash entire daemon. > --- > tabbed.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tabbed.c b/tabbed.c >

Re: [hackers] [tabbed][PATCH] Use utf8 for window name

2020-06-10 Thread Hiltjo Posthuma
On Thu, Jun 04, 2020 at 02:13:49PM +0500, Nikita Zlobin wrote: > At least some desktop tools (non-mainstream) are in trouble with tabbed > window name. > At the moment of try I have only tint2 and rofi. First just showes intitled, > but > second - "Invalid encoding". Though xfwm4 has no problem

<    1   2   3   4   5   >