Re: [hackers] [PATCH slstatus V2] Add basic backlight percentage support

From: Aaron Marcher <me_AT_drkhsh.at>
Date: Mon, 21 May 2018 18:43:10 +0200

David,

> Do you mean in bprintf call, so like this: bprintf("%d", ...) ?

Yes, thanks.

> The problem is that util.h actually requires stddef.h (it uses size_t
> which is is defined there). I think that include should eventually be
> moved there.

Okay then move it to util.h in your patch, forgot that.

> So even if I add support for FreeBSD you'll refuse it? I know the
> code, I've already implemented I just need to get a FreeBSD laptop to
> ensure it still works though :)

For now, yes. But please keep your awesome work saved so that we can add
it when we port slstatus eventually to FreeBSD! For now, Linux and
OpenBSD should be fully ported.

Cheers!
Aaron

-- 
Web: https://drkhsh.at/ or http://drkhsh5rv6pnahas.onion/
Gopher: gopher://drkhsh.at or gopher://drkhsh5rv6pnahas.onion
GPG: 0x7A65E38D55BE96FE
Fingerprint: 4688 907C 8720 3318 0D9F AFDE 7A65 E38D 55BE 96FE
Received on Mon May 21 2018 - 18:43:10 CEST

This archive was generated by hypermail 2.3.0 : Mon May 21 2018 - 18:48:36 CEST