diff options
| author | skal <pascal.massimino@gmail.com> | 2026-02-05 20:13:46 +0100 |
|---|---|---|
| committer | skal <pascal.massimino@gmail.com> | 2026-02-05 20:13:46 +0100 |
| commit | d55c13a902d4277e35010574a6cee23f65dd6a55 (patch) | |
| tree | 3066292d315983f1655716b68f76ec47ad651a70 /doc/HOWTO.md | |
| parent | 3b201f2cb4d5917bb2fd76051c073b71ad6b3b27 (diff) | |
feat(audio): Complete Phase 4 - Cleanup and Documentation (Task #56)
Completed final cleanup phase of Audio Lifecycle Refactor. Removed backwards
compatibility shims and updated documentation to reflect new AudioEngine-based
initialization patterns.
Changes:
1. Removed Backwards Compatibility:
- Removed synth_init() call from audio_init() in audio.cc
- Added comment explaining AudioEngine is the preferred initialization method
- All tests already explicitly call synth_init() or use AudioEngine
2. Documentation Updates:
- Updated HOWTO.md with AudioEngine usage examples and best practices
- Updated CONTRIBUTING.md with audio subsystem initialization protocols
- Documented when to use AudioEngine vs direct synth API calls
- Clarified that AudioEngine is a lifecycle manager, not a complete facade
3. Size Verification:
- Size-optimized build: 5.0MB (vs 6.2MB debug)
- AudioEngine overhead: <500 bytes (within acceptable limits)
- No size regression from refactor
Results:
- All 20 tests pass (100% pass rate)
- Demo runs successfully
- No backwards compatibility issues
- Clear documentation for future development
- Binary size impact negligible
Design Philosophy:
- AudioEngine manages initialization order (synth before tracker)
- Direct synth API calls remain valid for performance-critical paths
- Low-level tests can still use synth_init() directly if needed
- Preferred pattern: Use AudioEngine for lifecycle, direct APIs for operations
handoff(Claude): Completed Task #56 Phase 4 - All phases complete! Audio
Lifecycle Refactor is fully implemented, tested, and documented. The
fragile initialization order dependency has been eliminated.
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Diffstat (limited to 'doc/HOWTO.md')
| -rw-r--r-- | doc/HOWTO.md | 59 |
1 files changed, 59 insertions, 0 deletions
diff --git a/doc/HOWTO.md b/doc/HOWTO.md index e562679..5af3f05 100644 --- a/doc/HOWTO.md +++ b/doc/HOWTO.md @@ -6,6 +6,7 @@ This document describes the common commands for building and testing the project * **Real-time Audio Synthesis**: The demo features a multi-voice synthesizer that generates audio in real-time from spectrograms. * **Dynamic Sound Updates**: Spectrograms can be updated dynamically and safely during runtime for evolving soundscapes. +* **Unified Audio Engine**: The `AudioEngine` class manages audio subsystem initialization, ensuring correct setup order and eliminating initialization fragility. ## Building @@ -62,6 +63,64 @@ if you have the public ssh key authorized on the VPS, you can use to clone the repo and work on it. +## Audio System + +### AudioEngine + +The audio subsystem uses `AudioEngine` to manage initialization and lifecycle. This ensures correct setup order and eliminates initialization fragility. + +**Usage in production code:** + +```cpp +#include "audio/audio_engine.h" + +// Initialize audio backend (miniaudio) +audio_init(); + +// Initialize audio engine (manages synth + tracker) +static AudioEngine g_audio_engine; +g_audio_engine.init(); + +// In main loop: update with music time +g_audio_engine.update(music_time); + +// Cleanup +g_audio_engine.shutdown(); +audio_shutdown(); +``` + +**What to use AudioEngine for:** +- Initialization: `engine.init()` replaces separate `synth_init()` + `tracker_init()` calls +- Updates: `engine.update(music_time)` replaces `tracker_update()` +- Cleanup: `engine.shutdown()` ensures proper teardown +- Seeking: `engine.seek(time)` for timeline navigation (debug builds only) + +**Direct synth API usage:** +For performance-critical or low-level operations, direct synth API calls are valid: +- `synth_register_spectrogram()` - Register audio samples +- `synth_trigger_voice()` - Trigger sound playback +- `synth_get_output_peak()` - Get audio peak for visualization +- `synth_render()` - Low-level audio rendering + +**Testing:** +Tests should use `AudioEngine` for initialization: + +```cpp +#include "audio/audio_engine.h" + +void test_example() { + AudioEngine engine; + engine.init(); + + // Test code here + engine.update(1.0f); + + engine.shutdown(); +} +``` + +For low-level synth-only tests, you can still call `synth_init()` directly. + ## Debugging ### Seeking / Fast-Forward |
