Review "Brightness control using ddcutil" version 7

Review of "Brightness control using ddcutil" version 7

Details Page Preview

Brightness control using ddcutil

Brightness control for all the monitors detected by ddcutil This tool uses ddcutil as backend for communication with your display. Read setup instructions from: https://github.com/daitj/gnome-display-brightness-ddcutil/blob/master/README.md

Extension Homepage: https://github.com/daitj/gnome-display-brightness-ddcutil

No comments.

Diff Against

Files

Note: Binary files aren't shown on the web site. To see all files, please download the extension zipfile.

Shexli (experimental)

Shexli has not produced a result yet.

All Versions

Version	Status
59	Unreviewed
58	Active
57	Active
56	Active
55	Active
54	Active
53	Active
52	Active
51	Active
50	Active
49	Active
48	Active
47	Active
46	Active
45	Rejected
44	Active
43	Active
42	Inactive
41	Inactive
40	Inactive
39	Rejected
38	Inactive
37	Inactive
36	Inactive
35	Inactive
34	Inactive
33	Inactive
32	Inactive
31	Active
30	Active
29	Inactive
28	Inactive
27	Inactive
26	Rejected
25	Inactive
24	Inactive
23	Inactive
22	Inactive
21	Inactive
20	Inactive
19	Rejected
18	Rejected
17	Rejected
16	Inactive
15	Inactive
14	Inactive
13	Inactive
12	Inactive
11	Inactive
10	Inactive
9	Rejected
8	Rejected
7	Inactive
6	Inactive
5	Rejected
4	Inactive
3	Inactive
2	Inactive
1	Inactive

Previous Reviews on this Version

andyholmes active July 4, 2020

Looks good. You may want to add a try...catch around `communicate_utf8_finish()` as I believe it may throw errors. Unless it's the intended behaviour that your callback not be invoked in that case.