Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

convert to C #16

Open
wants to merge 19 commits into
base: master
Choose a base branch
from
Open

convert to C #16

wants to merge 19 commits into from

Conversation

rofl0r
Copy link

@rofl0r rofl0r commented Apr 4, 2022

No description provided.

rofl0r added 2 commits April 4, 2022 22:00
it's ridiculous to add dependency on a C++ compiler and the complexity
of a mixed build for the sake of using a single class and new/delete
operators.
@mckaygerhard
Copy link

This is a good one @kallisti5 !

@mckaygerhard
Copy link

Usin standar well know makefile is better rather than a scons dependency that pull to install more things in my linux

rofl0r added 7 commits April 5, 2022 13:36
i tried like 5 minutes to get a list of command line options using
the regular --help, -? and what have you, but didn't figure out
the magic incantation "-h" on my own.
for parameters requiring an argument when none was passed.
the 24bit, surface-based, directly-using-opengl code resulted in
huexpress running at 100% cpu load at about 30% native speed on
my machine. using SDL2's hardware-accelerated texture rendering
primitives instead (with 32bit textures), it now runs at 20%
cpu use at full speed.
this makes it possible to override settings from the ini file
from the command line.
this will also be used if the logfile fails to open.
@rofl0r
Copy link
Author

rofl0r commented Apr 5, 2022

added some commits to make rendering performance 15x faster, and fix the command line parser.

@kallisti5
Copy link
Owner

I'm not really interested in moving away from SCons. Lots of upstream packages / recipes are written around us using SCons. Could the makefile change be opened into a separate PR to discuss?

Everything else looks fine.

@rofl0r
Copy link
Author

rofl0r commented Apr 5, 2022

@kallisti5 i updated the SCons scripts and added a note to INSTALL that the makefile isn't supported, so imo it can be kept as a bonus but users that prefer scons know that it'll work and that's what's supposed to be used.
does this sound reasonable ?

rofl0r added 6 commits April 5, 2022 19:41
additionally to F9, which was used in the past, although it probably
never worked anyway, at least since the SDL2 move.
this happened when loading an iso from a .zip while no system card
roms were present in the paths searched.
PCEngine_Run() will cause accesses to uninitialized structures
after PCEngine_LoadFile() returned error.
@mckaygerhard
Copy link

I'm not really interested in moving away from SCons. Lots of upstream packages / recipes are written around us using SCons. Could the makefile change be opened into a separate PR to discuss?

Everything else looks fine.

the problem with scons is that it depends of upstream and in some distros is not packaged.. also it changed with time to time.. currently i can build a piece of software with makefile with any version of.. not the same with scons or cmake if there's some change across versions

@rofl0r
Copy link
Author

rofl0r commented Apr 6, 2022

I'm not really interested in moving away from SCons

also this PR doesn't remove or deprecate scons, it simply adds the Makefile as an option - and since it's now documented as unofficial you won't have to maintain it.

rofl0r added 2 commits April 7, 2022 17:56
gcc10 defaults to -fno-common and this results in a link error.

this patch also happens to be in freebsd's hugo tree:
https://cgit.freebsd.org/ports/tree/emulators/hugo/files/patch-osd__sdl__gfx.c
src/osd_keyboard.c:450:29: warning:
passing argument 1 of 'SDL_JoystickName'
makes pointer from integer without a cast [-Wint-conversion]
@mckaygerhard
Copy link

@rofl0r the makefile works like a baby.. .. about compilation it compiles in any machine.. i was using older debian wheeze

i tested and found a minimal detail.. when any rom started.. there's a peak of sound at start tht almost break my ears (with hearphones) i dont know if there's a inheric detail or a consecuence,

imagen

@rofl0r
Copy link
Author

rofl0r commented Apr 7, 2022

when any rom started.. there's a peak of sound at start tht almost break my ears

i experienced this too when i started hacking on huexpress (so it seems to be an old issue)

but it disappeared for me after b17df90 because before that commit huexpress was only running at like 20% speed so the sound was heavily distorted.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants