2016-05-22 18:00 GMT-03:00 Lucas Vuotto <l.vuott...@gmail.com>:

> Hello FRIGN,
>
> the patch is for ubase, not sbase, which, by definitio, is ugly :D
>
> Anyway, what you say may still hold true. The patch could be smaller if we
> only deal with clear, init and reset parameters, as stated by POSIX. This
> come by my mind, but after asking in #suckless, just got an answer from ??,
> telling my to go full ncurses tput (which reminds me, it
>
?? is Christoph Lohmann

> only deals with string capabilities now -- will fix if it gets merged).
>
> As for the line count, I don't think it's possibly to make any library
> dealing with terminfo smaller by a significant amount (say, 10% less
> lines). Take in account that the library must provide a parser for a
> context-free grammar.
>
> Aside, maybe tput is too much for ubase. It's listed in the TODO, but I
> don't know if it's really needed.
>
> Concluding, I sent the patch so you can judge if it's worthless to merge
> or no. It wasn't wasted time, because it helped me to find bugs in my
> library :D
>
> Cheers.
>
>
> -- lv.
>
>
>
> 2016-05-22 16:08 GMT-03:00 FRIGN <d...@frign.de>:
> > On Sun, 22 May 2016 16:01:43 -0300
> > Lucas Gabriel Vuotto <l.vuott...@gmail.com> wrote:
> >
> > Hey Lucas,
> >
> >> attached you will find two patches, one implementing a basic library
> for dealing
> >> with terminfo capabilities and another implementing the tput command
> using that
> >> library. The library itself is part of a bigger library from me, damned
> [0], for
> >> TUI creation.
> >
> >>  9 files changed, 2179 insertions(+), 3 deletions(-)
> >
> > :O
> >
> > definitely too much man.
> >
> > If we look at sbase, it has the following SLOC-count:
> >
> > Totals grouped by language (dominant language first):
> > ansic:        18805 (97.57%)
> > awk:            255 (1.32%)
> > sh:             213 (1.11%)
> >
> > do you really think it's a good idea to increase its size
> > by roughly 10% to accomodate one single command?
> >
> > Cheers
> >
> > FRIGN
> >
> > --
> > FRIGN <d...@frign.de>
> >
>
>

Reply via email to