-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Please manage your code submission and don't make changes without testing #12266
Comments
I'm curious which company you are from? Bouffalo? Do you have any specific questions about the points you mentioned above? |
@sakumisu although some points your raised need to be improved (i.e. we don't have functional test integrated with our CI), the root causes you defined are not true or correct. Please enlighten me: which mainline are you submitting those patches, because I failed to find them here in this mainline: https://github.com/apache/nuttx/commits?author=sakumisu So, before pointing fingers and blame existent contributors, please do your part and have a positive mind and action.
|
Thank you for your quickly answer. |
And when i search some codes, something is found, like this IOB_SECTION macro, did you think it should be CONFIG_IOB_SECTION? This is why i said no code review, you know, i care little about nuttx low level code, this is just a coincidence. |
@sakumisu many people (Samsung inclusive) decide to use lwIP because they think the NuttX NET stack is slow, actually it is not. By default the performance is not so good because all default parameters are focused on board with low amount of memory. You need to modify the network parameters to get better performance. |
Yes, I think you found a BUG: @xiaoxiang781216 please take a look: mm/iob/iob_initialize.c:#ifdef IOB_SECTION It should be CONFIG_IOB_SECTION Thank you very much @sakumisu !!! |
Yes, i did. Newer version can do better than before.Feel sorry that i study a little about nuttx net, i will continue to do. |
Many commits in a mini version can make others feel worried and confused about whether it i s stable, many people do not want too many changes.They will use old version but newer can do better, we should cost time to convince them. |
That's all, thank you sir for your answer again. |
Have you been reading the code seriously? The definition of IOB_SECTION in the makefile file, |
So you think this codestyle that you like and are proud of is excellent? You think that's what everyone's going to be paying attention to? Why can't it be simple? |
You can do it now, immediately, PLEASE! There is unnecessary to discuss further if you just complain! |
When we use nuttx code in our projects, i found many problems.
All of these are my suggestions, a little rude but useful, thank you, guys.
The text was updated successfully, but these errors were encountered: