Review of "GPU Indicator" version 0.1.0 (1)

Details Page Preview

Shows in the top bar whether the focused application renders on the integrated GPU or the NVIDIA GPU of a hybrid graphics laptop.

Extension Homepage
https://github.com/kemalasliyuksek/gpu-indicator

No comments.

FAQ

Files

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

Shexli (experimental) warning 2

Shexli found 2 issues that may need reviewer attention.

EGO-X-004 warning

extensions should avoid synchronous file IO in shell code

Shell code should avoid synchronous file IO APIs like `GLib.file_get_contents()` and `Gio.File.load_contents()`.

File Operations

  • gpu.js:21
    GLib.file_get_contents(path)

EGO-L-002 warning

objects created by extension should be destroyed in disable()

Objects assigned in `enable()` are missing matching `.destroy()` calls in `disable()` or its helper methods.

Destroy all objects

  • extension.js:49
    this._icon = new St.Icon({style_class: 'system-status-icon'})

All Versions

Version Status
0.1.1 (2) Unreviewed
0.1.0 (1) Rejected

Previous Reviews on this Version

kemalasliyuksek auto- rejected
Auto-rejected because of new version 0.1.1 (2) was uploaded