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

Game List: Add Remove HDD1 Cache option #14250

Merged
merged 3 commits into from Jul 23, 2023
Merged

Game List: Add Remove HDD1 Cache option #14250

merged 3 commits into from Jul 23, 2023

Conversation

elad335
Copy link
Contributor

@elad335 elad335 commented Jul 23, 2023

In addition, fix allowance of caches/games to be removed while the game is still running.
Imagine removing the currently running game lol!

@elad335 elad335 changed the title Qt: Add Remove HDD1 Cache option Game List: Add Remove HDD1 Cache option Jul 23, 2023
@Megamouse Megamouse added the GUI label Jul 23, 2023
rpcs3/rpcs3qt/game_list_frame.cpp Outdated Show resolved Hide resolved
rpcs3/rpcs3qt/game_list_frame.cpp Outdated Show resolved Hide resolved
rpcs3/rpcs3qt/game_list_frame.cpp Outdated Show resolved Hide resolved
rpcs3/rpcs3qt/game_list_frame.cpp Outdated Show resolved Hide resolved
rpcs3/rpcs3qt/game_list_frame.cpp Outdated Show resolved Hide resolved
rpcs3/rpcs3qt/game_list_frame.cpp Outdated Show resolved Hide resolved
rpcs3/rpcs3qt/game_list_frame.cpp Outdated Show resolved Hide resolved
rpcs3/rpcs3qt/game_list_frame.cpp Outdated Show resolved Hide resolved
rpcs3/rpcs3qt/game_list_frame.cpp Outdated Show resolved Hide resolved
rpcs3/rpcs3qt/game_list_frame.cpp Outdated Show resolved Hide resolved
@elad335 elad335 force-pushed the hdd12 branch 2 times, most recently from 4a62df7 to 1c29d65 Compare July 23, 2023 12:49
Comment on lines 1972 to 1975
ui->batchCreatePPUCachesAct->setEnabled(false);
ui->batchRemovePPUCachesAct->setEnabled(false);
ui->batchRemoveSPUCachesAct->setEnabled(false);
ui->batchRemoveShaderCachesAct->setEnabled(false);
Copy link
Contributor

Choose a reason for hiding this comment

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

that's not part of the toolbar

Comment on lines 1911 to 1914
ui->batchCreatePPUCachesAct->setEnabled(false);
ui->batchRemovePPUCachesAct->setEnabled(false);
ui->batchRemoveSPUCachesAct->setEnabled(false);
ui->batchRemoveShaderCachesAct->setEnabled(false);
Copy link
Contributor

Choose a reason for hiding this comment

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

this doesn't make any sense

@elad335 elad335 force-pushed the hdd12 branch 3 times, most recently from 493aaed to f93e395 Compare July 23, 2023 13:05
@Megamouse
Copy link
Contributor

lgtm. didn't test though.

@elad335 elad335 merged commit e303cc5 into RPCS3:master Jul 23, 2023
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants