[dev] test

2020-06-16 Thread Salvatore Cuzzilla
--- :wq, Salvatore.

[dev] Re: [slstatus] temperature module acts wierd on OpenBSD

2020-06-16 Thread messw1thdbest
Sorry, I messed up the formatting 22d21 < #include 48c47 < return bprintf("%d", (temp.value - 27315) / 1E6); --- > return bprintf("%d", (temp.value - 27315) / 100); Sent with ProtonMail Secure Email. ‐‐‐ Original Message ‐‐‐ On Tuesday, Jun

[dev] [slstatus] temperature module acts wierd on OpenBSD

2020-06-16 Thread messw1thdbest
Hi Suckless I think there is a problem with the temperature module, that is, on OpenBSD it only prints one digit, which by the way doesn't even change (stuck at 5). By making a small change, it now works properly. (I also removed stdio.h which looks unnecessary) 22d21 < #include 48c47 <

Re: [dev] Re: [slstatus] temperature module acts wierd on OpenBSD

2020-06-16 Thread Laslo Hunhold
On Tue, 16 Jun 2020 20:53:34 +0200 Mattias Andrée wrote: Dear Mattias, > I'm assuming temp.value i an `int`, as %d is used. The problem was > probably that `1E6` is actually a `double` rather than an `int`, > as the whole expression is promoted to `double`, because `bprintf` is > (I assume) vari

Re: [dev] [slstatus] temperature module acts wierd on OpenBSD

2020-06-16 Thread Joerg Jung
> On 16. Jun 2020, at 22:30, Joerg Jung wrote: > > >> On 16. Jun 2020, at 20:53, Mattias Andrée wrote: >> >> I'm assuming temp.value i an `int`, as %d is used. The problem was >> probably that `1E6` is actually a `double` rather than an `int`, >> as the whole expression is promoted to `doub

Re: [dev] [slstatus] temperature module acts wierd on OpenBSD

2020-06-16 Thread Joerg Jung
> On 16. Jun 2020, at 20:53, Mattias Andrée wrote: > > I'm assuming temp.value i an `int`, as %d is used. The problem was > probably that `1E6` is actually a `double` rather than an `int`, > as the whole expression is promoted to `double`, because `bprintf` is > (I assume) variadic, and the com

Re: [dev] Re: [slstatus] temperature module acts wierd on OpenBSD

2020-06-16 Thread Mattias Andrée
I'm assuming temp.value i an `int`, as %d is used. The problem was probably that `1E6` is actually a `double` rather than an `int`, as the whole expression is promoted to `double`, because `bprintf` is (I assume) variadic, and the compiler does not know to change the cast the expression back to `in

Re: [dev] Re: [slstatus] temperature module acts wierd on OpenBSD

2020-06-16 Thread Laslo Hunhold
On Tue, 16 Jun 2020 17:55:03 + messw1thdbest wrote: Dear messw1thdbest, > < return bprintf("%d", (temp.value - 27315) / 1E6); > --- > > return bprintf("%d", (temp.value - 27315)/100); I'm really intrigued by that; thanks for sending in this patch! W