reopening for a check/cleanup before considering the issue fixed, I see several places in the patch where the check for rtpm/rtmpt is duplicated without reason:
(rtmpt starts with rtmp so all the check for rtmpt are never called... plus we should check for "rtmp:" and "rtmpt:", it's better if we don't want to make rtmp a private keyword when naming local files...)
Fabrizio Giustina added a comment - 01/May/10 15:39 reopening for a check/cleanup before considering the issue fixed, I see several places in the patch where the check for rtpm/rtmpt is duplicated without reason:
(rtmpt starts with rtmp so all the check for rtmpt are never called... plus we should check for "rtmp:" and "rtmpt:", it's better if we don't want to make rtmp a private keyword when naming local files...)
reopening for a check/cleanup before considering the issue fixed, I see several places in the patch where the check for rtpm/rtmpt is duplicated without reason:
(rtmpt starts with rtmp so all the check for rtmpt are never called... plus we should check for "rtmp:" and "rtmpt:", it's better if we don't want to make rtmp a private keyword when naming local files...)