Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fixed bug : buffer don't have enough space #23

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

anchuanxu
Copy link

1:Solve the problem of buffer overflow when PCAP data is large;

2:Thread locks and condition variables are used to balance push and POP.

3:Based on the@iurii-provizio Work to develop and to 909a3bc#diff-66 ae3288ae851a5f0eb11df04565dd2c9de744f85221e3dc9870e1537e046a9e

4:Thanks to @Luffy.AN for your help

@iurii-provizio
Copy link

@anchuanxu seems reasonable, but there is no finish condition: in case something waits for a conditional variable, we can't destroy (and can't finish the parent app f.e.).

@KevinReny
Copy link

我想在HesaiLidar_General_ROS这个工程里面使用,但是您所改的文件是SDK中的。然而 HesaiLidar_General_ROS里面的SDK文件夹与这个单独的SDK不一样,直接进行替换似乎不行。我该如何解决这个问题呢

muratams added a commit to CMU-cabot/HesaiLidar_General_ROS that referenced this pull request Jun 11, 2024
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.

None yet

3 participants