Re: [PATCH v2 0/5] MR635: Draft: Implement transform graph. Partically implement effect drawing.

[Nikolay Sivov (＠nsivov)]

Node manipulation might be fine, I haven't checked it in full. However my impression was that we don't really need it right now? I believe I saw some examples where ID2D1DrawTransform is implemented for *Impl object directly, for which runtime presumably calls SetDrawInfo, letting effect set back shaders and resources.

Regarding prototyped drawing, creating render target on every draw seems wrong. It's possible you'll have to recreate something, but only when when parameters or inputs have changed. I certainly don't have a full picture of how this works, but shouldn't we be using MapOutputRectToInputRects() at some point?

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/635#note_6266