[haiku-commits] Re: haiku: hrev50371 - src/kits/media src/apps/mediaplayer/playlist src/add-ons/media/plugins/rtsp_streamer src/add-ons/media/plugins/http_streamer headers/private/media

  • From: Dario Casalinuovo <b.vitruvio@xxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Fri, 24 Jun 2016 22:36:55 +0200

On Fri, Jun 24, 2016 at 8:59 PM, marcus overhagen <marcusoverhagen@xxxxxxxxx

wrote:

On Fri, Jun 24, 2016 at 7:02 PM, <b.vitruvio@xxxxxxxxx> wrote:


  * There's no need to pass BUrl by pointer, this is
  potentially dangerous and leaky without any advantage,
  copying is definitely safer.

To avoid pointers *and* copying of complex or large objects, you usually
pass a const reference "const BUrl &"


Yes there's no reason to don't do it. Thanks for reporting.

Other related posts: