midou/citra
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

hid: Get rid of undefined behavior

Browse Source 
Modifying a variable twice across a sequence point.
This commit is contained in:
Lioncash 2015-05-27 12:20:09 -04:00
parent ad883db7a9
commit a3cad6c332
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/core/hle/service/hid/hid.cpp
View File

@ -58,7 +58,7 @@ void Update() {
mem->pad.current_state.hex = state.hex; mem->pad.current_state.hex = state.hex;
mem->pad.index = next_pad_index; mem->pad.index = next_pad_index;
++next_touch_index %= mem->pad.entries.size(); next_touch_index = (next_touch_index + 1) % mem->pad.entries.size();
// Get the previous Pad state // Get the previous Pad state
u32 last_entry_index = (mem->pad.index - 1) % mem->pad.entries.size(); u32 last_entry_index = (mem->pad.index - 1) % mem->pad.entries.size();
@ -88,7 +88,7 @@ void Update() {
} }
mem->touch.index = next_touch_index; mem->touch.index = next_touch_index;
++next_touch_index %= mem->touch.entries.size(); next_touch_index = (next_touch_index + 1) % mem->touch.entries.size();
// Get the current touch entry // Get the current touch entry
TouchDataEntry* touch_entry = &mem->touch.entries[mem->touch.index]; TouchDataEntry* touch_entry = &mem->touch.entries[mem->touch.index];