Skip to content
This repository has been archived by the owner on Feb 12, 2023. It is now read-only.

return-statement with no value [add Travis CI to project] #47

Closed
suhr opened this issue Jul 1, 2014 · 3 comments
Closed

return-statement with no value [add Travis CI to project] #47

suhr opened this issue Jul 1, 2014 · 3 comments

Comments

@suhr
Copy link
Contributor

suhr commented Jul 1, 2014

widget/camera.cpp: In member function ‘virtual bool Camera::present(const QVideoFrame&)’:
widget/camera.cpp:89:9: error: return-statement with no value, in function returning ‘bool’ [-fpermissive]
make: *** [camera.o] Error 1

Consider adding Travis CI to repo for checking build status after commits and pull requests.

@tux3
Copy link
Member

tux3 commented Jul 1, 2014

I'm always compiling locally before pushing. This was a mistake, I accidentally pushed without it and didn't notice.
Jenkins warned me after the push and history was rewritten, sorry.

@tux3 tux3 closed this as completed Jul 1, 2014
@suhr
Copy link
Contributor Author

suhr commented Jul 1, 2014

This was a mistake, I accidentally pushed without it and didn't notice.

Better to add git's pre-commit hook to automatize this.

@tux3
Copy link
Member

tux3 commented Jul 1, 2014

I guess, but it feels a bit redundant since I already have jenkins and local builds.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants