Add short circuit case for self assignment...

base/pickle.h

Index: base/pickle.h
===================================================================
--- base/pickle.h	(revision 22731)
+++ base/pickle.h	(working copy)
@@ -31,7 +31,7 @@
 //
 class Pickle {
  public:
-  ~Pickle();
+  virtual ~Pickle();

[wtc, 2011/09/21 23:37:04]

jar: I was wondering why the Pickle class has a virtual
destructor, and I tracked it down to this CL.

The Pickle class does not have any virtual method, so it
is strange for it to have a virtual destructor.

I am not convinced that the reason you gave is correct.
In any case, I suggest you add a comment to the source code
to explain why.  This will save the next person a few minutes
to answer this question.  Thanks.

 
   // Initialize a Pickle object using the default header size.
   Pickle();