[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Gnash-commit] [patch #8086] RTMP netStream patch
From: |
Sandro Santilli |
Subject: |
[Gnash-commit] [patch #8086] RTMP netStream patch |
Date: |
Fri, 06 Sep 2013 11:39:31 +0000 |
User-agent: |
Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:22.0) Gecko/20100101 Firefox/22.0 |
Follow-up Comment #38, patch #8086 (project gnash):
Things could be moved incrementally.
For a start the RTMP handling could be moved awan from netstream_play and into
NetStream::play, which if I read your code correctly seems to possibly need a
different signature (you are accepting an arbitrary set of arguments in the AS
NetStream.play method).
What I mean is that NetStream itself could be already the abstraction you're
looking for, but needs to be defined in a proper way.
Couldn't the RTMP specific code be in NetConnection::getStream, for example ?
Is it in netstream_play only to deal with the multiple-arguments or is there
any other reason ?
_______________________________________________________
Reply to this item at:
<http://savannah.gnu.org/patch/?8086>
_______________________________________________
Message sent via/by Savannah
http://savannah.gnu.org/
- [Gnash-commit] [patch #8086] RTMP netStream patch, (continued)
- [Gnash-commit] [patch #8086] RTMP netStream patch, Rob Savoye, 2013/09/04
- [Gnash-commit] [patch #8086] RTMP netStream patch, George Thomas, 2013/09/04
- [Gnash-commit] [patch #8086] RTMP netStream patch, Sandro Santilli, 2013/09/04
- [Gnash-commit] [patch #8086] RTMP netStream patch, Sandro Santilli, 2013/09/04
- [Gnash-commit] [patch #8086] RTMP netStream patch, Gabriele Giacone, 2013/09/04
- [Gnash-commit] [patch #8086] RTMP netStream patch, George Thomas, 2013/09/05
- [Gnash-commit] [patch #8086] RTMP netStream patch, Sandro Santilli, 2013/09/05
- [Gnash-commit] [patch #8086] RTMP netStream patch, George Thomas, 2013/09/05
- [Gnash-commit] [patch #8086] RTMP netStream patch, Sandro Santilli, 2013/09/05
- [Gnash-commit] [patch #8086] RTMP netStream patch, George Thomas, 2013/09/06
- [Gnash-commit] [patch #8086] RTMP netStream patch,
Sandro Santilli <=
- [Gnash-commit] [patch #8086] RTMP netStream patch, Sandro Santilli, 2013/09/06
- [Gnash-commit] [patch #8086] RTMP netStream patch, George Thomas, 2013/09/06
- [Gnash-commit] [patch #8086] RTMP netStream patch, Sandro Santilli, 2013/09/06
- [Gnash-commit] [patch #8086] RTMP netStream patch, George Thomas, 2013/09/14
- [Gnash-commit] [patch #8086] RTMP netStream patch, Sandro Santilli, 2013/09/14