Re: [dev] [st] Blank lines not preserved

2014-05-26 Thread Alexander S.
2014-05-26 19:36 GMT+04:00 FRIGN : > On Mon, 26 May 2014 18:42:50 +0400 > "Alexander S." wrote: > >> Well, for low colours, you did? Correct me if I'm wrong. > > Read this first: http://lists.suckless.org/dev/1405/21569.html > > The point is: We can't mess with the high colors (i.e. > 256-colour-s

Re: [dev] [st] Blank lines not preserved

2014-05-26 Thread FRIGN
On Mon, 26 May 2014 18:42:50 +0400 "Alexander S." wrote: > Well, for low colours, you did? Correct me if I'm wrong. Read this first: http://lists.suckless.org/dev/1405/21569.html The point is: We can't mess with the high colors (i.e. 256-colour-space/grayscale), especially not when our gamma-sc

Re: [dev] [st] Blank lines not preserved

2014-05-26 Thread Alexander S.
2014-05-26 17:25 GMT+04:00 FRIGN : > On Mon, 26 May 2014 09:00:01 +0400 > "Alexander S." wrote: > >> Guys, stop derailing the thread, please. If the issue concerns you, >> answer in the original thread to a solution I proposed. > > http://git.suckless.org/st/commit/?id=3544e354b2dcdfcced1a2f4aeedb

Re: [dev] [st] [PATCH] Refactor xsetcolorname()

2014-05-26 Thread FRIGN
On Mon, 26 May 2014 08:56:58 +0400 "Alexander S." wrote: > Your concern about repetitiveness is valid and bothered me when I > wrote that function, although back then xloadcols() was in much poorer > state. By now, we may as well reimplement xloadcols() in terms of > xsetcolorname(i, NULL). I'd b

Re: [dev] [st] Blank lines not preserved

2014-05-26 Thread FRIGN
On Mon, 26 May 2014 09:00:01 +0400 "Alexander S." wrote: > Guys, stop derailing the thread, please. If the issue concerns you, > answer in the original thread to a solution I proposed. http://git.suckless.org/st/commit/?id=3544e354b2dcdfcced1a2f4aeedb4d479abd543c I already fixed this issue. So

Re: [dev] [st] [PATCH] Refactor xsetcolorname()

2014-05-26 Thread FRIGN
On Mon, 26 May 2014 08:56:58 +0400 "Alexander S." wrote: Hey Alexander, thanks for your feedback! > this function implements STR command for changing a palette. Empty > name specifically tells to revert to system color, thus, if(!name) > clause is there for restoring a color that may have been