Review "Clipboard Indicator" version 2

Review of "Clipboard Indicator" version 2

Details Page Preview

Clipboard Indicator

by Tudmotu

The most popular clipboard manager for GNOME, with over 1M downloads

Extension Homepage: https://github.com/Tudmotu/gnome-shell-extension-clipboard-indicator

No comments.

Diff Against

Files

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

All Versions

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

Previous Reviews on this Version

Tudmotu posted a review May 24, 2014

After three months of using this extension, no major bugs seem to exist, even though I previously self-rejected that same code base. Also, added 3.12 to metadata.

fmuellner active May 28, 2014

Some comments: - you should consider switching to asynchronous IO for reading/writing the "registry"; blocking IO means the compositor won't redraw the screen while writing/reading the file, the window manager won't process user events on windows, ... - you should not call timeout_add() from the timeout handler - just return true (or in your case: recurse) to have the handler run repeatedly Those are nits that would be good to address, but no reason to reject this version altogether.