1251881 - use UniquePtr instead of ScopedDeletePtr in mozglue/linker/

Status: RESOLVED FIXED

(Core :: mozglue, defect)

Description

[Nathan Froyd [:froydnj]]

There are two instances:

- MappableSeekableZStream::Create uses ScopedDeletePtr, which is
  actually a little odd, since MappableSeekableZStream, as a Mappable,
  is refcounted, and it's unusual to use two different kinds of smart
  pointers for a single class.  I think the more natural thing would be
  for MappableSeekableZStream::Create to return an already_AddRefed, but
  that change spirals out of the realm of "simple refactoring" quickly.

- SzipCompress::run uses ScopedDeletePtr along with some dubious use of
  raw pointers.  I changed things out for UniquePtr and also tidied the
  memory management a bit, which I think makes things a little clearer.

Comment 1

[Nathan Froyd [:froydnj]]

Attachment: use UniquePtr instead of ScopedDeletePtr in mozglue/linker/

Comment 2

[Mike Hommey [:glandium]]

Comment on attachment 8724455 [details] [diff] [review]
use UniquePtr instead of ScopedDeletePtr in mozglue/linker/

Review of attachment 8724455 [details] [diff] [review]:
-----------------------------------------------------------------

::: mozglue/linker/Mappable.cpp
@@ +377,5 @@
>  MappableSeekableZStream::Create(const char *name, Zip *zip,
>                                  Zip::Stream *stream)
>  {
>    MOZ_ASSERT(stream->GetType() == Zip::Stream::STORE);
> +  UniquePtr<MappableSeekableZStream> mappable(new MappableSeekableZStream(zip));

Couldn't that work with a RefPtr, which would make more sense? FWIW, the use of ScopedDeletePtr (and AutoDeletePtr before that) here predates Mappables being refcounted...

Comment 3

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/af6363aa513b

Comment 4

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/af6363aa513b