<div dir="ltr">Cool, Thanks!</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, 13 Jun 2019 at 14:20, Thomas Guillem <<a href="mailto:thomas@gllm.fr">thomas@gllm.fr</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hello,<br>
<br>
The whole set has been merged, thanks !<br>
<br>
On Mon, Jun 3, 2019, at 09:42, <a href="mailto:pierre@videolabs.io" target="_blank">pierre@videolabs.io</a> wrote:<br>
> On 2019-06-01 11:58, Alexandre Janniaux wrote:<br>
> > Hi,<br>
> > <br>
> > vlc_tick doesn't really have meaning in the GUI world.<br>
> > <br>
> > Instead of adding a scale function on vlctick for UI purpose,<br>
> > you should wrap conversion methods to units that have meaning<br>
> > like seconds or milliseconds, matching the existing vlctick<br>
> > API. Then the computation can be done in primitive type.<br>
> <br>
> This is the goal of this class, to provide representation of time unit.<br>
> We may add toMilliseconds, toSeconds methods there if needed.<br>
> <br>
> > Greats,<br>
> > <br>
> > --<br>
> > Alexandre Janniaux<br>
> > VideoLabs<br>
> > <br>
> > On Fri, May 31, 2019 at 02:00:52PM +0300, Abel Tesfaye wrote:<br>
> >> From: Abel Tesfaye <<a href="mailto:Abeltesfaye45@gmail.com" target="_blank">Abeltesfaye45@gmail.com</a>><br>
> >> <br>
> >> ---<br>
> >>  modules/gui/qt/util/vlctick.cpp | 5 +++++<br>
> >>  modules/gui/qt/util/vlctick.hpp | 1 +<br>
> >>  2 files changed, 6 insertions(+)<br>
> >> <br>
> >> diff --git a/modules/gui/qt/util/vlctick.cpp <br>
> >> b/modules/gui/qt/util/vlctick.cpp<br>
> >> index bd0074da54..6cbf6f553a 100644<br>
> >> --- a/modules/gui/qt/util/vlctick.cpp<br>
> >> +++ b/modules/gui/qt/util/vlctick.cpp<br>
> >> @@ -49,3 +49,8 @@ QString VLCTick::toString() const<br>
> >>                  .arg(min, 2, 10, QChar('0'))<br>
> >>                  .arg(sec, 2, 10, QChar('0'));<br>
> >>  }<br>
> >> +<br>
> >> +VLCTick VLCTick::scale(float scalar) const<br>
> >> +{<br>
> >> +    return VLCTick(m_ticks*scalar);<br>
> >> +}<br>
> >> \ No newline at end of file<br>
> >> diff --git a/modules/gui/qt/util/vlctick.hpp <br>
> >> b/modules/gui/qt/util/vlctick.hpp<br>
> >> index 19ea5d1db5..e6f8f2b2aa 100644<br>
> >> --- a/modules/gui/qt/util/vlctick.hpp<br>
> >> +++ b/modules/gui/qt/util/vlctick.hpp<br>
> >> @@ -39,6 +39,7 @@ public:<br>
> >>       * @return time as HH:MM:SS<br>
> >>       */<br>
> >>      Q_INVOKABLE QString toString() const;<br>
> >> +    Q_INVOKABLE VLCTick scale(float) const;<br>
> >> <br>
> >>  private:<br>
> >>      vlc_tick_t m_ticks;<br>
> >> --<br>
> >> 2.21.0<br>
> >> <br>
> >> _______________________________________________<br>
> >> vlc-devel mailing list<br>
> >> To unsubscribe or modify your subscription options:<br>
> >> <a href="https://mailman.videolan.org/listinfo/vlc-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/listinfo/vlc-devel</a><br>
> > _______________________________________________<br>
> > vlc-devel mailing list<br>
> > To unsubscribe or modify your subscription options:<br>
> > <a href="https://mailman.videolan.org/listinfo/vlc-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/listinfo/vlc-devel</a><br>
> _______________________________________________<br>
> vlc-devel mailing list<br>
> To unsubscribe or modify your subscription options:<br>
> <a href="https://mailman.videolan.org/listinfo/vlc-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/listinfo/vlc-devel</a><br>
_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="https://mailman.videolan.org/listinfo/vlc-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/listinfo/vlc-devel</a></blockquote></div>