FEAT: garbage collect images on Win32 backend

[hiiamboris]

Replaces #4300
see the description there

[dockimbel]

@hiiamboris It still seems to be in conflict, could you please solve those?

[hiiamboris]

Sure

[hiiamboris]

OTOH, I don't see how this work is relevant to the new D2D backend.

I see @qtxie considered freeing up image resources:

red/runtime/platform/image-wic.reds

Line 53 in 50d58d6

; 2: if set, the image has been marked by the GC

red/runtime/collector.reds

Line 244 in 50d58d6

#if any [OS = 'macOS OS = 'Linux OS = 'Windows][

But it doesn't seem to work.
I propose he finishes what he has planned.

[dockimbel]

I think this PR might still be relevant. @qtxie what do you think?

[qtxie]

@hiiamboris OK. So you mark it by using series/flags. You can remove the comment for img-node!.

[hiiamboris]

@qtxie what comment? ;)

[qtxie]

@hiiamboris This one:

red/runtime/platform/image-wic.reds

Line 53 in 50d58d6

; 2: if set, the image has been marked by the GC

[hiiamboris]

@qtxie but did you implement any part of GC support for images in D2D backend or not?

[qtxie]

@hiiamboris Nope. The same as the other platforms, no GC support for image was implemented.

[hiiamboris]

I thought it's supported on other platforms?

[greggirwin]

Well, we certainly want it to be implemented.

[greggirwin]

@qtxie @dockimbel let's prioritize this. It's an absolute showstopper for real world apps, and seems like we're close to a solution from the above comments.

[dockimbel]

"garbage collect images on Win32 backend"

What about the other backends? We need all the backends to support the same features.

[hiiamboris]

Since we're on to 1.0, this GC has no value anyway. #4837
Need a scalable realtime implementation built first.

[dockimbel]

Since we're on to 1.0, this GC has no value anyway. #4837 Need a scalable realtime implementation built first.

The current GC is fine. See my detailed comments.