Skip to content

Conversation

@chelovechishko
Copy link
Contributor

No description provided.

@csoler
Copy link
Contributor

csoler commented Apr 17, 2022

This PR seems to cause a crash: open the statistics, then close it using the Quit button. After 2 secs, the program crashes.

@chelovechishko
Copy link
Contributor Author

commented out wa_deleteonclose.
retroshare should crash at usual pace now.

@csoler
Copy link
Contributor

csoler commented May 11, 2022

  • the PR still crashes when closing twice the statistics using the statistics button. Try with valgrind if it doesn't crash, valgrind will still tell you what's wrong. Or just cancel out that particular change: make the exit button to close the window instead of deleting it.

to free memory on close.

before opening->clicked through all tabs->after closing
without WA_DeleteOnClose:
 180->225->225 ; ->225->225...
with WA_DeleteOnClose:
 180->226->187 ; ->224->189...

There is a mud with spaceotabulation, so don't poke me for doing
automatic things for indentation, instead of understanding what should
be there for new code.
'tis no more saves last opened statistics' tab,
but hopefully nobody needs such a thing.
Only silhouette of word instead of time no more.
This is maximum length of time representation, but auto shrinkable.
As qt says: Ownership of the menu is not transferred to the tool button.
It still shrinks for some amount, but less.
What exactly cause such behaviour from post to post still unknown.
Copy link
Contributor

@defnax defnax left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

47fc190
This is a nice feature, would be great you do a short PR for this feature :)

@chelovechishko
Copy link
Contributor Author

chelovechishko commented Aug 9, 2023 via email

@G10h4ck G10h4ck force-pushed the master branch 2 times, most recently from 1f64d0b to 67762c1 Compare November 27, 2023 01:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants