View Issue Details

IDProjectCategoryView StatusLast Update
0001546OpenMPTUser Interfacepublic2022-01-09 14:59
ReporterLachesis Assigned ToSaga Musix  
PrioritylowSeveritytweakReproducibilityalways
Status assignedResolutionopen 
Platformx64OSWindowsOS Version10
Product VersionOpenMPT 1.30.01.00 / libopenmpt 0.6.0 (current stable) 
Summary0001546: Minor raw sample export/import issues.
Description

Putting several minor issues/complaints regarding raw samples here because they aren't really worth their own individual reports.

  • Sample export consistently forgets the selected file format was raw, every single time. Export a sample as raw and then open the sample export dialog again. FLAC will now be selected. This seems to be a clamping bug as S3I export is also affected. If it is intentional, it's still bad, because 99.9% of the time I use raw export I am exporting samples for software that does not support WAV or FLAC.
  • OpenMPT coerces the ".raw" extension when exporting raw, which adds an annoying extra step of manually removing that extension when generating samples for software that uses ".sam" or no extension. Example: old RISC OS versions have a 10 character filename limit and no concept of filename extensions. It would be nice to have some way for it to use whatever extension (or lack thereof) was entered, but I don't have any good suggestions for how.
  • Raw import by default similarly does not recognize ".sam" or raw samples without extensions (ST-01 et al...). This one isn't that bad because it does temporarily remember when "All Files" was selected.

My use case for raw samples is mainly testing very old junk so this issue is about as low priority as it gets...

TagsNo tags attached.
Has the bug occurred in previous versions?Yes
Tested code revision (in case you know it)

Activities

Saga Musix

Saga Musix

2022-01-09 14:59

administrator   ~0004982

I think I should explain one thing upfront which might clear up some of the confusion: OpenMPT tries to be smart (maybe too smart?) about which sample format it offers by default in the dialog, which is not always the one chosen in the configuration dialog: After loading a file, OpenMPT will deduce the format to use from its filename, but only (until now) if it ends in .wav or .flac (as of r16474 it also takes S3I into account). I'd be hesistant to add .raw to that logic because I think that most people, after loading a raw sample, wouldn't want to keep the sample in raw format, but rather convert it to WAV and FLAC, so that they can keep the middle-C frequency, loop points, etc... I totally get your use case but I think that defaulting to RAW format in this case would be dangerous (many people don't even realize how they could change the format from the dropdown in the Save As dialog...). Note that this logic no longer applies if a sample has been saved once, as then OpenMPT knows its new filename.

r16474 does two more things: It adds .sam to the list of raw sample extensions, and adds S3I to the list of default formats in the config dialog.

OpenMPT coerces the ".raw" extension when exporting raw

This is essentially a bit of a workaround for some of Windows' own smartness when it comes to deducing the correct filename. IIRC when you manually enter a filename but leave out the file extension, Windows will not always add the correct file extension when hitting the Save button, if the filename looks like something that already has a valid file extension. This again can cause a lot of confusion because now your sample that you just saved doesn't show up in the sample browser because it uses a bogus file extension.

I'm open to suggestions how this could be improved but I wouldn't want to hurt the default behaviour for users without this specific usecase.

Issue History

Date Modified Username Field Change
2022-01-09 04:58 Lachesis New Issue
2022-01-09 14:24 Saga Musix Assigned To => Saga Musix
2022-01-09 14:24 Saga Musix Status new => assigned
2022-01-09 14:59 Saga Musix Note Added: 0004982