[mlpack-git] [mlpack] Reduce (or fix) warnings when compiling with Visual Studio (#554)

[Ryan Curtin]

Ideally `uword` should be set to the same size as `size_t`. But you are saying that this does not happen on 64-bit versions of Windows? If so we can `#define ARMA_64BIT_WORD` in `src/mlpack/core/arma_extend/arma_extend.hpp`, but we should only do this on 64-bit Windows. On 32-bit Windows we should not set `ARMA_64BIT_WORD` though.

Does that seem like a reasonable solution?

If there are other unsafe casts in mlpack I am surprised that gcc or clang don't seem to be picking them up.

---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/mlpack/mlpack/issues/554#issuecomment-199320757
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.cc.gatech.edu/pipermail/mlpack-git/attachments/20160321/27ba4ae0/attachment.html>