View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0000925 | OpenMPT | libopenmpt | public | 2017-03-14 15:36 | 2023-01-24 12:02 |
Reporter | Saga Musix | Assigned To | manx | ||
Priority | normal | Severity | feature | Reproducibility | N/A |
Status | acknowledged | Resolution | open | ||
Target Version | OpenMPT 1.32 / libopenmpt 0.8 (goals) | ||||
Summary | 0000925: Support for external samples in libopenmpt | ||||
Description | libopenmpt does currently not support sample data (or any other kind of data) outside the user-provided module file. There are several multi-file formats supported by OpenMPT:
In the future, we may also support:
Apart from the fact that someone may simply want to play such modules using a libopenmpt-based player, there is also the possibility of someone wanting to use external samples e.g. for saving space when composing music for a game. There are also security implications when accessing external files, and the feature should probably be opt-in (and possibly fully backwards-compatible with previous library versions, e.g. by using a ctl value for providing the module file path). | ||||
Tags | No tags attached. | ||||
Has the bug occurred in previous versions? | |||||
Tested code revision (in case you know it) | |||||
When adding support for file I/O inside libopenmpt anyway, we can also extend the API to to specify a filename directly instead of having the API user do the file loading. The single reason for not supporting that right now, is the lack of all direct file I/O inside libopenmpt. A ctl for the existing functions might also be useful though. Not sure about that yet. |
|
Another possibility might be to have libopenmpt provide the list of external filenames to the application and have the application issue an explicit load_instrument or load_sample call after loading the module had already been loaded by libopenmpt. A variant of that approach would involve callbacks to the application while loading. However, that approach will be very difficult to implement in foreign language bindings. |
|
This might not be feasible with all supported module formats, because sometimes patch-ups are done after loading the external file, but before giving control back to the application. For example in the ITP format, we first load an instrument file and then may override some of the instrument properties if "Embed instruments' header" is used. We would have to be able to do these patch-ups after loading the file, which would complicate the code in many ways. I do not like that idea at all. |
|
From an implementation point-of-view, being able to use C++17 std::filesystem would simplify path traversal and related security issues a lot. |
|
OK, I had kind of expected such complications, thus this possibility is not on the table any more. |
|
We probably want at least 2 levels of external file loading:
With option 1 being the safer subset of option 2. |
|
Option 1 is also the more often seen and more useful variant. Option 2 is more likely to occur with unfinished / unreleased modules on an artist's computer. |
|
For platforms like Emscripten, Startrekker (and possibly TFMX in the future) support might be the most interesting case (for example for listening to Startrekker files on ModLand online). It's also the most peculiar case because we won't know in advance if the external instrument file will be present or not, and which file extension it will have. For a Startrekker module called foo.mod, the external instrument file could be named foo.mod.nt, foo.mod.NT, foo.mod.as or foo.mod.AS (ignoring mixed-case file extensions). It is also perfectly valid for this external instrument file to be missing in case no synthesized instruments are used. Emscripten can provide a virtual file system but as far as I understand, it won't help us when playing modules from an external source. Hence, a callback might be the most convenient way to handle this situation for Emscripten, even though it might send up to four bogus HTTP requests when loading a Startrekker module. Like in most other languages, Emscripten can use C callback functions. |
|
This topic has been brought up again because emoon's new HippoPlayer would have to support this in its own plugin API, and one if its default plugins uses libopenmpt.
In addition, any programming language that can interface in C should be able to use C callbacks (otherwise e.g. using the WinAPI would be more or less impossible). The C callback function signature could look like this: |
|
We alone would have to duplicate the security-conscious directory handling code at least twice in this case: once for openmpt123 and once for xmp-openmpt. We would not gain any ease of implementation either. Also, in the common use case of loading the files from a local file-system, this would unnecessarily complicate client code. APIs should be easy to use correctly and difficult to use incorrectly. Providing only a callback interface and offloading the responsibility to client code in this aspect would violate this API design principle. |
|
I'd rather defer this feature yet again, because there are realistic chances that we could depend on std::filesystem for libopenmpt 0.6. |
|
Date Modified | Username | Field | Change |
---|---|---|---|
2017-03-14 15:36 | Saga Musix | New Issue | |
2017-03-14 15:39 | manx | Target Version | => OpenMPT 1.?? (libopenmpt 1.0) (goals) |
2017-03-14 15:45 | manx | Note Added: 0002911 | |
2017-03-14 15:47 | manx | Note Added: 0002912 | |
2017-03-14 15:48 | manx | Assigned To | => manx |
2017-03-14 15:48 | manx | Status | new => assigned |
2017-03-14 15:48 | manx | Status | assigned => new |
2017-03-14 15:51 | Saga Musix | Note Added: 0002913 | |
2017-03-14 15:52 | manx | Note Added: 0002914 | |
2017-03-14 15:54 | manx | Note Added: 0002915 | |
2017-03-14 16:31 | manx | Note Added: 0002916 | |
2017-03-14 16:39 | Saga Musix | Note Added: 0002917 | |
2017-03-14 22:26 | Saga Musix | Description Updated | |
2017-03-20 12:07 | Saga Musix | Note Added: 0002921 | |
2017-08-27 09:27 | manx | Target Version | OpenMPT 1.?? (libopenmpt 1.0) (goals) => libopenmpt 0.4 (goals) |
2018-01-17 08:06 | manx | Relationship added | has duplicate 0001078 |
2018-01-29 21:04 | Saga Musix | Note Added: 0003400 | |
2018-01-29 21:06 | Saga Musix | Note Edited: 0003400 | |
2018-01-29 21:06 | Saga Musix | Note Edited: 0003400 | |
2018-01-30 16:19 | manx | Note Added: 0003401 | |
2018-02-14 20:09 | manx | Target Version | libopenmpt 0.4 (goals) => OpenMPT 1.29.01.00 / libopenmpt 0.5.0 (upgrade first) |
2018-08-02 15:07 | manx | Relationship added | parent of 0001135 |
2019-03-08 15:41 | manx | Note Added: 0003892 | |
2019-03-08 15:41 | manx | Target Version | OpenMPT 1.29.01.00 / libopenmpt 0.5.0 (upgrade first) => OpenMPT 1.?? (libopenmpt 1.0) (goals) |
2021-06-20 09:13 | manx | Relationship added | has duplicate 0001475 |
2021-06-20 11:08 | manx | Status | new => acknowledged |
2021-06-20 11:08 | manx | Target Version | OpenMPT 1.?? (libopenmpt 1.0) (goals) => OpenMPT 1.31.01.00 / libopenmpt 0.7.0 (upgrade first) |
2021-06-20 11:09 | manx | Relationship added | parent of 0001340 |
2021-11-16 19:01 | Saga Musix | Description Updated | |
2022-02-01 11:59 | Saga Musix | Description Updated | |
2022-06-25 17:18 | manx | Relationship deleted | parent of 0001340 |
2023-01-24 09:44 | manx | Target Version | OpenMPT 1.31.01.00 / libopenmpt 0.7.0 (upgrade first) => OpenMPT 1.?? (libopenmpt 1.0) (goals) |
2023-01-24 12:02 | manx | Target Version | OpenMPT 1.?? (libopenmpt 1.0) (goals) => OpenMPT 1.32 / libopenmpt 0.8 (goals) |