Re: [dev] [PATCH] Reorder-and-extend-glyph-attributes

2014-07-31 Thread FRIGN
On Thu, 31 Jul 2014 21:29:43 +0200 "Roberto E. Vargas Caballero" wrote: > I just patched it, but maybe could be a good idea define this issue > and follow a common criterion. Good idea. I'll do it next time. -- FRIGN

Re: [dev] [PATCH] Reorder-and-extend-glyph-attributes

2014-07-31 Thread Roberto E. Vargas Caballero
> Just push the patch, there's no need for me to sign it. Signing off is > only meant for the original authors (read about the SCO lawsuit[0]). The real meaning of signed-off depends of each project. In st, we are uing it as a way to indicate who verify the patch was correct. Thre is other tags th

Re: [dev] [PATCH] Reorder-and-extend-glyph-attributes

2014-07-31 Thread FRIGN
On Thu, 31 Jul 2014 20:17:34 +0200 "Roberto E. Vargas Caballero" wrote: > And it was for FRIGN, since the discussion was between him and me. Sorry > for not indicate the name. Just push the patch, there's no need for me to sign it. Signing off is only meant for the original authors (read about t

Re: [dev] [PATCH] Reorder-and-extend-glyph-attributes

2014-07-31 Thread Roberto E. Vargas Caballero
> I was talking about a git sign, something like: > > Signed-off-by: Roberto E. Vargas Caballero > > that it is included with 'git commit --amend -s'. Basically it was > a proposal of peer review, in the sense you tested it and you agree ;). And it was for FRIGN, since the discussion was

Re: [dev] [PATCH] Reorder-and-extend-glyph-attributes

2014-07-31 Thread Roberto E. Vargas Caballero
> seems legit, however I will not cryptographically sign it, since it is not my > work. ;) I was talking about a git sign, something like: Signed-off-by: Roberto E. Vargas Caballero that it is included with 'git commit --amend -s'. Basically it was a proposal of peer review, in the sens

Re: [dev] [PATCH] Reorder-and-extend-glyph-attributes

2014-07-31 Thread FRIGN
On Thu, 31 Jul 2014 19:45:47 +0200 "Roberto E. Vargas Caballero" wrote: > Ok, let's go. If you agree, sign the patch that I attach to this mail and then > I will apply it. Yeah, push it. It's good! -- FRIGN

Re: [dev] [PATCH] Reorder-and-extend-glyph-attributes

2014-07-31 Thread Markus Teich
Roberto E. Vargas Caballero wrote: > Ok, let's go. If you agree, sign the patch that I attach to this mail and then > I will apply it. Heyho Roberto, seems legit, however I will not cryptographically sign it, since it is not my work. ;) --Markus

Re: [dev] [PATCH] Reorder-and-extend-glyph-attributes

2014-07-31 Thread Roberto E. Vargas Caballero
> > > There is no such kind of user! Realize it! It's the wrong way to go > > > > Hahaha ;). I would like to see the opinion of another suckless developers. > > What do you think guys?. > > Heyho, > > at least there should be no such user who is unable to figure the fallthrough > thing out… Be a

Re: [dev] Introducing the imagefile-format

2014-07-31 Thread FRIGN
On Thu, 31 Jul 2014 18:14:36 +0100 Dimitris Papastamos wrote: > We also now have jpg2if[0] and gif2if[1] in the if-tools repo. > > [0] http://git.2f30.org/imagefile/tree/jpg2if.c > [1] http://git.2f30.org/imagefile/tree/gif2if.c Thanks man! It's always better having a native solution than havin

Re: [dev] Introducing the imagefile-format

2014-07-31 Thread Dimitris Papastamos
We also now have jpg2if[0] and gif2if[1] in the if-tools repo. [0] http://git.2f30.org/imagefile/tree/jpg2if.c [1] http://git.2f30.org/imagefile/tree/gif2if.c

Re: [dev] unit testing

2014-07-31 Thread Louis Santillan
I extracted kazuho's TAP code from his picogc repo [0] (example [1]). Not 3 lines but it can be run and integrate with any TAP parser [2]. [0] https://github.com/kazuho/picogc/blob/master/t/test.h [1] https://github.com/kazuho/picogc/blob/master/t/stack.cpp [2] http://en.wikipedia.org/wiki/Tes

Re: [dev] [PATCH] Reorder-and-extend-glyph-attributes

2014-07-31 Thread Markus Teich
Roberto E. Vargas Caballero wrote: > > Please don't bring this argument here. You are talking about an > > imaginary user who is keen on patching st for a faster blinking mode. > > > > There is no such kind of user! Realize it! It's the wrong way to go > > Hahaha ;). I would like to see the opini

Re: [dev] [PATCH] Reorder-and-extend-glyph-attributes

2014-07-31 Thread Roberto E. Vargas Caballero
> Please don't bring this argument here. You are talking about an > imaginary user who is keen on patching st for a faster blinking mode. > > There is no such kind of user! Realize it! It's the wrong way to go Hahaha ;). I would like to see the opinion of another suckless developers. What do you

Re: [dev] [PATCH] Reorder-and-extend-glyph-attributes

2014-07-31 Thread stanio
you're doing great job with st, guys, keep up! didn't want to distract you from the discussion but couldn't resist 20h's briliant irony. --s

Re: [dev] [PATCH] Reorder-and-extend-glyph-attributes

2014-07-31 Thread FRIGN
On Thu, 31 Jul 2014 14:29:09 +0200 "Roberto E. Vargas Caballero" wrote: > Well, I think is good idea have two flags instead of only one. > If someone really needs two blink modes, he must be able to patch it > easily, and do the or of both flags is not complex. Please don't bring this argument h

Re: [dev] [PATCH] Reorder-and-extend-glyph-attributes

2014-07-31 Thread FRIGN
On Thu, 31 Jul 2014 14:09:23 +0200 sta...@cs.tu-berlin.de wrote: > then he clearly needs at least two blinking modes: one for each. what > about hotmail? Haha, get lost :P As Roberto presented in his previous response, the terminal-implementations are way too different from each other anyway. It

Re: [dev] [PATCH] Reorder-and-extend-glyph-attributes

2014-07-31 Thread Roberto E. Vargas Caballero
> Exactly! Keep it the way it is and just fall through in the switch. > It's 100 times simpler and it keeps the code lean. Well, I think is good idea have two flags instead of only one. If someone really needs two blink modes, he must be able to patch it easily, and do the or of both flags is not

Re: [dev] [PATCH] Reorder-and-extend-glyph-attributes

2014-07-31 Thread stanio
* Markus Teich 2014-07-31 12:39 > at suckless conf last year 20h mentioned, he originally implemented blinking > support to warn him about outlook and gmail users in mutt. epic! then he clearly needs at least two blinking modes: one for each. what about hotmail?

Re: [dev] [PATCH] Reorder-and-extend-glyph-attributes

2014-07-31 Thread FRIGN
On Thu, 31 Jul 2014 13:50:18 +0200 "Roberto E. Vargas Caballero" wrote: > I was talking about the normal blinking. Sure that! No one needs two modes. ;) > It seems that almost of the developer agree that we only need one > blinking. The thing we have to do now is blink in both cases (with > the

Re: [dev] [PATCH] Reorder-and-extend-glyph-attributes

2014-07-31 Thread Roberto E. Vargas Caballero
> > I use it in mutt to show the mails that are in delete state. > > Do you need two blinking modes for that? I was talking about the normal blinking. > Seems like the topic has closed itself. It seems that almost of the developer agree that we only need one blinking. The thing we have to do no

Re: [dev] unit testing

2014-07-31 Thread Dimitris Papastamos
On Thu, Jul 31, 2014 at 10:57:23AM +0200, Markus Teich wrote: > Heyho, > > I just stumbled upon MinUnit[0], a 3-line unit test framework for C. Do you > know > other similarly simple solutions? Another alternative is to use some other programming language to prototype your algorithm/data structu

Re: [dev] unit testing

2014-07-31 Thread Dimitris Papastamos
On Thu, Jul 31, 2014 at 10:57:23AM +0200, Markus Teich wrote: > Heyho, > > I just stumbled upon MinUnit[0], a 3-line unit test framework for C. Do you > know > other similarly simple solutions? For anything non-trivial I generally test it separately, I simply copy the function into another .c an

Re: [dev] [PATCH] Reorder-and-extend-glyph-attributes

2014-07-31 Thread FRIGN
On Thu, 31 Jul 2014 12:54:15 +0200 "Roberto E. Vargas Caballero" wrote: > I use it in mutt to show the mails that are in delete state. Do you need two blinking modes for that? Seems like the topic has closed itself. -- FRIGN

Re: [dev] [PATCH] Reorder-and-extend-glyph-attributes

2014-07-31 Thread Roberto E. Vargas Caballero
> at suckless conf last year 20h mentioned, he originally implemented blinking > support to warn him about outlook and gmail users in mutt. I don't know of any > better use case for blinking. I use it in mutt to show the mails that are in delete state. -- Roberto E. Vargas Caballero

Re: [dev] [PATCH] Reorder-and-extend-glyph-attributes

2014-07-31 Thread FRIGN
On Thu, 31 Jul 2014 12:41:17 +0200 Markus Teich wrote: > at suckless conf last year 20h mentioned, he originally implemented blinking > support to warn him about outlook and gmail users in mutt. I don't know of any > better use case for blinking. I like his sense of humour. -- FRIGN

Re: [dev] [PATCH] Reorder-and-extend-glyph-attributes

2014-07-31 Thread Markus Teich
FRIGN wrote: > I honestly have to say that I don't favor the fast-blinking-patch at > all. How often do you stumble upon blinking-tags anyway? Heyho, at suckless conf last year 20h mentioned, he originally implemented blinking support to warn him about outlook and gmail users in mutt. I don't kno

Re: [dev] [PATCH] Reorder-and-extend-glyph-attributes

2014-07-31 Thread FRIGN
On Thu, 31 Jul 2014 12:10:43 +0200 "Roberto E. Vargas Caballero" wrote: > Yeah, it is true. I will apply it only if it fits well in the new > main-loop. I have modified the original patch and now it may be applied > to HEAD. The patch itself is not bad, and the complexity it pays for two > blinks

Re: [dev] [PATCH] Reorder-and-extend-glyph-attributes

2014-07-31 Thread Roberto E. Vargas Caballero
> I honestly have to say that I don't favor the fast-blinking-patch at > all. How often do you stumble upon blinking-tags anyway? Even if there > is a broad application of it in some program I don't know about, I'm > absolutely sure that the dependency on two blinking speeds is just > ridiculuous!

Re: [dev] [PATCH] Reorder-and-extend-glyph-attributes

2014-07-31 Thread FRIGN
On Thu, 31 Jul 2014 11:32:11 +0200 "Roberto E. Vargas Caballero" wrote: > The case 26 is a bit different, because we implemented fast blinking > with normal blinking, and he sent a patch to have a correct fast > blinking. I have not applied it yet, but I think I am going to apply it. I honestly

Re: [dev] [PATCH] Reorder-and-extend-glyph-attributes

2014-07-31 Thread Roberto E. Vargas Caballero
> > I think you can find information about them in the same reference that > > Michael Forney posted. > > Yeah, I just wanted to know if he had another document with different > behaviors, I’m particularly curious about his cases 21 and 26. The case 21 was in the code before the patches of Anders

Re: [dev] unit testing

2014-07-31 Thread Roberto E. Vargas Caballero
Hi, > I just stumbled upon MinUnit[0], a 3-line unit test framework for C. Do you > know > other similarly simple solutions? I wrote ago a simple framework for it long time: test.h: #ifndef UTEST_H_ #define UTEST_H_ #ifdef NDEBUG #undef NDEBUG #endif

[dev] unit testing

2014-07-31 Thread Markus Teich
Heyho, I just stumbled upon MinUnit[0], a 3-line unit test framework for C. Do you know other similarly simple solutions? --Markus (This is a repost since the original mail from yesterday seems to have gotten swallowed…) [0] http://www.jera.com/techinfo/jtns/jtn002.html