Skip to content

fix: remove non-existent BASS_Free import from sound_lib#3

Closed
Orinks wants to merge 1 commit intomainfrom
fix/bass-free-import
Closed

fix: remove non-existent BASS_Free import from sound_lib#3
Orinks wants to merge 1 commit intomainfrom
fix/bass-free-import

Conversation

@Orinks
Copy link
Owner

@Orinks Orinks commented Feb 1, 2026

Summary

Fixes #2

The BASS_Free symbol doesn't exist in sound_lib.main. This caused an ImportError that prevented all tests from even collecting.

Changes

  • Remove unused sound_lib.main import (BassError, bass_call_0, BASS_Free)
  • Remove BASS_Free() call in cleanup() - sound_lib handles cleanup automatically when the Output object is garbage collected

Testing

  • Tests now collect successfully (previously 2 collection errors)
  • Audio tests fail on headless server as expected (no audio device)

Fixes #2

The BASS_Free symbol doesn't exist in sound_lib.main. The previous
import caused an ImportError that prevented all tests from running.

Changes:
- Remove unused sound_lib.main import (BassError, bass_call_0, BASS_Free)
- Remove BASS_Free() call in cleanup() - sound_lib handles cleanup
  automatically when the Output object is garbage collected
Base automatically changed from dev to main February 1, 2026 21:43
@Orinks
Copy link
Owner Author

Orinks commented Feb 4, 2026

The BASS_Free import fix in PR #4 targeted the old directory structure. The current codebase still has it — will fix separately.

@Orinks Orinks closed this Feb 4, 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.

Fix: BASS_Free import error breaks tests

1 participant