[Merge] lp:~lorn-potter/qtubuntu-media/add-buffering_changed into lp:qtubuntu-media

Jim Hodapp jim.hodapp at canonical.com
Fri Jun 10 12:41:40 UTC 2016


Review: Needs Fixing

Just a couple of fixes left.

Diff comments:

> 
> === modified file 'tests/unit/player.cpp'
> --- tests/unit/player.cpp	2016-03-17 19:44:54 +0000
> +++ tests/unit/player.cpp	2016-06-09 18:34:52 +0000
> @@ -311,6 +311,12 @@
>      return ret;
>  }
>  
> +const core::Signal<int>& TestPlayer::buffering_changed() const
> +{
> +    static core::Signal<int> dur;

Should be static const core::Signal<int> dur;

> +    return dur;
> +}
> +
>  const std::shared_ptr<Service> Service::Client::instance()
>  {
>      return NULL;
> 
> === modified file 'tests/unit/player_mock.cpp'
> --- tests/unit/player_mock.cpp	2015-06-08 19:16:35 +0000
> +++ tests/unit/player_mock.cpp	2016-06-09 18:34:52 +0000
> @@ -308,6 +308,12 @@
>      return ret;
>  }
>  
> +const core::Signal<int>& PlayerMock::buffering_changed() const
> +{
> +    static core::Signal<int> dur;

Should be static const core::Signal<int> dur;

> +    return dur;
> +}
> +
>  }
>  }
>  }


-- 
https://code.launchpad.net/~lorn-potter/qtubuntu-media/add-buffering_changed/+merge/296731
Your team Ubuntu Phablet Team is subscribed to branch lp:qtubuntu-media.



More information about the Ubuntu-reviews mailing list