Re: [dev] sple - A simple PDF links/emails extracotr.

2015-05-06 Thread Jason Woofenden
Hi Hypsurus, I hope you're having fun coding. Don't let me detract from that. But if you just need to extract links from pdfs, you can do so with existing tools, eg: pdftohtml -stdout foo.pdf | sed -ne 's/\(^\|\n\)\n\([^\n]*\)\n[^\n]*/\1\2/gp; t; s/href="\([^"]\+\)"/\n\n\1\n/g; D' Sorry if that

Re: [dev] [vis] [PATCH] Add '--' as end of options.

2015-05-06 Thread Matias Linares
I attached the patch with the corrections on vis.c and the new info on the man page. Sorry for the previous, bad tested, patch. Greetings. -- XMPP/Mail: matiasl...@openmailbox.org GPG: http://www.famaf.unc.edu.ar/~mlm0111/pubkey.asc | BBD8 CADC C236 AD16 From 20b251fe256171252193caabab0dff4b100

[dev] [dwm] [PATCH] support _NET_SUPPORTING_WM_CHECK

2015-05-06 Thread Jason Woofenden
As documented here: http://standards.freedesktop.org/wm-spec/wm-spec-1.3.html#idm140130317670464 --- I developed this in the hopes that it would fix issues with feh running fullscreen. It didn't help with that (feh using the innacurate _MOTIF_WM_HINTS method) but it should at least stop gtk+ 3.3.

Re: [dev] [vis][RFC][PATCH 2/2] Implement the execution of external commands and cmd_substitute

2015-05-06 Thread Marc André Tanner
Thanks, this will require more time to properly review than I currently have. What follows are therefore only a few general remarks. > The code for the read/write loops communicating through a pipe with the > external process is very ugly. There must be a better way to implement > the pipe communi

Re: [dev] [vis] [PATCH] Add '--' as end of options.

2015-05-06 Thread Matias Linares
> Thanks for the patch, however it doesn't do what it describes? > > $ vis -- -v > > Still prints the version instead of opening a file called "-v". Yeah, I just tried opening files. Sorry for this :/ > Also it would be nice to add a note to the man page. I will redo the patch and I'll add th

Re: [dev] [vis][RFC][PATCH 1/2] Replace first '/' of substitute command with \0

2015-05-06 Thread Marc André Tanner
On Tue, May 05, 2015 at 06:25:29PM +0200, Silvan Jegen wrote: > Replacing the first '/' allows the 's' command name to be correctly > identified even though it is part of its own argument. Special casing '/' seems wrong, an arbitrary delimiter can be used :s,foo,bar is perfectly valid. The end

Re: [dev] [vis][PATCH] Use the command name without the Filerange in argv

2015-05-06 Thread Marc André Tanner
Thanks, will apply. -- Marc André Tanner >< http://www.brain-dump.org/ >< GPG key: CF7D56C0

Re: [dev] [vis][PATCH] Make the '.' Filerange work on one line

2015-05-06 Thread Marc André Tanner
When an incomplete range is specified i.e. only a start position the cursor should jump to it. This patch breaks this functionality. For example :nn (absolute line), :+n (relative line), :'m mark (move to mark) is broken. You probably want something ugly like the following? On Tue, May 05, 2015 a

Re: [dev] [vis] [PATCH] Add '--' as end of options.

2015-05-06 Thread Marc André Tanner
> Some programs (like ranger) expect the editor to have this feature. Thanks for the patch, however it doesn't do what it describes? $ vis -- -v Still prints the version instead of opening a file called "-v". Also it would be nice to add a note to the man page. -- Marc André Tanner >< http:

Re: [dev] Odds and ends (2/2): $XEMBED

2015-05-06 Thread Connor Lane Smith
On 6 May 2015 at 15:28, Jason Woofenden wrote: > If you don't replace the terminal window, then you're just talking > about window management, which should perhaps be done with the > window manager. The XEMBED thing only works on a few clients. The > WM can meddle with just about all the clients.

Re: [dev] Odds and ends (2/2): $XEMBED

2015-05-06 Thread Connor Lane Smith
On 6 May 2015 at 11:59, Connor Lane Smith wrote: > With that in mind, might 9embed (perhaps under another name) be > suitable to add to the tabbed distribution? Having had Christoph's approval, I've attached a patch which adds the utility (under the name of xembed) to tabbed. Thanks, cls 0001-

Re: [dev] [slock] [PATCH] Option to not show failure color on clear

2015-05-06 Thread FRIGN
On Wed, 6 May 2015 18:27:01 +0200 Markus Teich wrote: > What were your thoughts, sin and dsp? I like the new behaviour, namely making the state dirty as soon as a key has been pressed or a wrong password entered. It's a big contribution to security and you often wonder if somebody tried to acce

Re: [dev] [slock] [PATCH] Option to not show failure color on clear

2015-05-06 Thread Markus Teich
Nick Currier wrote: > On Wed, May 6, 2015 at 9:20 AM, Markus Teich > wrote: > > I think the new behaviour actually is a more reasonable default. > > I had talked to both sin and dsp about this and they thought it best to keep > the existing behaviour. However, it's easy enough to change if the c

Re: [dev] [slock] [PATCH] Option to not show failure color on clear

2015-05-06 Thread Nick Currier
On Wed, May 6, 2015 at 9:20 AM, Markus Teich wrote: > I think the new behaviour actually is a more reasonable default. I had talked to both sin and dsp about this and they thought it best to keep the existing behaviour. However, it's easy enough to change if the consensus is to change the default

Re: [dev] [vis][RFC][PATCH 1/2] Replace first '/' of substitute command with \0

2015-05-06 Thread Silvan Jegen
On Wed, May 06, 2015 at 09:00:17AM -0700, Eric Pruitt wrote: > On Wed, May 06, 2015 at 05:12:25PM +0200, Silvan Jegen wrote: > > On Wed, May 06, 2015 at 07:24:30AM -0700, Eric Pruitt wrote: > > > On Wed, May 06, 2015 at 10:10:21AM +0200, Silvan Jegen wrote: > > > > I do it to indicate that I agree

Re: [dev] [vis][RFC][PATCH 1/2] Replace first '/' of substitute command with \0

2015-05-06 Thread Eric Pruitt
On Wed, May 06, 2015 at 05:12:25PM +0200, Silvan Jegen wrote: > On Wed, May 06, 2015 at 07:24:30AM -0700, Eric Pruitt wrote: > > On Wed, May 06, 2015 at 10:10:21AM +0200, Silvan Jegen wrote: > > > I do it to indicate that I agree with the licensing terms of the > > > existing code and to make expli

Re: [dev] [ST] [PATCH] Clean up xdraws and optimize glyph drawing with non-unit kerning values

2015-05-06 Thread suigin
Here's a third version of the patch with some minor changes. It removes the curly braces around the body of the `if' clause in `xmakeglyphfontspecs' that checks for ATTR_WDUMMY. It simplifies some of the conditional logic in `xdrawglyphfontspecs', by pulling out the check to see if `base.fg' is the

Re: [dev] [vis][RFC][PATCH 1/2] Replace first '/' of substitute command with \0

2015-05-06 Thread Nick
Quoth Silvan Jegen: > What I am doing with this tag is expressing my good intentions. Good intentions are assumed for all the patches here, which is reasonable and right. If some lawsuit came about (which it won't) the norms of behaviour on the list should trump any git tags.

Re: [dev] [vis][RFC][PATCH 1/2] Replace first '/' of substitute command with \0

2015-05-06 Thread Silvan Jegen
On Wed, May 06, 2015 at 07:24:30AM -0700, Eric Pruitt wrote: > On Wed, May 06, 2015 at 10:10:21AM +0200, Silvan Jegen wrote: > > I do it to indicate that I agree with the licensing terms of the > > existing code and to make explicit that I want to let you use this > > code (that to my best knowledg

Re: [dev] [slock] [PATCH] Option to not show failure color on clear

2015-05-06 Thread Markus Teich
Heyho, Nick Currier wrote: > This patch adds the failonclear boolean option so that, when false, > the failure color will not appear until a failed login attempt has > been made. > It also maintains the existing behaviour (failure on clear) by default. I think the new behaviour actually is a more

Re: [dev] Odds and ends (2/2): $XEMBED

2015-05-06 Thread Raphaël Proust
On 6 May 2015 at 15:28, Jason Woofenden wrote: > This would be a little cleaner if you set your WM to stack new > windows just after the focused one. Assuming your window is stacked and not floating… More seriously, the behaviour 9embed provides is to have a window content being swapped whilst i

Re: [dev] Odds and ends (2/2): $XEMBED

2015-05-06 Thread Jason Woofenden
> In fact, leaving that for tabbed to do could well be enough. If you > run surf so it opens in a new tab then you can switch back to the > terminal if you wish, which couldn't be done in Plan 9, but seems > reasonably useful. If you don't replace the terminal window, then you're just talking abou

Re: [dev] [vis][RFC][PATCH 1/2] Replace first '/' of substitute command with \0

2015-05-06 Thread Eric Pruitt
On Wed, May 06, 2015 at 10:10:21AM +0200, Silvan Jegen wrote: > I do it to indicate that I agree with the licensing terms of the > existing code and to make explicit that I want to let you use this > code (that to my best knowledge is of my own devising, does not > infringe any patents yaddayaddaya

[dev] [slock] [PATCH] Option to not show failure color on clear

2015-05-06 Thread Nick Currier
Hello, This patch adds the failonclear boolean option so that, when false, the failure color will not appear until a failed login attempt has been made. It also maintains the existing behaviour (failure on clear) by default. Thanks, hexid From b5fa0f2cabf03ae3d09a29b8fb5327109ec2e0e0 Mon Sep 17 0

Re: [dev] Odds and ends (2/2): $XEMBED

2015-05-06 Thread Connor Lane Smith
On 6 May 2015 at 11:43, Connor Lane Smith wrote: > I've not found time to add the functionality into a terminal like st > yet, but I was happy to discover that tabbed already sets XEMBED, and > the wrapper works as expected: 'surf &' will open surf in a new > window, whereas 'surf' will open it in

Re: [dev] Odds and ends (2/2): $XEMBED

2015-05-06 Thread Connor Lane Smith
On 4 May 2015 at 11:56, Roberto E. Vargas Caballero wrote: > The idea is good, but it means that you have to modify all the programs > to accept this new feature. If you want to experiment with this idea > you can try to add this variable in st and read it in surf. I've written a wrapper which ma

Re: [dev] [vis][RFC][PATCH 1/2] Replace first '/' of substitute command with \0

2015-05-06 Thread Silvan Jegen
Heyho On Wed, May 6, 2015 at 3:50 AM, Eric Pruitt wrote: > On Tue, May 05, 2015 at 06:25:29PM +0200, Silvan Jegen wrote: >> Signed-off-by: Silvan Jegen > > What's the point in signing off on patches where you're the sole author? > Seems like redundant clutter to me. I do it to indicate that I a