Memory leak in backlight driver

[selairi]

Sorry, I din not realize that I was in lxqt repository and I pushed this:
00a3949

Backlight doesn't delete the backlight driver after use and it builds a new one.

Expected Behavior

Change the backlight.
After 60 seconds backlight driver is deleted.

Current Behavior

Backlight driver doesn't be deleted and new one is build.

Possible Solution

The solution is close the stream to backlight driver.
Sorry, I din not realize that I was in lxqt repo and I pushed the solution:
00a3949
I was thinking that I was pushing to my own repo.

Steps to Reproduce (for bugs)

1. Open qps to check backlight driver
2. Change the backlight (lxqt-config-brightness) for several minutes.
3. Check qps and there are lots of backlight drivers

Context

System Information

• Distribution & Version:
• Kernel:
• Qt Version:
• libqtxdg Version: 0.14
• lxqt-build-tools Version:
• Package version:

[tsujan]

@selairi
00a3949 looks good to me, although I haven't had time to test it (I think you did).

[selairi]

@tsujan Yes, I test it. If you create a new LXQt::Backlight object and then you change the backlight, LXQt::Backlight will run a new backlight driver and it will kill it after 60 seconds.

I found the bug when I was testing the tool to change the backlight when computer is idle. For testing, I selected 5 seconds of idleness and when I checked the PID of lxqt-powermanager to kill it and run a new version, I found lots of backlight drivers running. A new one each 60 seconds. LXQt::Backlight was not killing the backlight driver.

[tsujan]

@selairi
Yes, I'd guessed that you found and fixed the issue while testing your solution for https://github.com/lxqt/lxqt-powermanagement/issues//188

[selairi]

@selairi
Yes, I'd guessed that you found and fixed the issue while testing your solution for https://github.com/lxqt/lxqt-powermanagement/issues//188

Yes, you are right. I found it while testing that tool.

[tsujan]

The commit was merged. No point in keeping this open.