Skip to content

Merge remote-tracking branch 'origin/releases' into mc/merge/1.0_b0e18814#3014

Closed
beeender wants to merge 3 commits intomasterfrom
mc/merge/1.0_b0e18814
Closed

Merge remote-tracking branch 'origin/releases' into mc/merge/1.0_b0e18814#3014
beeender wants to merge 3 commits intomasterfrom
mc/merge/1.0_b0e18814

Conversation

@beeender
Copy link
Copy Markdown
Contributor

No description provided.

emanuelez and others added 3 commits June 16, 2016 10:01
* Add a Dockerfile to the repo

This is done to facilitate CI and build reproducibility
* Enable -Wmissing-declarations and -Werror to ensure all global
  functions are defined with a proper declaration. JNI function not
  found problem can only be seen at run time, thus we need to do so.
* Add static keyword for local functions.
* Fix wrong JNI function declaration.
* Remove useless functions.
* TableView distinct should return void.
* Rename JNI cpp files.
@beeender
Copy link
Copy Markdown
Contributor Author

retest this please

@beeender
Copy link
Copy Markdown
Contributor Author

CI: retest this please

@beeender
Copy link
Copy Markdown
Contributor Author

CI: Please help me to retest this, thanks.

@beeender
Copy link
Copy Markdown
Contributor Author

CI:retest

@beeender
Copy link
Copy Markdown
Contributor Author

CI: retest

@kneth kneth added the S:Review label Jun 16, 2016
@beeender
Copy link
Copy Markdown
Contributor Author

merge #3016 instead.

@beeender beeender closed this Jun 16, 2016
@beeender beeender removed the S:Review label Jun 16, 2016
@beeender beeender deleted the mc/merge/1.0_b0e18814 branch June 16, 2016 16:02
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 16, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants