Skip to content

[Linux - GTK3/GTK2] Action Replay code menu (+ other cheat menu improvements) #847

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.

Sign up for GitHub

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

Merged
merged 11 commits into from
Oct 25, 2024
Merged

[Linux - GTK3/GTK2] Action Replay code menu (+ other cheat menu improvements) #847

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
8a3302c
GTK cheats UI inputs/displays hex for address (offset)
En-En-Code Sep 26, 2024
8f73a9c
Added range bound to keep internal cheat data within specified size
En-En-Code Sep 27, 2024
ce58ebb
Added index and cheat type data to ListStore
En-En-Code Sep 29, 2024
9f18dfb
Filter raw and AR cheats, display both filters in UI, patch up raw up…
En-En-Code Oct 1, 2024
497b398
Action Replay UI elements [GTK]
En-En-Code Oct 7, 2024
0599143
Formating
En-En-Code Oct 8, 2024
058fc06
Memory leak fixes
En-En-Code Oct 10, 2024
c9f3a6b
Minimize formatting diffs
En-En-Code Oct 11, 2024
c9461e8
Backport to GTK2
En-En-Code Oct 14, 2024
ee9f38d
Makefile.am includes utilsGTK.*
En-En-Code Oct 15, 2024
a01d71f
Needed fixes
En-En-Code Oct 25, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 11 additions & 1 deletion desmume/src/cheatSystem.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/*
Copyright (C) 2009-2023 DeSmuME team
Copyright (C) 2009-2024 DeSmuME team

This file is free software: you can redistribute it and/or modify
it under the terms of the GNU General Public License as published by
Expand Down Expand Up @@ -849,6 +849,16 @@ bool CHEATS::remove(const size_t pos)
return didRemoveItem;
}

void CHEATS::toggle(bool enabled, const size_t pos)
{
this->_list[pos].enabled = (enabled) ? 1 : 0;
}

void CHEATS::toggle(u8 enabled, const size_t pos)
{
this->toggle((enabled != 0), pos);
}

void CHEATS::getListReset()
{
this->_currentGet = 0;
Expand Down
5 changes: 4 additions & 1 deletion desmume/src/cheatSystem.h
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/*
Copyright (C) 2009-2023 DeSmuME team
Copyright (C) 2009-2024 DeSmuME team

This file is free software: you can redistribute it and/or modify
it under the terms of the GNU General Public License as published by
Expand Down Expand Up @@ -120,6 +120,9 @@ class CHEATS

bool remove(const size_t pos);

void toggle(bool enabled, const size_t pos);
void toggle(u8 enablbed, const size_t pos);
Copy link
Collaborator

Choose a reason for hiding this comment

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

typo. do we really need an overloaded func for this ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Maybe not, but the entire cheatSystem overloaded enabled this way: add L99-100, update L103-104, add_AR L109-110, etc. I followed suit, assuming there was a design reason behind it.

Copy link
Collaborator

Choose a reason for hiding this comment

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

then it's probably fine.


void getListReset();
bool getList(CHEATS_LIST *cheat);
CHEATS_LIST* getListPtr();
Expand Down
1 change: 1 addition & 0 deletions desmume/src/frontend/posix/gtk/Makefile.am
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,7 @@ desmume_SOURCES = \
desmume.h desmume.cpp \
dTool.h dToolsList.cpp \
tools/ioregsView.cpp tools/ioregsView.h \
utilsGTK.h utilsGTK.cpp \
cheatsGTK.h cheatsGTK.cpp \
main.cpp main.h

Expand Down
Loading