Skip to content

Commit

Permalink
Change engine pack size message back to debug
Browse files Browse the repository at this point in the history
I think this is a good standard to follow for the rest of the size-related messages. They should have bounds checking which raises an exception if it fails, and debug logs which print out the size in case you need to know how much space is left.
  • Loading branch information
charasyn committed Apr 7, 2024
1 parent cda70b3 commit 9c36b59
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion coilsnake/model/eb/musicpack.py
Original file line number Diff line number Diff line change
Expand Up @@ -897,7 +897,7 @@ def save_to_parts(self) -> None:
raise InvalidUserDataError("Data for engine pack ${:02X} is too long by {} bytes. "
"Maybe your \"in-engine\" songs are too large.".format(self.pack_num, overage))
# Have a helpful debug output for the user
log.info("Engine pack has %d bytes of free space available.", DYNAMIC_SONG_DATA_START - song_output_ptr)
log.debug("Engine pack has %d bytes of free space available.", DYNAMIC_SONG_DATA_START - song_output_ptr)
output_parts += always_loaded_song_parts

# Get dynamically loaded song data that is in this pack (Gas Station 1 in vanilla)
Expand Down

0 comments on commit 9c36b59

Please sign in to comment.