← Back to team overview

hugin-devs team mailing list archive

[Bug 1332323] Re: Fix issues reported by valgrind

 

updated fourth patch

** Patch added: "Use_stack_allocated_CheckpointPixels_instead_of_unique_ptr_tracked_one_.diff"
   https://bugs.launchpad.net/enblend/+bug/1332323/+attachment/4193646/+files/Use_stack_allocated_CheckpointPixels_instead_of_unique_ptr_tracked_one_.diff

-- 
You received this bug notification because you are a member of Hugin
Developers, which is subscribed to Enblend.
https://bugs.launchpad.net/bugs/1332323

Title:
  Fix issues reported by valgrind

Status in Enblend:
  Fix Committed

Bug description:
  Hi everyone,

  I ran enblend under valgrind which reported several issues.

  The first one is a simple missing initialization.

  The other ones are mostly about bad usage of boost::scoped_ptr<>. I have two 
  different approaches to fix this:

  1. Directly write into std::string buffer instead of using a temporary char[] 
  buffer. This is guaranteed behavior by C++11, see e.g.
  http://herbsutter.com/2008/04/07/cringe-not-vectors-are-guaranteed-to-be-contiguous/#comment-483

  2. Use a boost::tokenizer. As it does not need a char[] buffer, different to
  strtok, no need to allocate one. This has the nice side effect to get rid
  of the homegrown enblend::strtoken_r tokenizer.

  Kind regards,

  Stefan

To manage notifications about this bug go to:
https://bugs.launchpad.net/enblend/+bug/1332323/+subscriptions


References