Re: Review Request 128937: fix extract image extracting from proxy producer instead of original producer

2016-09-18 Thread The DiveO

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128937/
---

(Updated Sept. 18, 2016, 10:15 p.m.)


Status
--

This change has been marked as submitted.


Review request for Kdenlive.


Changes
---

Submitted with commit 16d894c61d4d6875b7bed4297a30a410c33bcb28 by Jean-Baptiste 
Mardelle to branch master.


Repository: kdenlive


Description
---

When extract frame detects a proxy it tries to use the property "resource" for 
extracting from the original clip; however, this property points to the proxy 
clip, but not the original. Instead, the property "kdenlive:originalurl" looks 
like what originally was intended. Not least the preceeding comment mentions to 
use the original clip url ... and that strongly points towards the property 
"kdenlive:originalurl".


Diffs
-

  src/monitor/monitor.cpp 9d452aa 

Diff: https://git.reviewboard.kde.org/r/128937/diff/


Testing
---

Using fresh recent Kdenlive git master.


Thanks,

The DiveO



Review Request 128937: fix extract image extracting from proxy producer instead of original producer

2016-09-18 Thread The DiveO

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128937/
---

Review request for Kdenlive.


Repository: kdenlive


Description
---

When extract frame detects a proxy it tries to use the property "resource" for 
extracting from the original clip; however, this property points to the proxy 
clip, but not the original. Instead, the property "kdenlive:originalurl" looks 
like what originally was intended. Not least the preceeding comment mentions to 
use the original clip url ... and that strongly points towards the property 
"kdenlive:originalurl".


Diffs
-

  src/monitor/monitor.cpp 9d452aa 

Diff: https://git.reviewboard.kde.org/r/128937/diff/


Testing
---

Using fresh recent Kdenlive git master.


Thanks,

The DiveO