[DhtScanner] doesn't compile on LLVM 7.0.2 #92

Closed
sim590 opened this Issue Jul 18, 2016 · 4 comments

Comments

Projects
None yet
3 participants
@sim590
Contributor

sim590 commented Jul 18, 2016

As Travis pointed out when checking errors in #79, DhtScanner doesn't compile on LLVM. However, everything else compiles, so this is not high priority. The code is now merged in master. I'm highlighting the issue here so that it appears to be a known issue.

@sim590 sim590 added the bug label Jul 18, 2016

@sim590 sim590 self-assigned this Jul 18, 2016

@kaldoran

This comment has been minimized.

Show comment
Hide comment
@kaldoran

kaldoran Jul 19, 2016

Collaborator

Isn't this f474f64 patch correct this issue ?
If it does, please close this issue :)

Collaborator

kaldoran commented Jul 19, 2016

Isn't this f474f64 patch correct this issue ?
If it does, please close this issue :)

@sim590

This comment has been minimized.

Show comment
Hide comment
@sim590

sim590 Jul 19, 2016

Contributor

Actually, f474f64 gets rid of some warnings, but it doesn't resolves the errors.

Contributor

sim590 commented Jul 19, 2016

Actually, f474f64 gets rid of some warnings, but it doesn't resolves the errors.

@sim590

This comment has been minimized.

Show comment
Hide comment
@sim590

sim590 Jul 22, 2016

Contributor

This issue seems not to apply for LLVM 7.3.0 (Apple version) which is one of the latest release at this point. This bug report may fade in the dark space of forgotten history... Sometime.

Contributor

sim590 commented Jul 22, 2016

This issue seems not to apply for LLVM 7.3.0 (Apple version) which is one of the latest release at this point. This bug report may fade in the dark space of forgotten history... Sometime.

@aberaud

This comment has been minimized.

Show comment
Hide comment
@aberaud

aberaud Nov 21, 2016

Contributor

Fixed since we use a decent LLVM/Clang version in Travis with Docker

Contributor

aberaud commented Nov 21, 2016

Fixed since we use a decent LLVM/Clang version in Travis with Docker

@aberaud aberaud closed this Nov 21, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment