Review of "GPU Supergfxctl Switch" version 13

Details Page Preview

Simple GPU Profile switcher Gnome-Shell-Extension for ASUS laptops using Supergfxctl

Extension Homepage
https://github.com/chikobara/GPU-Switcher-Supergfxctl

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) warning 3

Shexli found 3 issues that may need reviewer attention.

EGO-A-004 warning

extension files should not contain excessive ungated console logging

File contains 20 ungated console.log/warn/error calls (threshold: 5).

No excessive logging

  • extension.js:99
    console.log(`NotifyGfx signal received, new mode index: ${newMode}`)
  • extension.js:169
    console.log(`Command failed, retrying in ${RETRY_DELAY}ms...`)
  • extension.js:215
    console.log(`Activating profile: ${profile}`)
  • extension.js:226
    console.log(`Profile ${profile} is already active. Skipping activation.`)
  • extension.js:247
    console.log(`Profile ${profile} activated successfully`)
  • extension.js:269
    console.log(`Setting active profile: ${profile}`)
  • extension.js:283
    console.log(`Synchronizing profile: ${this._activeProfile}`)
  • extension.js:361
    console.log(`Updated icon: ${this._indicator.icon_name}, Visible: ${this._indicator.visible}`)
  • extension.js:114
              console.error(
                "Failed to fetch supported profiles after multiple attempts"
              )
  • extension.js:129
    console.error(`Error parsing supported profiles: ${e.message}`)

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:74
    this._profileSection = new PopupMenu.PopupMenuSection()

EGO-L-005 warning

owned object references should be released in disable()

Owned references that are cleaned up in `disable()` should also be released with `null` or `undefined`.

Destroy all objects

  • extension.js:74
    this._profileSection = new PopupMenu.PopupMenuSection()

All Versions

Version Status
13 Active
12 Active
11 Active
10 Active
9 Active
8 Active
7 Rejected
6 Rejected
5 Inactive
4 Inactive
3 Inactive
2 Rejected
1 Rejected

Previous Reviews on this Version

fmuellner active