Pokopom icon indicating copy to clipboard operation
Pokopom copied to clipboard

Please add DEmul compatibility.

Open Nucleoprotein opened this issue 9 years ago • 41 comments

As topic says - please add DEmul compatibility :p

Nucleoprotein avatar Oct 10 '15 23:10 Nucleoprotein

Sure, why not.

KrossX avatar Oct 11 '15 02:10 KrossX

Dang, that was hard. I only tested Rez, and rumble seems to work at least.

KrossX avatar Oct 28 '15 00:10 KrossX

Thanks, will test ASAP.

PS: typo in filename - Pokopom/playstation_dualschock2.cpp

Nucleoprotein avatar Oct 28 '15 13:10 Nucleoprotein

Fixed! =D

KrossX avatar Oct 28 '15 19:10 KrossX

Is this fully working and done? You can close the isssue if it is, and if it is thanks a bunch I really like Pokopom's support for so many emulators! :smiley:

DerekTurtleRoe avatar Nov 12 '15 03:11 DerekTurtleRoe

No~~~~ not "fully working", just barely working. Definitely not done either as my reverse engineering skills are quite limited.

KrossX avatar Nov 12 '15 05:11 KrossX

Oh, well the code is in the repo, right? So that means we can test it? If so, I will test it if you want. :+1:

Thanks for the quick port, at least. Amazing work!

DerekTurtleRoe avatar Nov 12 '15 05:11 DerekTurtleRoe

As always, use at your own risk. I kinda know the barebones state of the code, but is hard to improve without source code to know the emulator API.

KrossX avatar Nov 12 '15 14:11 KrossX

So, how did you add support for DEmul if they don't have an open API? :open_mouth:

DerekTurtleRoe avatar Nov 12 '15 23:11 DerekTurtleRoe

That's called reverse engineering.

KrossX avatar Nov 13 '15 00:11 KrossX

Oh, I didn't think you would go to such lengths to just add input support for DEmul. :stuck_out_tongue_closed_eyes:

DerekTurtleRoe avatar Nov 13 '15 05:11 DerekTurtleRoe

I didn't know either. It's all tapcio's fault.

KrossX avatar Nov 13 '15 15:11 KrossX

:trollface:

Nucleoprotein avatar Nov 13 '15 15:11 Nucleoprotein

But hey, it works! :stuck_out_tongue_winking_eye:

DerekTurtleRoe avatar Nov 13 '15 21:11 DerekTurtleRoe

@KrossX Did you see this?

https://github.com/p1pkin/demul

DerekTurtleRoe avatar Dec 22 '15 08:12 DerekTurtleRoe

This is too old - they changed ABI in latest versions. Latest padDemul.dll exports only getType, getName and getInterface.

Nucleoprotein avatar Dec 22 '15 10:12 Nucleoprotein

That's from 0.5.x I think and from what I remember, it didn't even work as a reference. =_="

KrossX avatar Dec 22 '15 16:12 KrossX

@tapcio @KrossX Sounds really confusing if you are trying to develop a plugin. :laughing:

Did you guys know the source was available before that repo was created? The Google Code page never had source as far as I know, so I'm not sure where it was hosted, but apparently I had been supid and it was available? shrug

DerekTurtleRoe avatar Dec 22 '15 22:12 DerekTurtleRoe

That old source was about the first thing I found.

I think other people making plugins is of no importance to that project. It might be even better for them to just merge the hole thing instead of having plugins to begin with.

KrossX avatar Dec 22 '15 23:12 KrossX

Oh, I guess I didn't Google hard enough? :hear_no_evil:

Yeah, DEmul seems pretty good all by itself, not sure why you would want to add a plugin architecture to the mix. Always causes more problems than solves.

Although, your plugin is pretty great. I use it for all my emulators (except Project64 sometimes for emulation of N64DD peripherals and such).

Anyway, are you saying the source is older than what they are actually releasing? If they haven't released the source to the alpha versions, I can understand that, just because they want to test it first I guess.

DerekTurtleRoe avatar Dec 23 '15 00:12 DerekTurtleRoe

That source code is for Demul 5.x circa 2013, so yup, old source code is old.

Other than that, there could be many reasons why they might not want to open source it now or at all. Maybe that old source code was the test, and maybe that test failed.

KrossX avatar Dec 23 '15 03:12 KrossX

@KrossX Alright, I am talking to them about it now.

I also am trying to get them to release the plugin API, so hopefully you can have some more to go on soon. :smile:

DerekTurtleRoe avatar Dec 23 '15 04:12 DerekTurtleRoe

Thanks, that could help quite a bit.

I wouldn't bug them too much though, as even just a header would be extra work to do.

KrossX avatar Dec 23 '15 05:12 KrossX

@KrossX

even just a header would be extra work to do

More work for you or them? :laughing:

DerekTurtleRoe avatar Dec 23 '15 05:12 DerekTurtleRoe

I wouldn't mind making it but that, would require source access to begin with.

KrossX avatar Dec 23 '15 14:12 KrossX

@KrossX OK. I am still chatting with them about their plans. :speak_no_evil:

DerekTurtleRoe avatar Dec 24 '15 08:12 DerekTurtleRoe

DEmul x86 v0.7 Alpha BUILD 221215 crashes with Pokopom :disappointed:

Nucleoprotein avatar Jul 12 '16 22:07 Nucleoprotein

Done! Seems to be working again with the new commit. It seems they added a few new variables to their big structure. No build though so you'll have to make one for yourself, sorry about that.

KrossX avatar Jul 13 '16 19:07 KrossX

Thanks! :+1:

Nucleoprotein avatar Jul 13 '16 21:07 Nucleoprotein

Is demul now better than nulldc? Is nulldc support ever coming back?

shinra358 avatar Jul 23 '16 20:07 shinra358