1. 29 Feb, 2016 5 commits
    • scoder's avatar
      Merge pull request #494 from Nikratio/bugfix2 · 812cd069
      scoder authored
      __Pyx_PyInt_TrueDivideObjC: switch comparison order
      812cd069
    • scoder's avatar
      Merge pull request #493 from Nikratio/master · ebf10c7f
      scoder authored
      FROM_PY_FUNCTION: Cast to sdigit after unary minus, not before
      ebf10c7f
    • Nikolaus Rath's avatar
      __Pyx_PyInt_TrueDivideObjC: switch comparison order · 81290209
      Nikolaus Rath authored
      The current order results in compiler warnings on 32 bit machines, e.g.
      
      src/llfuse.c: In function '__Pyx_PyInt_TrueDivideObjC':
      src/llfuse.c:43980:17: warning: left shift count >= width of type
                       if (8 * sizeof(long) <= 53 || (__Pyx_sst_abs(size) <= 52 / PyLong_SHIFT) || likely(labs(a) <= (1L << 53))) {
      
      Switching the order so that the left shift is closer to the sizeof test
      avoids the warning, presumably because it makes it easier for the
      compiler to see that the left shift is only executed on 64 bit.
      
      Thanks to Christian Neukirchen for doing most of the work!
      81290209
    • Nikolaus Rath's avatar
      __Pyx_PyInt_TrueDivideObjC: switch comparison order · 9c100bce
      Nikolaus Rath authored
      The current order results in compiler warnings on 32 bit machines, e.g.
      
      src/llfuse.c: In function '__Pyx_PyInt_TrueDivideObjC':
      src/llfuse.c:43980:17: warning: left shift count >= width of type
                       if (8 * sizeof(long) <= 53 || (__Pyx_sst_abs(size) <= 52 / PyLong_SHIFT) || likely(labs(a) <= (1L << 53))) {
      
      Switching the order so that the left shift is closer to the sizeof test
      avoids the warning, presumably because it makes it easier for the
      compiler to see that the left shift is only executed on 64 bit.
      
      Thanks to Christian Neukirchen for doing most of the work!
      9c100bce
    • Nikolaus Rath's avatar
      FROM_PY_FUNCTION: Cast to sdigit after unary minus, not before · 6e8b60bb
      Nikolaus Rath authored
      The current cast isn't doing anything, because the unary minus implicitly
      upcasts to int again, resulting in warnings about potentially value altering
      conversions on 32 bit systems.
      
      Fixes issue #877.
      6e8b60bb
  2. 21 Feb, 2016 7 commits
  3. 19 Feb, 2016 2 commits
  4. 12 Feb, 2016 4 commits
  5. 09 Feb, 2016 3 commits
  6. 08 Feb, 2016 3 commits
  7. 07 Feb, 2016 3 commits
  8. 05 Feb, 2016 2 commits
  9. 02 Feb, 2016 7 commits
  10. 28 Jan, 2016 1 commit
  11. 27 Jan, 2016 2 commits
  12. 26 Jan, 2016 1 commit