1. 23 May, 2017 12 commits
  2. 22 May, 2017 2 commits
  3. 21 May, 2017 4 commits
  4. 20 May, 2017 3 commits
  5. 19 May, 2017 1 commit
  6. 16 May, 2017 3 commits
  7. 15 May, 2017 8 commits
  8. 13 May, 2017 4 commits
  9. 12 May, 2017 3 commits
    • Dridi Boukelmoune's avatar
      Don't assert on vtc_error · 94b70875
      Dridi Boukelmoune authored
      Reported by Coverity:
      
          ID 1406102:  Incorrect expression  (ASSERT_SIDE_EFFECT)
          /bin/varnishtest/vtc_http2.c: 852 in receive_frame()
          846                         s->wf = 0;
          847                         AZ(pthread_cond_signal(&s->cond));
          848                 }
          849                 continue;
          850         }
          851         AZ(pthread_mutex_unlock(&hp->mtx));
          >>> CID 1406102:  Incorrect expression  (ASSERT_SIDE_EFFECT)
          >>> Argument "vtc_error" of assert() has a side effect because the
          >>> variable is volatile.  The containing function might work
          >>> differently in a non-debug build.
          852         assert(vtc_error || vsb == NULL);
          853         return (NULL);
          854 }
          855
          856 #define STRTOU32(n, ss, p, v, c)                             \
          857         do {                                                 \
      
      Although I'm not sure how relevant it is since assert() from vas.h
      doesn't remove side effects when no-op'd.
      94b70875
    • Nils Goroll's avatar
      yes, the parameter names are confusing · f211c5cc
      Nils Goroll authored
      f211c5cc
    • Nils Goroll's avatar
      39e20baf