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

Update aos_transport.c #41

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

Update aos_transport.c #41

wants to merge 28 commits into from

Conversation

JingGe
Copy link

@JingGe JingGe commented Jul 9, 2020

Replace the pintf long format with long long.

Background info: got the following errors while making the project:
error: format specifies type 'long'
but the argument has type 'int64_t' (aka 'long long') [-Werror,-Wformat]
header = apr_psprintf(t->pool, "Content-Length: %" APR_INT64_T_FMT, t->req->body_len);

shabicheng and others added 28 commits December 27, 2017 11:03
[modify] force rebuild when SEND_TIME_ERROR
[fix] missing init producer with 0
[add] add log_producer_client_get_flush_thread_id for linux
[add] add interface&send_timeout&destroy_timeout config for producer
[add] add log_producer_client_add_raw_log_buffer
[add] add LOG_PRODUCER_SEND_EXIT_BUFFERED
[modify] refine readme
[modify] refine max buffer size update logic
[fix] can't shutdown when sub manager is error
解决 ”sender线程池销毁最大等待时间“ 配置使用错误问题。
[modify] change default compile mode to Relese
Replace the pintf long format with long long.
 
Background info: got the following errors while making the project: 
error: format specifies type 'long'
      but the argument has type 'int64_t' (aka 'long long') [-Werror,-Wformat]
        header = apr_psprintf(t->pool, "Content-Length: %" APR_INT64_T_FMT, t->req->body_len);
@CLAassistant
Copy link

CLAassistant commented Jul 9, 2020

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
0 out of 4 committers have signed the CLA.

❌ zhinong.ly
❌ 666charles
❌ shabicheng
❌ JingGe


zhinong.ly seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

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.

4 participants