On Tuesday, April 5, 2022, Phyllis Smith <phylsmith2017@gmail.com> wrote:
Checked into GIT, Manuel's DPX list (read but not write) new files and modifications to existing files.
Added in the changes Andrew provided in addition to accommodate other O/S like Termux (hopefully did not miss anything so if Andrew could check, that would be appreciated).

I'll wait a bit until second half of ogg ifdefing lands, so git hopefully rebase with less rejects... 

I had already redone the libvpx.patch when it failed so I could compile, but later put in Andrew's 0070-Fix-libdpx-patch.patch anyway.  And of course, added in 0071-Add-without-libdpx-to-configure.patch and did a rebuild.

i think I missed piece adding LIBDPX among group of deps so libdpx's status displayed at the end of configure

0059-Add-libdpx-status-to-configure-summary.patch

from megapile_3

BUT, I have not been able to actually get a DPX list file to load (hoping Andrea can).

how you tried to construct said list? manually, by script, as output from another program? 



 
BTW, was the 1 line fix in filelist.C an existing bug?

+++ b/cinelerra-5.1/cinelerra/filelist.C
@@ -115,7 +115,7 @@ int FileList::open_file(int rd, int wr)
                                int width = asset->width, height = asset->height;
                                char string[BCTEXTLEN];
                                int len = strlen(list_prefix);
- int ret = fread(string, 1, strlen(list_prefix), stream);
+ int ret = fread(string, 1, len, stream);
                                fclose(stream);
                                result = len == ret ? 0 : 1;
                                if( !result && !strncasecmp(string, list_prefix, len)) { 



yeah, it seem to just finally use this 'len' variable as intended... 



Thanks for the auxiliary help. ...Phyllis

On Sun, Apr 3, 2022 at 10:07 AM Andrew Randrianasulu <randrianasulu@gmail.com> wrote:
try my series? 



On Sunday, April 3, 2022, Phyllis Smith <phylsmith2017@gmail.com> wrote:
I have been testing Manuel's Cinelerra DPXlist changes.  the last 2 days, but have not yet gotten it to compile.  I will send his files/patches but it is included at:


On Sun, Apr 3, 2022 at 8:53 AM Andrew Randrianasulu via Cin <cin@lists.cinelerra-gg.org> wrote:
hopefully "c++17" thing just cosmetical for now? I'll try to remove this line... 





On Saturday, March 5, 2022, Andrew Randrianasulu <randrianasulu@gmail.com> wrote:
note: requires c++17 language support, so my gcc 5.5 out of action.... 

---------- Forwarded message ----------
From: Андрей Рандрианасулу <randrik@mail.ru>
Date: Saturday, March 5, 2022
Subject: cinelerra dpx support patches
To: randrianasulu <randrianasulu@gmail.com>


 
not tested yet
--
Андрей  Рандрианасулу

--
Cin mailing list
Cin@lists.cinelerra-gg.org
https://lists.cinelerra-gg.org/mailman/listinfo/cin