Skip to content

Issue 323#410

Open
rzhu15 wants to merge 3 commits intodevelopfrom
issue-323
Open

Issue 323#410
rzhu15 wants to merge 3 commits intodevelopfrom
issue-323

Conversation

@rzhu15
Copy link
Copy Markdown

@rzhu15 rzhu15 commented Apr 9, 2026

Summary

Implements the TODO at line 289 of MediaClipboardWidget.h.

Previously, attempting to send a file to a DAW-linked track of a
different format would show a "File Type Mismatch" error with no
recovery path. This change attempts format conversion using JUCE's
built-in AudioFormatManager before falling back to the error dialog.

Supported conversions

  • WAV ↔ AIFF (.aif / .aiff)
  • WAV ↔ FLAC
  • AIFF ↔ FLAC

Implementation notes

  • No new dependencies — uses juce::juce_audio_formats which is
    already linked in CMakeLists.txt
  • Conversion writes to a temp file first, then copies over the
    original to avoid corrupting the DAW file on failure
  • Temp file is cleaned up after copy regardless of outcome

Richard Zhu and others added 3 commits April 8, 2026 23:11
When the selected file's format doesn't match the DAW-linked
file's format, attempt conversion using JUCE's AudioFormatManager
rather than showing an error. Supports WAV, AIFF (.aif/.aiff),
and FLAC.
When the selected file's format doesn't match the DAW-linked
file's format, attempt conversion using JUCE's AudioFormatManager
rather than showing an error. Supports WAV, AIFF (.aif/.aiff),
and FLAC.
@cwitkowitz cwitkowitz mentioned this pull request Apr 9, 2026
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.

2 participants