Re: [squid-dev] [PATCH] remove client_side_request.cci

2017-01-07 Thread Amos Jeffries
On 2017-01-06 17:54, Amos Jeffries wrote: This patch removes client_side_request.cci file, moving the methods inline to the client_side_request.h. Applied as v5 r15001. Amos ___ squid-dev mailing list squid-dev@lists.squid-cache.org

Re: [squid-dev] [PATCH] remove --disable-inline and related macros

2017-01-07 Thread Alex Rousskov
On 01/07/2017 07:33 PM, Amos Jeffries wrote: > This now just depends on the client_side_request_cci_remove_mk1.patch > submitted earlier. > > With the .cci files gone there is no longer any use of the SQUID_INLINE > macro. Which in turn removes the need for _USE_INLINE_ macro and then > the

[squid-dev] [PATCH] remove --disable-inline and related macros

2017-01-07 Thread Amos Jeffries
This now just depends on the client_side_request_cci_remove_mk1.patch submitted earlier. With the .cci files gone there is no longer any use of the SQUID_INLINE macro. Which in turn removes the need for _USE_INLINE_ macro and then the --disable-inline build option which sets that. The

Re: [squid-dev] [PATCH] remove ip/Qos.cci

2017-01-07 Thread Amos Jeffries
On 2017-01-08 06:33, Kinkie wrote: Looks good, except that +Qos::Config() {STUB} Surrounding the STUB macro in brackets is not the common style (although it is harmless and could actually be a good idea) +1, possibly with this change There was some reason for that which I've forgotten the

Re: [squid-dev] [PATCH] remove ip/Qos.cci

2017-01-07 Thread Alex Rousskov
On 01/07/2017 04:46 AM, Amos Jeffries wrote: > This patch removes ip/Qos.cci file moving its content to ip/QosConfig.cc. > > Also, move the stub file to src/tests/stub_libip.cc and update to use > tests/STUB.h interface. > +bool > +Ip::Qos::Config::isHitTosActive() const > +{ > +return

Re: [squid-dev] [PATCH] remove ip/Qos.cci

2017-01-07 Thread Kinkie
Looks good, except that +Qos::Config() {STUB} Surrounding the STUB macro in brackets is not the common style (although it is harmless and could actually be a good idea) +1, possibly with this change On Sat, Jan 7, 2017 at 11:46 AM, Amos Jeffries wrote: > This patch

[squid-dev] Jenkins build is back to normal : 4-matrix » clang,d-ubuntu-trusty #25

2017-01-07 Thread noc
https://build.squid-cache.org/job/4-matrix/compiler=clang,label=d-ubuntu-trusty/25/ ___ squid-dev mailing list squid-dev@lists.squid-cache.org http://lists.squid-cache.org/listinfo/squid-dev

[squid-dev] [PATCH] remove ip/Qos.cci

2017-01-07 Thread Amos Jeffries
This patch removes ip/Qos.cci file moving its content to ip/QosConfig.cc. Also, move the stub file to src/tests/stub_libip.cc and update to use tests/STUB.h interface. Amos === modified file 'src/ip/Makefile.am' --- src/ip/Makefile.am 2017-01-01 00:12:22 + +++ src/ip/Makefile.am

[squid-dev] Jenkins build is back to normal : 5-matrix » gcc,d-opensuse-13.2 #57

2017-01-07 Thread noc
https://build.squid-cache.org/job/5-matrix/compiler=gcc,label=d-opensuse-13.2/57/ ___ squid-dev mailing list squid-dev@lists.squid-cache.org http://lists.squid-cache.org/listinfo/squid-dev

[squid-dev] Jenkins build is back to normal : 5-matrix » gcc,d-debian-jessie #57

2017-01-07 Thread noc
https://build.squid-cache.org/job/5-matrix/compiler=gcc,label=d-debian-jessie/57/ ___ squid-dev mailing list squid-dev@lists.squid-cache.org http://lists.squid-cache.org/listinfo/squid-dev

[squid-dev] Jenkins build is back to normal : 5-matrix » clang,d-opensuse-13.2 #57

2017-01-07 Thread noc
https://build.squid-cache.org/job/5-matrix/compiler=clang,label=d-opensuse-13.2/57/ ___ squid-dev mailing list squid-dev@lists.squid-cache.org http://lists.squid-cache.org/listinfo/squid-dev

[squid-dev] Jenkins build is back to normal : 5-matrix » gcc,d-ubuntu-utopic #57

2017-01-07 Thread noc
https://build.squid-cache.org/job/5-matrix/compiler=gcc,label=d-ubuntu-utopic/57/ ___ squid-dev mailing list squid-dev@lists.squid-cache.org http://lists.squid-cache.org/listinfo/squid-dev

[squid-dev] Jenkins build is back to normal : 5-matrix » gcc,d-debian-wheezy #57

2017-01-07 Thread noc
https://build.squid-cache.org/job/5-matrix/compiler=gcc,label=d-debian-wheezy/57/ ___ squid-dev mailing list squid-dev@lists.squid-cache.org http://lists.squid-cache.org/listinfo/squid-dev

[squid-dev] Jenkins build is back to normal : 5-matrix » clang,d-debian-jessie #57

2017-01-07 Thread noc
https://build.squid-cache.org/job/5-matrix/compiler=clang,label=d-debian-jessie/57/ ___ squid-dev mailing list squid-dev@lists.squid-cache.org http://lists.squid-cache.org/listinfo/squid-dev

[squid-dev] Jenkins build is back to normal : 5-matrix » gcc,d-ubuntu-vivid #57

2017-01-07 Thread noc
https://build.squid-cache.org/job/5-matrix/compiler=gcc,label=d-ubuntu-vivid/57/ ___ squid-dev mailing list squid-dev@lists.squid-cache.org http://lists.squid-cache.org/listinfo/squid-dev

[squid-dev] Jenkins build is back to normal : 5-matrix » clang,d-ubuntu-utopic #57

2017-01-07 Thread noc
https://build.squid-cache.org/job/5-matrix/compiler=clang,label=d-ubuntu-utopic/57/ ___ squid-dev mailing list squid-dev@lists.squid-cache.org http://lists.squid-cache.org/listinfo/squid-dev

[squid-dev] Jenkins build is back to normal : 5-matrix » clang,d-fedora-21 #57

2017-01-07 Thread noc
https://build.squid-cache.org/job/5-matrix/compiler=clang,label=d-fedora-21/57/ ___ squid-dev mailing list squid-dev@lists.squid-cache.org http://lists.squid-cache.org/listinfo/squid-dev

[squid-dev] Jenkins build is back to normal : 5-matrix » gcc,d-debian-unstable #57

2017-01-07 Thread noc
https://build.squid-cache.org/job/5-matrix/compiler=gcc,label=d-debian-unstable/57/ ___ squid-dev mailing list squid-dev@lists.squid-cache.org http://lists.squid-cache.org/listinfo/squid-dev