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

util/channel.h: remove an invalid const specifier #79

Merged
merged 1 commit into from
Apr 1, 2020

Conversation

knz
Copy link

@knz knz commented Apr 1, 2020

This error was found by Clang 10.

Informs cockroachdb/cockroach#46300


This change is Reviewable

@knz knz requested a review from petermattis April 1, 2020 08:48
@knz knz changed the title util/channel.go: remove an invalid const specifier util/channel.h: remove an invalid const specifier Apr 1, 2020
This error was found by Clang 10.
Copy link

@petermattis petermattis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@knz
Copy link
Author

knz commented Apr 1, 2020

@petermattis there's a non-Java test failure at https://travis-ci.org/github/cockroachdb/rocksdb/jobs/669596695?utm_medium=notification&utm_source=github_status

something to do with column families. Is this ok?

@petermattis
Copy link

Hmm, I can't see why this change would have caused that. I also can't find the button in Travis to restart the build. Did they remove it?

@petermattis
Copy link

I also can't find the button in Travis to restart the build. Did they remove it?

Logging out of Travis and back in fixed that problem. I've restarted the build.

@knz
Copy link
Author

knz commented Apr 1, 2020

ok so all the remaining failures are java-related. merging.

@knz knz merged commit 9257207 into crl-release-6.2.1 Apr 1, 2020
@knz knz deleted the knz/clang-10-patch branch April 1, 2020 16:12
@knz knz restored the knz/clang-10-patch branch April 1, 2020 16:18
@knz knz deleted the knz/clang-10-patch branch April 1, 2020 18:23
peilun-conflux pushed a commit to Conflux-Chain/rocksdb that referenced this pull request Aug 4, 2020
Thegaram pushed a commit to Thegaram/rocksdb that referenced this pull request Apr 2, 2021
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.

None yet

2 participants