On Wed, Jun 27, 2018 at 3:13 PM Kingsley G. Morse Jr. <kings...@loaner.com>
wrote:
> Hi Dan,
>
> Thank you very much for freely sharing your time,
> skills and software.
>
> I can imagine mlt becoming even more popular.
>
> The main reason I'm writing is I happened to
> notice a plan to use mlt_pool by default.
>
> Maybe you remember it resulted in a large,
> wasteful memory footprint back in March.
>
> (That email follows.)
>
>
The biggest improvement in memory reduction from your case came from the
change in the affine transition:
https://github.com/mltframework/mlt/commit/e426a4ed62c7f674db4498766d6928a9598a7ca9
> You very reasonably and responsibly anticipated
> more testing.
>
> Humble suggestion:
>
> Use my little kdenlive project to test if mlt_pool
> still uses too much memory.
>
> All my files are at
>
> https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=890848
>
> I'd like to avoid a regression.
>
>
The change to stop using mlt_pool is causing stability regressions, which
is less desirable than the memory saving it provides.
> Thanks,
> Kingsley
>
> On 06/27/2018 14:14, GitHub wrote:
> > Branch: refs/heads/master
> > Home: https://github.com/mltframework/mlt
> > Commit: 6f7e7915b039e24cd82207d7db9ac780f0d84bb3
> >
> https://github.com/mltframework/mlt/commit/6f7e7915b039e24cd82207d7db9ac780f0d84bb3
> > Author: Dan Dennedy <d...@dennedy.org>
> > Date: 2018-06-27 (Wed, 27 Jun 2018)
> >
> > Changed paths:
> > M src/framework/mlt_pool.c
> >
> > Log Message:
> > -----------
> > Use mlt_pool by default pending more testing.
> >
> > Use `-DUSE_MLT_POOL=0` in CFLAGS to turn off mlt_pool.
> >
> > Fixes https://github.com/mltframework/shotcut/issues/604
> >
> >
> >
> > **NOTE:** This service been marked for deprecation:
> https://developer.github.com/changes/2018-04-25-github-services-deprecation/
> >
> > Functionality will be removed from GitHub.com on January 31st,
> 2019.
>
> >
> ------------------------------------------------------------------------------
> > Check out the vibrant tech community on one of the world's most
> > engaging tech sites, Slashdot.org! http://sdm.link/slashdot
>
> > _______________________________________________
> > Mlt-devel mailing list
> > Mlt-devel@lists.sourceforge.net
> > https://lists.sourceforge.net/lists/listinfo/mlt-devel
>
> >> Date: Fri, 16 Mar 2018 16:39:15 -0700
> >> From: GitHub <nore...@github.com>
> >> To: mlt-devel@lists.sourceforge.net
> >> Message-ID:
> <5aac55a3cdd15_183ac2ac3e0121c0897...@hookshot-fe-dfcc362.cp1-iad.github.net.mail
> >
> >>
> >> Branch: refs/heads/master
> >> Home: https://github.com/mltframework/mlt
> >> Commit: 71167e65b4ac126546b7524e406bd81f991c2abf
> >>
> https://github.com/mltframework/mlt/commit/71167e65b4ac126546b7524e406bd81f991c2abf
> >> Author: Dan Dennedy <d...@dennedy.org>
> >> Date: 2018-03-16 (Fri, 16 Mar 2018)
> >>
> >> Changed paths:
> >> M src/framework/mlt_pool.c
> >>
> >> Log Message:
> >> -----------
> >> Require C macro USE_MLT_POOL to use memory pool.
> >>
> >> mlt_pool can result in a large, wasteful memory footprint for the
> >> application. Relates to issue #271. Add "-DUSE_MLT_POOL" to CFLAGS to
> >> make MLT use its mlt_pool memory pool. Otherwise, it will use malloc(),
> >> realloc(), and free().
> >>
> >>
> >> Commit: e426a4ed62c7f674db4498766d6928a9598a7ca9
> >>
> https://github.com/mltframework/mlt/commit/e426a4ed62c7f674db4498766d6928a9598a7ca9
> >> Author: Dan Dennedy <d...@dennedy.org>
> >> Date: 2018-03-16 (Fri, 16 Mar 2018)
> >>
> >> Changed paths:
> >> M src/modules/plus/transition_affine.c
> >>
> >> Log Message:
> >> -----------
> >> Reduce memory usage of affine esp. with large images.
> >>
> >>
> >> Compare:
> https://github.com/mltframework/mlt/compare/3021e900b5c6...e426a4ed62c7
> >> ----==_mimepart_5aac55a3cd983_183ac2ac3e0121c08977be
> >> Content-Type: text/plain; charset="us-ascii"
> >> MIME-Version: 1.0
> >> Content-Transfer-Encoding: 7bit
> >> Content-Disposition: inline
> >>
> >> _______________________________________________
> >> Mlt-devel mailing list
> >> Mlt-devel@lists.sourceforge.net
> >> https://lists.sourceforge.net/lists/listinfo/mlt-devel
> >>
> >> ----==_mimepart_5aac55a3cd983_183ac2ac3e0121c08977be--
>
> --
> Time is the fire in which we all burn.
>
>
>
> ------------------------------------------------------------------------------
> Check out the vibrant tech community on one of the world's most
> engaging tech sites, Slashdot.org! http://sdm.link/slashdot
> _______________________________________________
> Mlt-devel mailing list
> Mlt-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/mlt-devel
>
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Mlt-devel mailing list
Mlt-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mlt-devel