Skip to content

fix: BDSM barrel drop crash on servers#786

Merged
ACGaming merged 2 commits intoACGaming:mainfrom
bruberu:main
Jan 20, 2026
Merged

fix: BDSM barrel drop crash on servers#786
ACGaming merged 2 commits intoACGaming:mainfrom
bruberu:main

Conversation

@bruberu
Copy link
Contributor

@bruberu bruberu commented Jan 19, 2026

This has a patch for Funwayguy/BarrelsDrumsStorageAndMore#25.

To reproduce the original glitch, turn on InControl/BDS&M, fill a barrel completely, and then spam click items out until the server crashes.

@what-the-diff
Copy link

what-the-diff bot commented Jan 19, 2026

PR Summary

  • New Dependency Added for "Barrels, Drums, Storage & More" Mod
    The dependencies.gradle file now includes a reference to a new support module. This change indicates that our project relies on features in this mod to function.

  • New Debugging for "bdsandm"
    The gradle.properties file now has a new debugging setting concerning "bdsandm". This means developers have a new tool to troubleshoot issues.

  • New Configuration Settings for the Mod
    Within UTConfigMods.java, a new section is introduced for tailoring "Barrels, Drums, Storage & More" mod configurations. This update adds an important setting related to thread safety, ensuring the safe execution of tasks.

  • Enhanced Thread Safety with Server Handler Mixin
    The new UTServerHandlerMixin.java file includes a "mixin" - a piece of code that helps improve the thread safety of the network operations associated with the "bdsandm" mod. This enhances the reliability of these operations.

  • Mixin Loader Update
    The UTMixinLoader.java file now recognizes the new "bdsandm" mixin, making it part of the toolkit we use for enhancing our codebase.

  • Localization Support for Mod
    Added entries in en_us.lang to offer English language support for this new mod, making it user-friendly for English speaking players.

  • New Mixin Configuration File
    The new mixins.bdsandm.json file helps register the new mixin class for this mod. This ensures the new mixin's successful integration with the rest of the project.

@ACGaming ACGaming merged commit 0d3effc into ACGaming:main Jan 20, 2026
1 check passed
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