Hi!
This is probably super out of time and I’m being picky, but shouldn’t the IsBarrelMoving method be named different?
I mean, to tell wether the barrel is moving or not seems to be logically a responsibility of the barrel (thinking on terms of code, not of feasibility with what we have).
What about something more like “IsAimLocked()” which will be related to aim (AimComponent) and will hide the “how” we do it…