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

Import changes from Gargoyle #7

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

cspiegel
Copy link
Contributor

  • Multimedia code is now Glk agnostic instead of requiring Gargoyle (but see below).
  • GARGLK was effectively being used as a proxy for Unix Glk; a new HUGO_GLKUNIX macro is now tested so all Unix Glks can be used.
  • Sound loading code has been consolidated to avoid duplication.

The multimedia loading is still idiosyncratic: Hugo doesn't use Blorb, so workarounds have to be done. If GLK_MODULE_GARGLK_FILE_RESOURCES is available, resources are loaded directly from Hugo's resource files. Otherwise, the rather messy method of writing out PIC and SND files as described in the Blorb specification §16 is used. If a particular Glk implementation supports neither of these, it won't get sound and graphics.

* Multimedia code is now Glk agnostic instead of requiring Gargoyle (but
  see below).
* GARGLK was effectively being used as a proxy for Unix Glk; a new
  HUGO_GLKUNIX macro is now tested so all Unix Glks can be used.
* Sound loading code has been consolidated to avoid duplication.

The multimedia loading is still idiosyncratic: Hugo doesn't use Blorb,
so workarounds have to be done. If GLK_MODULE_GARGLK_FILE_RESOURCES is
available, resources are loaded directly from Hugo's resource files.
Otherwise, the rather messy method of writing out PIC and SND files as
described in the Blorb specification §16 is used. If a particular Glk
implementation supports neither of these, it won't get sound and
graphics.
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.

1 participant