Re: [PATCH 0/6] MR3359: gdiplus: Set GIF properties correctly.

[Esme Povirk (＠madewokherd)]

On Fri Jul 21 22:45:28 2023 +0000, Jeffrey Smith wrote:
> Assuming the property is not present...
> - Currently `x = get_gif_frame_property(frm, fmt, prop);` will result in
>
> `x` being 0.
> - With this change, `get_gif_frame_property(frm, fmt, prop, &x);` will
>
> leave `x` unchanged.
> - Alternately (and more repetitiously) this could be implemented with a
>
> default parameter, and pass in the original value: `x =
> get_gif_frame_property(frm, fmt, prop, x);`
> This functionality is made use of in the following commit.  To create
> the data for `PropertyTagFrameDelay`, we loop over a set of frames, and
> create a list of delays, one per frame.  If a delay is not specified for
> a frame, it will get the same delay as the previous frame.  So if we
> have (GCE w/ delay:10) (frame 1 image) (frame 2 image), we should get
> [10, 10] for the delays, but we currently get [10, 0].

I see, thank you for explaining.