GobySoft / goby3

The Goby Underwater Autonomy Project Version 3
Other
24 stars 11 forks source link

Prevent the QueueManager from adding user frames over the max_frame_b… #290

Closed edsanville closed 1 year ago

edsanville commented 1 year ago

…ytes limit

tsaubergine commented 1 year ago

I don't think there is a bug in QueueManager that is adding user frames over the limit? I've fixed the bug in DynamicBuffer (https://github.com/GobySoft/goby3/pull/292) (DynamicBuffer is essentially a templated re-write of QueueManager and is what gobyd uses) that I think relates to the issue you may be trying to fix here.

If you think there's a problem in QueueManager's logic, please write a simple test case that exhibits it.