Lioncash
9dd29895cd
TASInputDlg: Replace includes with forward declarations
2017-04-03 04:29:15 -04:00
Lioncash
2a968f02d7
TASInputDlg: Make constructor explicit
2017-04-03 04:29:14 -04:00
Lioncash
28395c6302
TASInputDlg: Simplify event queueing calls
...
wxQueueEvent/wxPostEvent are useful when the event is being dispatched
to another separate window, but aren't really necessary when the event
will be handled by the same window it's dispatched from.
GetEventHandler() is unnecessary here for the same reason. It's an event
intended to be handled by the dialog itself.
2017-04-03 04:29:14 -04:00
Lioncash
598a92426f
TASInputDlg: Get rid of unnecessary event casts
...
wxCommandEvent already stores the necessary data; the control object
itself doesn't need to be retrieved and casted.
2017-04-03 04:29:14 -04:00
Lioncash
2af5b64f32
TASInputDlg: Apply auto to UI control initializations where appropriate
...
As all UI controls are essentially constructed with new expressions, the
type is already visible on the right-hand side, so repeating the type
twice isn't necessary.
2017-04-03 04:29:11 -04:00
Lioncash
b809db52d1
TASInputDlg: Use std::array where applicable
2017-04-03 04:28:11 -04:00
Mat M
c626fd7c74
Merge pull request #5195 from leoetlino/unneeded-fwd
...
IOS/IPC: Remove unneeded forward declaration
2017-04-02 06:50:17 -04:00
Lioncash
05872336a9
ExpressionParser: Rename ParseStatus' Success member to Successful
...
This clashes with X11's preprocessor define named Success (because using
non-prefixed lowercase identifiers in C was apparently a fantastic idea
at some point), causing compilation errors.
2017-04-02 06:13:18 -04:00
Lioncash
c09f7bd84b
Frame: Convert TAS dialog C array to std::array
2017-04-02 04:11:58 -04:00
Lioncash
678905764a
Movie/Frame: Amend variable naming
...
Amends variable naming related to translation unit locals and TAS input
dialogs that were modified in the previous commit.
2017-04-02 04:11:58 -04:00
Lioncash
7f0203a5b0
Frame: Make TAS dialogs private
...
Amends the TAS callbacks to internally store functions using
std::function instead of raw function pointers. This allows binding
extra contextual state via lambda functions, as well as keeping the
dialogs internal to the main frame (on top of being a more flexible
interface).
2017-04-02 04:11:47 -04:00
Lioncash
5620f78b5c
Core: Amend VS filter tag
2017-04-02 00:30:26 -04:00
Mat M
a65a176777
Merge pull request #5192 from vtj/geckofix
...
Fix disappearing gecko codes
2017-04-01 18:01:28 -04:00
Mat M
67e950d02c
Merge pull request #5168 from ligfx/vertexloadertestuninitialized
...
VertexLoaderTest: fix maybe-uninitialized warning
2017-03-31 17:26:20 -04:00
Mat M
0aec9fa4db
Merge pull request #5149 from JosJuice/di-directory
...
Move DVD code to a new directory
2017-03-31 12:05:37 -04:00
Mat M
542fb347f1
Merge pull request #5160 from mahdihijazi/enhance_style
...
android: Change the color of the selected slot to make it more visible
2017-03-31 03:57:32 -04:00
Mat M
f5dcfb157d
Merge pull request #5151 from mahdihijazi/update_sdk
...
Update to Android SDK 25
2017-03-31 00:27:10 -04:00
Mat M
006381bb47
Merge pull request #5184 from leoetlino/dead-code
...
Common: Remove dead code in NandPaths
2017-03-31 00:21:49 -04:00
Mat M
6a595aabe9
Merge pull request #5153 from ligfx/stubhostnorequestfullscreen
...
StubHost: remove unused Host_RequestFullscreen()
2017-03-31 00:21:04 -04:00
Mat M
6b198679bc
Merge pull request #5129 from ligfx/breakpointprintf
...
Breakpoints.cpp: fix format string warnings
2017-03-31 00:13:06 -04:00
Mat M
80a812322e
Merge pull request #5152 from ligfx/disciomaybeuninitialized
...
DiscIO: initialize out args to ReadSwapped (fixes warning)
2017-03-31 00:11:53 -04:00
Mat M
774d2420e3
Merge pull request #5158 from ligfx/coretimingstatic
...
CoreTimingTest: make AdvanceAndCheck() static (fix warning)
2017-03-31 00:09:20 -04:00
Lioncash
6495becaf7
Frame: Remove GetGameListCtrl()
...
This is currently unused and shouldn't actually be a part of the frame's
public interface. The event system should be used instead to dispatch
messages to the game list control if necessary.
2017-03-30 16:05:20 -04:00
Lioncash
497292cb1b
MainMenuBar: Amend MEGA signature help tooltip
...
Keeps it consistent with the regular signature DB option
2017-03-28 17:14:04 -04:00
Lioncash
5637bbd1cf
MEGASignatureDB: Remove unused variable in Apply
2017-03-28 16:19:19 -04:00
Lioncash
eec4b705d5
MEGASignatureDB: Move objects where applicable
...
Signatures during loading and strings during reference retrieval can be
moved.
2017-03-28 16:18:31 -04:00
Lioncash
c0171975ea
MEGASignatureDB: Move implementation details into the cpp file
...
These functions aren't dependent on any database state and can be hidden.
2017-03-28 15:43:21 -04:00
Lioncash
8d98ac6509
CPU: Convert state enum to an enum class
...
Gets enum constants out of the immediate namespace. Also makes it
strongly typed like the other state enums.
2017-03-28 11:48:28 -04:00
Lioncash
41fbc659bf
Fifo: Remove unnecessary integral casts
...
FIFO_SIZE is a u32, so %u can just be used instead.
2017-03-28 10:35:26 -04:00
Lioncash
9ebd84e54a
VertexLoaderManager: Return debug strings by value
...
This also renames AppendListToString to VertexLoadersToString.
2017-03-26 23:50:09 -04:00
Lioncash
9859533ab4
VertexLoaderBase: Return debug strings by value
...
An out parameter for this sort of thing is a C++03 hold-over. This also
renames AppendToString to ToString.
2017-03-26 23:49:59 -04:00
Lioncash
d44844ba29
XFStructs: Get rid of an unnecessary pointer cast
...
Also gets rid of implicit sign conversions.
2017-03-26 23:09:30 -04:00
Lioncash
62db55dee2
Fifo: const correctness
...
PushFifoAuxBuffer only memcpys data using ptr as the source pointer, so
it can be a pointer to const data because of that.
2017-03-26 23:08:33 -04:00
Lioncash
8033a72f0b
VertexLoaderBase: Use std::array where applicable
2017-03-26 22:12:20 -04:00
Lioncash
adbffc24e9
VideoInterface: Make clock frequency array constexpr
...
This is only ever read from.
2017-03-26 19:30:44 -04:00
Lioncash
ec5637fe46
VideoInterface: Utilize std::array where applicable
2017-03-26 19:28:56 -04:00
Lioncash
ae29cd042e
VideoInterface: Initialize union members directly where applicable
2017-03-26 18:42:09 -04:00
Lioncash
0a7a6142eb
GCMemcardDirectory: Move flush interval constant into the cpp file
...
This isn't necessary to be within the header.
2017-03-26 17:55:30 -04:00
Lioncash
a5c5645bed
DSPIntMultiplier: Place translation unit local functions in an anonymous namespace
...
Unlike inline, this makes the functions internally linked.
2017-03-26 16:42:35 -04:00
Lioncash
57c21b9576
TextureCacheBase: Convert bound_textures from a C array to a std::array
...
Prevents array-to-pointer decay and simplifies some code.
2017-03-25 14:53:25 -04:00
Lioncash
b711daee5f
BitField: Get rid of a C-style cast
...
This can simply be the max value of the unsigned type.
2017-03-23 15:28:58 -04:00
Lioncash
e43c495ce5
BitField: Convert typedefs to using aliases
2017-03-23 15:28:50 -04:00
Lioncash
caef08988e
BitField: Make mostly constexpr capable
...
Makes the constructor and retrieval functions constexpr.
2017-03-23 11:45:45 -04:00
Lioncash
da2e34f69b
VertexLoaderBase: In-class initialize class members where applicable
...
Same thing, less code.
2017-03-23 08:37:18 -04:00
Lioncash
a7ec2d3831
Arm64Emitter: Get rid of pointer casts in PoisonMemory
...
The previous code invokes undefined behavior.
2017-03-23 07:10:21 -04:00
Lioncash
1d6d85963f
ARDecrypt: const correctness
2017-03-22 20:35:00 -04:00
Lioncash
29ca22905b
IniFile: Replace a character erase with pop_back()
...
Same thing, more straightforward.
2017-03-22 19:32:10 -04:00
Lioncash
35959bdaf9
IniFile: Replace string joining code with JoinString
2017-03-22 19:30:15 -04:00
Lioncash
b92871111a
IniFile: std::move a std::string in GetLines
...
Also gets rid of an unnecessary string copy.
2017-03-22 19:09:25 -04:00
Lioncash
dbdf693c81
IniFile: Use character literals instead of string literals where applicable
...
Character overloads are generally better overall (range checks aren't
necessary, etc).
2017-03-22 19:03:17 -04:00
Lioncash
d8998b6392
IniFile: Provide an rvalue reference overload for SetLines
...
Allows moving in vectors instead of performing an unnecessary copy.
2017-03-22 18:49:13 -04:00
Lioncash
46d74a7760
IniFile: Make Section's string constructor instances take strings by value
...
As the name is immediately stored into a class member, a move here is a
better choice.
This also moves the constructor implementations into the cpp file to
avoid an otherwise unnecessary inclusion in the header. This is also
likely a better choice as Section contains several non-trivial members,
so this would avoid potentially inlining a bunch of setup and teardown
code related to them as a side-benefit.
2017-03-22 18:47:19 -04:00
Lioncash
99adc73383
IniFile: Make Section constructor explicit
2017-03-22 18:07:23 -04:00
Lioncash
2f52d04e30
GeckoCodeConfig: Use compare instead of substr for comparing substrings
...
Gets rid of an unnecessary string construction.
2017-03-21 18:12:40 -04:00
Lioncash
0d0e9f626d
GeckoCodeConfig: Amend unnecessary string literals to char literals
...
Also gets rid of an unnecessary std::string construction in a loop.
std::string already has an operator+ overload to prepend characters.
2017-03-21 18:03:06 -04:00
Lioncash
28235b5cb6
GeckoCodeConfig: Remove unused commented out code
2017-03-21 18:03:06 -04:00
Lioncash
f0fa692457
GeckoCodeConfig: Move gecko code title building to its own function
...
Keeps it separate from the rest of the saving code and also allows for
easy rvalue-reference moving into the lines vector as a side-benefit.
2017-03-21 18:03:02 -04:00
Lioncash
f91292eff2
GeckoCodeConfig: Return vector by value for LoadCodes()
...
Using an out-param is a leftover from C++03. Action Replay codes already
return the vector of codes by value as well.
2017-03-21 16:58:13 -04:00
Lioncash
780dffcb13
FifoDataFile: In-class initialize member variables where applicable
2017-03-21 15:41:05 -04:00
Lioncash
4e5e7cff0a
FifoRecorder: In-class initialize member variables where applicable
2017-03-21 15:41:05 -04:00
Lioncash
eda27ee397
FifoPlayer: In-class initialize member variables where applicable
2017-03-21 15:40:56 -04:00
Lioncash
0d1bc53e55
JitBase: Rename MergeAllowedNextInstructions to CanMergeNextInstructions
...
This is more indicative that it's checking for something
2017-03-21 13:50:03 -04:00
Lioncash
f98211bfcf
JitBase: Make MergeAllowedNextInstructions a const member function
2017-03-21 13:45:18 -04:00
Lioncash
35c230a418
NetPlayServer: const correctness
2017-03-19 09:36:40 -04:00
Lioncash
025eac9062
NetPlayClient: const correctness
2017-03-19 09:32:58 -04:00
Lioncash
8e999b209a
NetPlayClient: Eliminate variable shadowing
...
Gets rid of two -Wshadow warnings
2017-03-18 22:58:07 -04:00
Lioncash
3404057732
ES: Remove unused string format argument in LaunchPPCTitle
...
Gets rid of a -Wformat-extra-args warning.
2017-03-18 22:44:46 -04:00
Lioncash
2b8abddadf
Memmap: Replace '0' constant with nullptr
2017-03-18 22:35:35 -04:00
Lioncash
6cf3c2262c
SI: Convert SI buffer from a C array to std::array
...
Stronger typing (no implicit pointer decay), nicer aggregate
initialization.
Also gets rid of magic constants where applicable.
2017-03-16 05:37:03 -04:00
Lioncash
f0eeb3c63a
SI: Namespace device classes
...
Places all of the SI code under the SerialInterface namespace instead of
only the main source file. This keeps all SI code under a common name,
as well as out of the global namespace
2017-03-16 04:41:39 -04:00
Lioncash
9c3b35fccb
EXI: Move enum constants into the ExpansionInterface namespace
...
Given these are related to EXI itself, these should be within the
namespace itself.
2017-03-16 02:06:29 -04:00
Lioncash
a728d858b3
SI: Move MAX_SI_CHANNELS into the SerialInterface namespace
...
Given this constant is related to the serial interface, it shouldn't be
outside of the namespace.
2017-03-16 01:57:57 -04:00
Lioncash
46991e2bdd
SI: Amend variable naming
2017-03-16 01:37:50 -04:00
Lioncash
4df8ac7820
GCMemcardRaw: Amend variable naming
2017-03-15 00:16:45 -04:00
Lioncash
79246d9642
GCMemcardDirectory: Amend variable naming
2017-03-14 23:58:31 -04:00
Lioncash
647bec4c17
GCMemcard: Amend variable naming for MemoryCardBase
2017-03-14 23:58:27 -04:00
Lioncash
09d4871067
GCMemcard: Move memcard source files to their own directory
...
GCMemcard.h has quite a bit of different classes implemented within it
that could likely be split up into other files to make it a little
easier to read. However, they should be moved into their own folder
first so that they don't clutter up the base HW directory.
2017-03-14 21:55:25 -04:00
Lioncash
8b8a1f3b35
SI_DeviceKeyboard: Amend variable naming
2017-03-14 18:15:25 -04:00
Lioncash
4478b9bc9a
SI_DeviceKeyboard: In-class initialize variables
2017-03-14 18:03:36 -04:00
Lioncash
c45f8e2e3c
SI_DeviceGCSteeringWheel: Amend variable naming
2017-03-14 17:54:18 -04:00
Lioncash
89f5b3b5d9
SI_DeviceGCAdapter: Amend variable naming
2017-03-14 17:47:56 -04:00
Lioncash
e68e99ef71
SI_DeviceDanceMat: Amend variable naming
2017-03-14 17:46:09 -04:00
Lioncash
46fd06b956
SI_DeviceGCController: Amend variable names
2017-03-14 05:15:48 -04:00
Lioncash
890f799fdd
SI_DeviceGCController: In-class initialize variables
2017-03-14 03:34:52 -04:00
Lioncash
deda29d7ff
SI_DeviceGBA: Amend printf formatting specifier for size_t
...
%zu is the correct specifier for printing size_t.
2017-03-13 15:28:07 -04:00
Lioncash
8d2078de2d
SI_DeviceGBA: Amend variable naming
2017-03-13 15:27:36 -04:00
Lioncash
9f40596de5
SI_DeviceGBA: Make GBASockServer's constructor explicit
...
Prevents potential implicit construction scenarios involving ints.
2017-03-13 14:53:50 -04:00
Lioncash
14f261079f
SI_DeviceGBA: Place implementations within the cpp file
...
Prevents potential recompilation of other files if these ever need to
change.
2017-03-13 14:51:18 -04:00
Lioncash
64aa7f3b8f
SI_DeviceGBA: Use std::array class members instead of C arrays
2017-03-13 14:46:35 -04:00
Lioncash
ea59d30e9f
SI_DeviceGBA: In-class initialize variables
2017-03-13 13:52:45 -04:00
Lioncash
70b1933661
SI_DeviceGBA: Store the device number as an int
...
This prevents truncation when assigning to this member in the
constructor. This isn't size-critical code, so opting for the more
straightforward assignment is fine here.
2017-03-13 13:30:02 -04:00
Lioncash
31644eaf93
DSPLLE: Remove commented out code
...
Just throwing out the trash
2017-03-13 12:36:53 -04:00
Lioncash
f183d6759a
HW/DSP: Remove prefixed underscores from parameter names
...
Avoids stepping on potentially reserved names and is more consistent
with the rest of the surrounding code.
2017-03-13 12:27:38 -04:00
Lioncash
ade7718636
DSPInterpreter: Remove unnecessary const specifiers from function declaration parameters
2017-03-12 17:38:04 -04:00
Lioncash
a33cd805f9
DSPEmitter: Remove unnecessary const specifiers from function declaration parameters
...
These only matter for types being passed by value within the definition,
not the declaration.
2017-03-12 17:37:07 -04:00
Lioncash
26e9c54886
DSPEmitter: Make helper functions private
...
Given none of these are used outside of the DSPEmitter class (nor does
it really make sense to allow them to be used outside of the class),
these should all be made private.
2017-03-11 20:33:07 -05:00
Mat M
71273c05a9
Merge pull request #5054 from leoetlino/es-get-contents
...
IOS/ES: Implement ES_GetStoredContents ioctlvs properly
2017-03-11 10:52:53 -05:00
Mat M
3d041fa4ea
Merge pull request #5055 from leoetlino/es-ticket-view-fix
...
IOS/ES: Fix GetViews to not check for the TMD
2017-03-11 10:50:50 -05:00
Lioncash
b61ce51d3b
IOS/Network/IP/Top: Separate behaviors into their own functions
...
Localizes behavior and separates the code out from the IOCtl/IOCtlV
switch blocks.
2017-03-09 20:58:45 -05:00
Lioncash
40cdd32760
GCMemcard: Mark class constructors explicit where applicable
...
Prevents potential implicit instantation.
2017-03-08 19:11:16 -05:00