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.

Sign up for GitHub

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

Jump to bottom

Start fixing build warnings #16

Merged
merged 3 commits into from Dec 24, 2013
Merged

Start fixing build warnings #16

merged 3 commits into from Dec 24, 2013

Conversation

remicollet
Copy link
Contributor

No description provided.

@remicollet
Fix src/network/ProcessPool.c:145:3: warning: implicit declaration of…
c9b6856 
… function 'wait' [-Wimplicit-function-declaration]

Fix ProcessPool.c:131:9: warning: unused variable 'writer_pti' [-Wunused-variable]
Fix ProcessPool.c:131:6: warning: unused variable 'i' [-Wunused-variable]
Fix src/network/ProcessPool.c:71:6: warning: 'ret' may be used uninitialized in this function [-Wmaybe-uninitialized]
@remicollet
Fix [-Wunused-variable]
e9218a6
@remicollet
Fix warning: passing argument 3 of 'swConnection_close' from incompat…
c2a19b6 
…ible pointer type [enabled by default]
@remicollet
Copy link
Contributor Author

There are really a lot of build warnings :(

Some are only cleanups (unused-variable), but other could raise unexpected behavior (implicit function declaration, ncompatble pointer type, ...)

I wil try to fix some of them if I found more time... And yes, I use -Wall flags.

matyhtf added a commit that referenced this pull request Dec 24, 2013
@matyhtf
Merge pull request #16 from remicollet/master
1c1ed64 
Merged. Thank you.
@matyhtf matyhtf merged commit 1c1ed64 into swoole:master Dec 24, 2013
@w3yyb w3yyb mentioned this pull request Jan 23, 2015
Closed
@gyf19 gyf19 mentioned this pull request May 28, 2015
Closed
@ghost ghost mentioned this pull request May 29, 2015
Closed
@AJSoft AJSoft mentioned this pull request Aug 20, 2015
Closed
@iyidan iyidan mentioned this pull request Dec 11, 2015
Closed
@eyehere eyehere mentioned this pull request Dec 25, 2015
Closed
@benzhan benzhan mentioned this pull request Jan 8, 2016
Closed
@lscgzwd lscgzwd mentioned this pull request Mar 16, 2016
Closed
@koolob koolob mentioned this pull request Mar 18, 2016
Closed
@jishipu jishipu mentioned this pull request Mar 19, 2016
Closed
@badxpgy badxpgy mentioned this pull request Apr 26, 2016
Closed
@koolob koolob mentioned this pull request May 18, 2016
Closed
@hppking hppking mentioned this pull request Jun 2, 2016
Closed
This was referenced Jun 3, 2016
Closed
Closed
@badxpgy badxpgy mentioned this pull request Jun 17, 2016
Closed
@WuRongzong WuRongzong mentioned this pull request Jul 16, 2016
Closed
@badxpgy badxpgy mentioned this pull request Aug 23, 2016
Closed
@badxpgy badxpgy mentioned this pull request Sep 13, 2016
Closed
@yongchuan yongchuan mentioned this pull request Sep 28, 2016
Closed
@wangdianwen wangdianwen mentioned this pull request Nov 2, 2016
Closed
@lscgzwd lscgzwd mentioned this pull request Nov 24, 2016
Closed
@jqhph jqhph mentioned this pull request Dec 19, 2016
Closed
@peze peze mentioned this pull request Dec 31, 2016
Closed
@zhang-bin zhang-bin mentioned this pull request Jan 25, 2018
Closed
@kakuilan kakuilan mentioned this pull request Mar 20, 2018
Closed
@jianwubayiba jianwubayiba mentioned this pull request May 5, 2018
Closed
@GargantuaX GargantuaX mentioned this pull request May 8, 2018
Closed
@JiankangMai JiankangMai mentioned this pull request Jun 3, 2018
Closed
@117311730 117311730 mentioned this pull request Jun 26, 2018
Closed
@bnubobby bnubobby mentioned this pull request Jul 2, 2018
Closed
@X-C3LL X-C3LL mentioned this pull request Aug 14, 2018
Closed
twose pushed a commit that referenced this pull request Aug 24, 2018
@Inokinoki
Merge pull request #16 from swoole/master
bb6a4a1 
Sync upstream
@hustneverland hustneverland mentioned this pull request Aug 26, 2018
Closed
@xiaoshenge xiaoshenge mentioned this pull request Sep 18, 2018
Closed
@shehuaqigai shehuaqigai mentioned this pull request Sep 19, 2018
Closed
@JesseWjs JesseWjs mentioned this pull request Oct 10, 2018
Closed
@forgetyn forgetyn mentioned this pull request Sep 24, 2019
Closed
@shh12580 shh12580 mentioned this pull request Oct 10, 2019
Closed
@kenashkov kenashkov mentioned this pull request Nov 29, 2019
Closed
twose added a commit that referenced this pull request Mar 6, 2020
@twose
Update library
b9f9948 
+ Update Constant.php
+ FastCGI support (#15)
+ Add Coroutine\batch function (#16)
+ Support FastCGI Proxy (#17)
+ Support CURLOPT_HTTPAUTH
@twose twose mentioned this pull request Mar 6, 2020
Merged
matyhtf pushed a commit that referenced this pull request Mar 6, 2020
@twose
Update Library (#3165)
4f1e08c 
* Update library

+ Update Constant.php
+ FastCGI support (#15)
+ Add Coroutine\batch function (#16)
+ Support FastCGI Proxy (#17)
+ Support CURLOPT_HTTPAUTH

* Fix tests for latest PHP version
@jaki2012 jaki2012 mentioned this pull request Jun 5, 2020
Closed
@Frank-JY Frank-JY mentioned this pull request Aug 9, 2020
Closed
@laofu360 laofu360 mentioned this pull request Nov 7, 2020
Open
@pradeepsingh01 pradeepsingh01 mentioned this pull request May 4, 2021
Closed
@imdake imdake mentioned this pull request Jul 13, 2021
Closed
@w7391108 w7391108 mentioned this pull request Dec 24, 2021
Closed
@shuipf shuipf mentioned this pull request May 16, 2022
Closed
@wuyiwai wuyiwai mentioned this pull request Jan 11, 2023
Open
@wilbur-yu wilbur-yu mentioned this pull request May 23, 2023
Open
@WongYueKeung WongYueKeung mentioned this pull request Jun 29, 2023
Closed
@dajiangdajiang dajiangdajiang mentioned this pull request Aug 24, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@remicollet @matyhtf