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

若干小改进 #286

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

若干小改进 #286

wants to merge 8 commits into from

Conversation

akofer
Copy link

@akofer akofer commented Dec 31, 2022

详情见每一次提交的日志。

Your Name added 8 commits December 31, 2022 17:16
coost/test/tw.cc:320:21: warning: unused variable 't' [-Wunused-variable]
        const char* t;

coost/include/co/unitest.h:20:23: warning: field 'line' will be initialized after field 'c' [-Wreorder-ctor]
        : file(file), line(line), c(c), msg(std::move(msg)) {
The max num of file descriptors is 1024 by default on most linux dists. To
avoid open file failures, change the connection num to be less than 1024.
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

1 participant