Skip to content

Conversation

@juhaylinen
Copy link
Contributor

Description

Fixes below warning from GCC_ARM compiler:
[Warning] InternetSocket.h@237,10: 'InternetSocket::_factory_allocated' will be initialized after [-Wreorder]
[Warning] InternetSocket.h@236,23: 'volatile unsigned int InternetSocket::_pending' [-Wreorder]
[Warning] InternetSocket.cpp@22,1: when initialized here [-Wreorder]

Pull request type

[X] Fix
[ ] Refactor
[ ] Target update
[ ] Functionality change
[ ] Breaking change
[Warning] InternetSocket.h@237,10: 'InternetSocket::_factory_allocated' will be initialized after [-Wreorder]
[Warning] InternetSocket.h@236,23:   'volatile unsigned int InternetSocket::_pending' [-Wreorder]
[Warning] InternetSocket.cpp@22,1:   when initialized here [-Wreorder]
@0xc0170 0xc0170 requested a review from a team August 24, 2018 12:16
@0xc0170
Copy link
Contributor

0xc0170 commented Aug 27, 2018

/morph build

@mbed-ci
Copy link

mbed-ci commented Aug 27, 2018

Build : SUCCESS

Build number : 2922
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/7883/

Triggering tests

/morph test
/morph uvisor-test
/morph export-build
/morph mbed2-build

@cmonr
Copy link
Contributor

cmonr commented Aug 28, 2018

Huh. I don't think the failure is related to the PR. @juhaylinen Could you confirm?

@juhaylinen
Copy link
Contributor Author

@cmonr Looks like random fail. Please restart the tests.

@0xc0170
Copy link
Contributor

0xc0170 commented Aug 28, 2018

/morph test

@juhaylinen
Copy link
Contributor Author

Greentea sync failure. Please restart.

@0xc0170
Copy link
Contributor

0xc0170 commented Sep 7, 2018

Greentea sync failure. Please restart.

Will do once rc2 PR are completed

@adbridge
Copy link
Contributor

/morph build

@mbed-ci
Copy link

mbed-ci commented Sep 10, 2018

Build : SUCCESS

Build number : 3044
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/7883/

Triggering tests

/morph test
/morph export-build
/morph mbed2-build

@cmonr
Copy link
Contributor

cmonr commented Sep 11, 2018

Test CI needs to be restarted. Will restart aftetr 5.10-rc2 is generated.

@0xc0170
Copy link
Contributor

0xc0170 commented Sep 18, 2018

/morph test

@cmonr cmonr merged commit c356f1f into ARMmbed:master Sep 18, 2018
@cmonr cmonr removed the needs: CI label Sep 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment