tgohle/0003-SDG2042X-PyQt-GUI-for-Linux
1
0
Fork 0
Code Issues 6 Pull Requests Actions Packages Projects Releases 1 Wiki Activity

[Task]: Screenshot preview window lifetime handling is fragile #7

New Issue
Open
opened 2026-04-17 11:31:40 +00:00 by tgohle · 0 comments
tgohle commented 2026-04-17 11:31:40 +00:00
Owner
Copy Link

Problem

The screenshot preview window is stored as a single reference:

self._scr_win = w

Why this is a problem

The preview widget has no parent and only one stored reference.

If another screenshot is taken before the previous preview window is closed, the old reference is overwritten. That creates a risk that the previous window object can be garbage-collected while still visible.

Fix approach

Use a safer window-lifetime strategy, for example:

  • keep preview windows in a list
  • remove closed windows from that list
  • or guard against replacing a currently visible window

Status

Still open.
Not fixed in the current v0.2 working copy.

### Problem The screenshot preview window is stored as a single reference: ```python self._scr_win = w ``` ### Why this is a problem The preview widget has no parent and only one stored reference. If another screenshot is taken before the previous preview window is closed, the old reference is overwritten. That creates a risk that the previous window object can be garbage-collected while still visible. ### Fix approach Use a safer window-lifetime strategy, for example: - keep preview windows in a list - remove closed windows from that list - or guard against replacing a currently visible window ### Status Still open. Not fixed in the current v0.2 working copy.
tgohle added the
area/screenshot
kind
task
3
severity
low
3
state
confirmed
2
 labels 2026-04-17 11:45:35 +00:00
tgohle self-assigned this 2026-04-17 12:09:17 +00:00
tgohle changed title from Screenshot preview window lifetime handling is fragile to [Task]: Screenshot preview window lifetime handling is fragile 2026-04-18 11:20:40 +00:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
area/arb
area/config
area/gui-basic
area/presets
area/screenshot
area/transport
kind
bug
1
Bug
kind
enhancement
2
Enhancement
kind
task
3
Task 2 do
severity
high
1
Severity High
severity
low
3
Severity Low
severity
medium
2
Severity Medium
state
blocked
3
Blocked
state
confirmed
2
its confirmed
state
needs-test
1
Needs Test
state
regression
4
State - Regression
No Label
area/screenshot
kind
task
3
severity
low
3
state
confirmed
2
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
tgohle
No Assignees tgohle
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: tgohle/0003-SDG2042X-PyQt-GUI-for-Linux#7
No description provided.
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?