Memory leak in backlight driver about liblxqt HOT 5 CLOSED

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

from liblxqt.

@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.

from liblxqt.

@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

from liblxqt.

@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.

from liblxqt.

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

from liblxqt.