Skip to content

Qt: add log level settings to debug tab - #18278

Merged
Megamouse merged 2 commits into
RPCS3:masterfrom
Megamouse:cellGemFix
Mar 6, 2026
Merged

Megamouse merged 2 commits into
RPCS3:masterfrom
Megamouse:cellGemFix

Conversation

@Megamouse

Copy link
Copy Markdown
Contributor
  • Add log level settings to debug tab
  • Update yaml-cpp since there was a bug in our custom exception handling commit

@digant73

digant73 commented Mar 5, 2026 •

Copy link
Copy Markdown
Contributor

is it normal that even setting log level to Notice for sys_net like in the following screenshot then sys_net functions are still logged in the Log panel when log level in Log panel is set to Success?
In order to suppress the logging I have to set a log level above Success in the Log Level dialog box, but this logic is exactly the opposite of the logic used in Log panel where I need to increase the log level above TODO in order any TODO trace is suppressed.

image image

@Megamouse

Copy link
Copy Markdown
Contributor Author

I chose the wrong wording.
It's an existing setting that logs anything <= log level (even trace if you set it to trace).
The log level in the log frame does the same for the log frame (of course only if stuff was logged in the first place).
So it's not opposite. Idk why you said that

@Megamouse
Megamouse force-pushed the cellGemFix branch 6 times, most recently from 835a0db to 3dbd2d0 Compare March 6, 2026 07:47
@digant73

digant73 commented Mar 6, 2026

Copy link
Copy Markdown
Contributor

I chose the wrong wording. It's an existing setting that logs anything <= log level (even trace if you set it to trace). The log level in the log frame does the same for the log frame (of course only if stuff was logged in the first place). So it's not opposite. Idk why you said that

Ok, I understood now.

@Megamouse
Megamouse merged commit 0d80e30 into RPCS3:master Mar 6, 2026
10 checks passed
@Megamouse
Megamouse deleted the cellGemFix branch March 6, 2026 14:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants