Project

General

Profile

Actions

Emulator Issues #6757

closed

[Pull Request] (eventually) JPeterson's Motion Plus

Added by webgeek1234 about 11 years ago.

Status:
Won't fix
Priority:
Normal
Assignee:
-
% Done:

0%

Operating system:
N/A
Issue type:
Bug
Milestone:
Regression:
No
Relates to usability:
No
Relates to performance:
No
Easy:
No
Relates to maintainability:
No
Regression start:
Fixed in:

Description

https://forums.dolphin-emu.org/Thread-to-pass-the-request-for-wiimotion?pid=298660#pid298660

Nothing has changed since the patch I posted in the linked post (yes, I'm Steel01 on the forum). So, it's still no where near review worthy. But since I didn't have the brain/will power to work on this further tonight, I'm adding this issue as Jhonn suggested. I won't be around my development machine for another few days. I expect that someone with input subsystem knowledge could clean this up (and find what I missed) in fairly short order. I do have a couple comments to add to the three I posted in that post.

  1. I've been playing Tales of Symphonia: Dawn of the New World with my patched version for several hours and haven't had a problem with the basic wiimote controls.

  2. I noticed when fiddling with the settings that swing always goes down and/or right, even when trying to go up and/or left. Think I broke a sign check in the merge somewhere.

All development and testing has been done on a Fedora 19 x86_64 machine. I can make no comments on how this affects a Windows build. But since no files were added or removed and I didn't see any ifdefs, I doubt it will affect any arch/platform differently.

Actions #1

Updated by NeoBrainX about 11 years ago

  • Status changed from New to Won't fix

Yeah, we're not interested in cleaning up JP's mess.

Actions #2

Updated by NeoBrainX about 11 years ago

Maybe let me elaborate a bit more on this:
If anyone from the development team had been interested in cleaning up JP's patches, it would've already been done by now regardless of any merging or build issues since these are (in comparison) trivial to fix. There's no point of having this pull request in our issue tracker since it does serve no purpose for the development team and any external contributor will never come across this issue anyway.

Actions

Also available in: Atom PDF