Fix two deallocation bugs identified by Coverity Prevent.
authorkmillikin@chromium.org <kmillikin@chromium.org@ce2b1a6d-e550-0410-aec6-3dcde31c8c00>
Wed, 22 Jul 2009 10:23:19 +0000 (10:23 +0000)
committerkmillikin@chromium.org <kmillikin@chromium.org@ce2b1a6d-e550-0410-aec6-3dcde31c8c00>
Wed, 22 Jul 2009 10:23:19 +0000 (10:23 +0000)
commit1cbe7a240a0bdbde8ccedaececac01b24e75d155
tree5af761232d43227d5150dc9faf235886fff5b846
parent2183309385d5f49d1d188fead733e833251b7a6f
Fix two deallocation bugs identified by Coverity Prevent.

1. The tables array allocated in the CompilationSubCache constructor
   was never deallocated.  Fixed by adding destructor.

2. The buffer allocated in one of the constructors of the
   NoAllocationStringAllocator was never deallocated.  It seems that
   this class sometimes owns the buffer (if it allocated one itself)
   and sometimes doesn't (if it was passed one).  Simple fix is to
   remove the offending constructor which was never used anyway.

Review URL: http://codereview.chromium.org/155917

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@2520 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
src/compilation-cache.cc
src/string-stream.cc
src/string-stream.h