subject:"\[squid\-dev\] \[PATCH\] ServerBump class cleanup"

Re: [squid-dev] [PATCH] ServerBump class cleanup

2016-11-30 Thread Alex Rousskov

On 11/30/2016 06:34 AM, Amos Jeffries wrote: > This patch is a general cleanup of coding styles and current code > requirements for the ServerBump class. > -void attachServerSSL(SSL *); ///< Sets the server SSL object > +void attachServerSession(const Security::SessionPointer &); ///<

Re: [squid-dev] [PATCH] ServerBump class cleanup

2016-11-30 Thread Kinkie

Looks reasonable to me. +1 On Wed, Nov 30, 2016 at 1:34 PM, Amos Jeffries wrote: > This patch is a general cleanup of coding styles and current code > requirements for the ServerBump class. > > Is there any other general cleanup that should be done to this class > while I'm

[squid-dev] [PATCH] ServerBump class cleanup

2016-11-30 Thread Amos Jeffries

This patch is a general cleanup of coding styles and current code requirements for the ServerBump class. Is there any other general cleanup that should be done to this class while I'm at it ? Amos === modified file 'src/ssl/PeekingPeerConnector.cc' --- src/ssl/PeekingPeerConnector.cc