Do some light multitap cleanup

After seeing the regressions from multitap I took a quick look at the
code, and there's quite a bit of cleanup needed.

This fixes the initial really obvious stuff, a useless copy and a
duplicate in_keystate that can cause compilation issues, along with
bringing styling closer to the pcsx standard so it's a bit more readable.

The multitap changeset also seems to have been done with an editor that
mangled line endings, I'll address that in a followup commit.

Merge pull request #85 from alexis-puska/master_merged

Master merged

fix multitap and analog

Merge branch 'master' of https://github.com/libretro/pcsx_rearmed into master_merged

Merge pull request #83 from GregorR/libretro-git-version

Report git version with library_version

Report git version with library_version

Merge pull request #81 from joolswills/dynarec_enable

enable dynarec core - was hardcoded to off - fixes #78

enable dynarec core - was hardcoded to off - fixes #78

Merge pull request #76 from heronr/analog_range_fix

Fixed analog stick ranges to be [0, 255]

Fixed analog stick ranges to be [0, 255]

Merge pull request #75 from frangarcj/master

(VITA) Use RWX path

(VITA) Use RWX path

Backport drc: fix a mistake from w^x cahnge

Define NO_WRITE_EXEC for Android

Merge pull request #73 from frangarcj/master

Fix software buffer

Fix software buffer

(new_dynarec) Cleanups

(new_dynarec) Update

Merge pull request #72 from frangarcj/master

Add support for software framebuffer

Add support for software framebuffer

Update new_dynarec - working towards a common shared ari64
codebase for both mupen64plus and pcsx rearmed - see libretro/ari64

Update makefile

Rearrange files for new_dynarec

Merge pull request #71 from frangarcj/master

Optimizations and quit error

(Vita) Disable debug

(Vita) Fix deinit bug

(VITA) Enable O3 optimizations

Merge pull request #70 from d3m3vilurr/fix-undef-ref-deinit_vita_mmap

Fix build error about undefined reference to `deinit_vita_mmap'

Fix build error about undefined reference to `deinit_vita_mmap'

```
frontend/libretro.o: In function `retro_deinit':
libretro.c:(.text+0x3769): undefined reference to `deinit_vita_mmap'
collect2: error: ld returned 1 exit status
make: *** [pcsx_rearmed_libretro.so] Error 1
```
Introduced #68

Merge pull request #68 from frangarcj/master

(VITA) Dynarec working

(VITA) Dynarec working

Merge pull request #67 from frangarcj/patch-1

Fix block creation

Update libretro.c

Merge branch 'mainline' into libretro

partially revert c4052f4d79cf

See github issue #74 for details.

drc: some vita and 3ds support

not tested, mostly just guesswork

drc: avoid MAP_FIXED

it's almost never a good idea, it will override some other mapping
and make things crash

partially revert "(VITA) Some dynarec"

See github issue #74 for details.

Revert "(VITA) Another dynarec fix"

This reverts commit d9e6b42f224f10d8b6f2454da1f5d80735f8e0d6.
See github issue #74 for details.

Revert "(VITA) More dynarec fixes"

This reverts commit 1c2e3fc3baf93e7c17f678d0d187e9f424360bb2.
See github issue #74 for details.

Revert "redports backport"

This reverts commit f82bcc681184536aa6d3ee410b938671c1ba2ead.

drc: try to support w^x platforms like iOS

untested...

drc: remove unnecessary cache flushing

should already be flushed or not matter at this point

drc: enable and fix warnings

these were kept for compat with Ari64's code, which is removed, see
previous commit messages

drc: mark things static

drc: strip eol blanks

Because people send patches while using editors that strip blanks
and the patch becomes a mess.

drc: drop less obvious dead code

drc: drop heaps of dead code

I've kept it around to keep the code similar to Ari64's version,
so that it would be easier to merge back his fixes. However Mupen64plus
has long reformatted the code and it kind of went different direction
anyway, so there is no point to keep all this code now.

redports backport

Merge pull request #64 from orbea/debug

OPENGL_DEBUG is not needed

OPENGL_DEBUG is not needed

Merge pull request #61 from frangarcj/master

(VITA) I think this is not going to work but if someone can test it

Merge pull request #62 from orbea/debug

Use DEBUG=1 properly for debug builds

Use DEBUG=1 properly for debug builds

(VITA) I think that is not VM Memory and do some alignment

(VITA) More dynarec fixes

(VITA) Another dynarec fix

(VITA) Some dynarec

(Vita) Have to set DRC_CACHE_BASE to 0 for now

(Vita) Try to enable dynarec now

Merge pull request #55 from vanfanel/master

Please, don't use for loop initial declarations because it forces us …

Please, don't use for loop initial declarations because it forces us to use higher C std

Merge remote-tracking branch 'libretro/master'

Merge pull request #54 from Kekun/fix/set-controller-port-device

libretro: Split update_controller_port_device()

libretro: Split update_controller_port_device()

Split update_controller_port_device() into
update_controller_port_variable() updating the controllers from the
variables, update_controller_port_device() updating the controllers from
retro_set_controller_port_device() and controller_port_variable()
sharing getting the controller variable between the two.

Avoid setting to PSE_PAD_TYPE_NONE controllers whose variable is set to
'default' when updating them.

Merge pull request #53 from Kekun/fix/set-controller-port-device

Fix/set controller port device

libretro: Check 'environ_cb' in controller setters

Check that 'environ_cb' is valid before using it in update_multitap()
and update_controller_port_device() as they may be called from
retro_set_controller_port_device() before 'environ_cb' is set.

libretro: Fix key passing of update_controller_port_device()

Change the way the key of the controler type variables is passed to one
which doesn't crash.

Merge pull request #52 from Kekun/fix/set-controller-port-device

Fix/set controller port device

libretro: Implement retro_set_controller_port_device()

Add the 'default' value to controller type variables and set it as the
default value and implements the retro_set_controller_port_device()
function. When the variable's value is 'default', the value set via the
function is used, otherwise the value of the variable is used.

This allows the frontends to use retro_set_controller_port_device() or
the variables, pleasing both the users and frontends prefering the
standard API and the ones prefering the custom API.

libretro: Allow to auto detect multitaps

Add the 'auto' multitape vairable value.

This allows to automatically detect multitaps based on the number of
plugged controllers.

Add PSE_PAD_TYPE_NONE

Allow to set no controller to a port.

This will be used in the next commit to allow auto detecting whether
multitaps are plugged.

libretro: Refactor reading controller type

Refactor the way the controller type variables are read as we were doing
8 times almost exactly the same thing.

This makes the code more readable and maintainable.

Merge pull request #51 from alexis-puska/master

add rumble support for controler 1 and 2

comment some debugging printf.

comment some debugging printf.

libretro: clean up

libretro: fix a typo

libretro: try to prevent bad builds

libretro: try to prevent bad builds

Merge pull request #49 from meepingsnesroms/master

Fix compiling with ndk r11 >

Fix compiling with ndk r11 >

pad index fix

refactor some code + bug fix

fix compilation error

code refactor

Code refactor to build the response of request 0x42 read the pad key
state

Add dual shop request/response

Add dualshock request/response to declare Analog pad correctly and
retrieved the vibration information.

fixed lag with analog pad device

Fixed lag with analog pad device, some code is override when have a pad
in analog mode.

Merge pull request #47 from alexis-puska/master

Suppress segmentation fault and crazy integer value.

init default value for dfInput plugins

init default value for dfInput plugins

suppress hack for crazy value on read port 2

pass pad index in pad datas structure.

Merge pull request #39 from dankcushions/master

Added multitap 1 & 2 core options - adds 3-8 player support.

Style nits

Defensive code to avoid SEGFAULT

comment debugging printf

fix analog value

fix analog value

Init in_type fully

second implementation of multitap

add game/core option to enabled or disabled multitap.
Test with micro machine V3 and 5 devices, 2 multitap enabled, all
device react good i can’t test the 6 7 8 select player but i can select
8 gamer. The 2 multitap works fine.

i detect a bug, when we change the game pad type to analog, the library
crash, see it next time

Added multitap core option & style nits

Initial multitap support

Merge pull request #37 from loganmc10/master

Add rpi3 platform