Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

"TControl.InvalidatePreferredSize loop detected" Error #82

Closed
Horridge opened this issue Mar 7, 2024 · 5 comments
Closed

"TControl.InvalidatePreferredSize loop detected" Error #82

Horridge opened this issue Mar 7, 2024 · 5 comments
Labels
base-transgui-bug Has a corresponding issue in base transgui repo bug Something isn't working
Milestone

Comments

@Horridge
Copy link

Horridge commented Mar 7, 2024

Received an error dialogue box this morning,
image

Unfortunately don't know the cause, happened over night whilst the application would've been minimised to system tray. Running v5.18.4.f, with 1727 torrents (one was added via Radarr this morning, if relevant). Closing transgui and re-opening without issue.

@lighterowl
Copy link
Owner

Hi, thanks for the report.

I've seen this one before a couple of times but unfortunately haven't been able to track down the cause. I can only assume this is somehow related to all the code that overrides Lazarus' own logic for resizing and placing widgets (basically, the TBaseForm class, among others), but this is only speculation at this point.

@lighterowl lighterowl added the bug Something isn't working label Mar 7, 2024
@cvn
Copy link

cvn commented Jun 29, 2024

Hi, I get this error just by selecting certain torrents. It seems to be consistent, every time I select one of the problem torrents, it shows this error, and then the app quits. I'm on MacOS 14.5, transgui 5.18.4.f.

One torrent that triggers the issue is /description.php?id=7689899 on TPB.

Screenshot 2024-06-29 at 3 29 40 PM

@lighterowl lighterowl added the base-transgui-bug Has a corresponding issue in base transgui repo label Jul 7, 2024
@lighterowl
Copy link
Owner

transmission-remote-gui#913
transmission-remote-gui#1364
transmission-remote-gui#1459 (closed, but only because it stopped happening at literally every run)

@lighterowl
Copy link
Owner

lighterowl commented Jul 21, 2024

The changes I made today (in 438ccb8) should make this go away but I wouldn't dare to call this "fixed" as the root cause is still there. I just made the call that triggers this exception wrapped in a "swallow everything" block. At worst, some UI will be broken (i.e. badly sized) but at least the app won't just stop outright for no reason. You can grab the last build from develop for testing if you want to try it out.

The root cause is all the autosizing hacks whose purpose I just can't figure out and at the same time cannot remove because the UI breaks completely without them.

@lighterowl
Copy link
Owner

I decided to bite the bullet and get rid of BaseForm completely, so this should not be a problem as of the next release (5.18.6).

@lighterowl lighterowl added this to the 5.18.6 milestone Aug 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
base-transgui-bug Has a corresponding issue in base transgui repo bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants