- 23 May, 2017 4 commits
-
-
Poul-Henning Kamp authored
-
Poul-Henning Kamp authored
-
Poul-Henning Kamp authored
The .VSC file steals .RST's directive syntax, but it is not a RST file, and it is not processed as such. Maybe later it will become a .RST file, but that is TBD. A C-struct for use in varnishd is built, (along with a JSON structure which describes the counters.) For now that structure is identical with what comes out of include/tbl/vsc_f_main, so nothing special happens in the VSM yet.
-
Poul-Henning Kamp authored
-
- 22 May, 2017 2 commits
-
-
Poul-Henning Kamp authored
fully understood circular-ish make distcheck dependency problem.
-
Poul-Henning Kamp authored
-
- 21 May, 2017 4 commits
-
-
Poul-Henning Kamp authored
-
Poul-Henning Kamp authored
-
Poul-Henning Kamp authored
-
Poul-Henning Kamp authored
-
- 20 May, 2017 3 commits
-
-
Federico G. Schwindt authored
-
Federico G. Schwindt authored
-
Federico G. Schwindt authored
-
- 19 May, 2017 1 commit
-
-
Federico G. Schwindt authored
-
- 16 May, 2017 3 commits
-
-
Nils Goroll authored
Thank you to @RantyDave Closes #2326
-
Geoff Simmons authored
-
Dridi Boukelmoune authored
It's sometimes confusing new users, and it managed to confuse me too a couple months ago.
-
- 15 May, 2017 8 commits
-
-
Dridi Boukelmoune authored
-
Poul-Henning Kamp authored
-
Poul-Henning Kamp authored
-
Poul-Henning Kamp authored
-
Poul-Henning Kamp authored
-
Dridi Boukelmoune authored
Fixes #2329
-
Poul-Henning Kamp authored
-
Poul-Henning Kamp authored
-
- 13 May, 2017 4 commits
-
-
Federico G. Schwindt authored
-
Federico G. Schwindt authored
-
Federico G. Schwindt authored
Spotted by coverity.
-
Federico G. Schwindt authored
Spotted by coverity.
-
- 12 May, 2017 10 commits
-
-
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.
-
Nils Goroll authored
-
Nils Goroll authored
-
Nils Goroll authored
The name idle_send_timeout does not seem to describe well where the timeout applies (individual write()/writev() calls for arbitraty amounts of data). The fact that the explanation is complicated might be an indication that a different timeout implementation be more appropriate. HTTP2 should probably implement send_timeout somewhen.
-
Nils Goroll authored
-
Nils Goroll authored
-
Nils Goroll authored
-
Dridi Boukelmoune authored
Spotted by Coverity: /lib/libvgz/deflate.c: 605 in deflateParams() 599 } 600 if (s->level != level) { 601 if (s->level == 0 && s->matches != 0) { 602 if (s->matches == 1) 603 slide_hash(s); 604 else >>> CID 1401053: Control flow issues (NESTING_INDENT_MISMATCH) >>> The macro on this line expands into multiple statements, only >>> the first of which is nested within the preceding parent while >>> the rest are not. 605 CLEAR_HASH(s); 606 s->matches = 0; 607 } 608 s->level = level; 609 s->max_lazy_match = configuration_table[level].max_lazy; 610 s->good_match = configuration_table[level].good_length;
-
Federico G. Schwindt authored
We now process the inclusive globs first, and then the exclusive ones. The former are accumulative, while the latter will filter the output from the first. Discussed in detail with and final diff from @mbgrydeland. Tests tweaking by yours truly. Fixes #2320.
-
Federico G. Schwindt authored
-
- 08 May, 2017 1 commit
-
-
Martin Blix Grydeland authored
If an object has OA_VARY, we should succeed in getting it. This might help to narrow down #2319
-