Re: [dev][sbase] seq possible bug

2013-06-20 Thread Galos, David 
> Printing the format string in validfmt() shows that it is correct.
> Printing after 'fmt = ARGF();' in line 33 shows clipped format. I have
> no idea how ARGF works and cannot troubleshoot past that. Thanks.

Indeed, to quote the Plan9 manpage: "ARGF must be called
just once for each option argument. " I will submit a patch making use of
a temporary variable.

Re: [dev][sbase] seq possible bug

2013-06-20 Thread Galos, David 
> Indeed, to quote the Plan9 manpage: "ARGF must be called
> just once for each option argument. " I will submit a patch making use of
> a temporary variable.

In addition to adding and using a temporary variable to fix the
ARGF() problem, I also shrunk the switch on argc to be shorter
and more concise.


sbase-fix-seq.diff
Description: Binary data

[dev] [sbase] [patch v3] Add md5sum

2013-06-20 Thread stateless 
Added LICENSE.lpl as well and updated the license and copyright details for
the md5 code.

Thanks,
stateless


0001-Add-md5sum.patch
Description: Binary data

[dev] [st] [patch] Add support for size property

2013-06-20 Thread Mariano Bono 
I've written a little patch for st that add the support for font size
property, so it's possibile to pass font in the form:

Monospace-10
or
Monospace:size=10

I don't know Xlib nor Fontconfig so i hope it's all correct.

-- 
Mariano
diff --git a/st.c b/st.c
index 2811876..582889f 100644
--- a/st.c
+++ b/st.c
@@ -2715,6 +2715,7 @@ xloadfonts(char *fontstr, int fontsize) {
FcPattern *pattern;
FcResult result;
double fontval;
+   double DPI;
 
if(fontstr[0] == '-') {
pattern = XftXlfdParse(fontstr, False, False);
@@ -2734,12 +2735,21 @@ xloadfonts(char *fontstr, int fontsize) {
if(result == FcResultMatch) {
usedfontsize = (int)fontval;
} else {
-   /*
-* Default font size is 12, if none given. This is to
-* have a known usedfontsize value.
-*/
-   FcPatternAddDouble(pattern, FC_PIXEL_SIZE, 12);
-   usedfontsize = 12;
+   /* Try to get size in Pt */
+   result = FcPatternGetDouble(pattern, FC_SIZE, 0, 
&fontval);
+   if(result == FcResultMatch) {
+   DPI = XDisplayWidth(xw.dpy, xw.scr) / 
+   (XDisplayWidthMM(xw.dpy, xw.scr) / 
25.4);
+   usedfontsize = (int)DPI * fontval / 72;
+   FcPatternAddDouble(pattern, FC_PIXEL_SIZE, 
usedfontsize);
+   } else {
+   /*
+* Default font size is 12, if none given. This 
+* is to have a known usedfontsize value.
+*/
+   FcPatternAddDouble(pattern, FC_PIXEL_SIZE, 12);
+   usedfontsize = 12;
+   }
}
}

Re: [dev] [sbase] [patch] cal

2013-06-20 Thread Jesse Ogle 
Various touch ups to code and man. Fixed problem that occurred if
multiple -3 flags were given.

On Wed, Jun 19, 2013 at 5:24 PM, Christoph Lohmann <2...@r-36.net> wrote:
> Greetings.
>
> On Wed, 19 Jun 2013 23:24:50 +0200 Jesse Ogle  wrote:
>> Two small patches.
>
> Thanks, both have been applied. Please break long strings.
>
>
> Sincerely,
>
> Christoph Lohmann
>
>


sbase-cal-various-clean-ups.patch
Description: Binary data

Re: [dev] [st] [patch] Fixed lock up when system time jumps back

2013-06-20 Thread Silvan Jegen 
On Wed, Jun 19, 2013 at 09:26:13PM +0100, Rob wrote:
> It seems st enters a heavy draw cycle when system time is moved back (e.g.
> ntp).
> This patch should fix it.

I had the same problem and this patch seems to have fixed it though
some more testing may be advisable. Thanks!

I was trying to debug the problem using gdb but the backtrace hang
somewhere in libc and I could not make sense of it.

Re: [dev][sbase] seq possible bug

2013-06-20 Thread Galos, David 
Revised patch: this one also matches the usage to the manpage, fixes
the troff warning in the manpage, and adds a SEE ALSO section to the
manpage.


sbase-fix-seq.diff
Description: Binary data