Reply to thread

Message

<blockquote data-quote="gemx" data-source="post: 561196" data-attributes="member: 26324">Thanks this also saves several ms of CPU time <img src="data:image/gif;base64,R0lGODlhAQABAIAAAAAAAP///yH5BAEAAAAALAAAAAABAAEAAAIBRAA7" class="smilie smilie--sprite smilie--sprite1" alt=":)" title="Smile&nbsp; &nbsp; :)" loading="lazy" data-shortname=":)" />Here is the updated patchBTW: I don't agree with I-Loop in the aspect as the analog clock enhancement is clearly a new feature whereas this patch helps to save CPU time under certain circumstances.I must admit that this is also not really a bugfix - it just improves timing - so it's something in between a feature and a bugfix.At least the LockRectangle call instead of SurfaceLoader.SaveToStream already saves several ms.Also due to the clever locking mechanism inside Framegrabber between OnNewFrame and GetCurrentImage() this won't stall the rendering process not matter how long it takes.</blockquote>

[QUOTE="gemx, post: 561196, member: 26324"] Thanks this also saves several ms of CPU time :) Here is the updated patch BTW: I don't agree with I-Loop in the aspect as the analog clock enhancement is clearly a new feature whereas this patch helps to save CPU time under certain circumstances. I must admit that this is also not really a bugfix - it just improves timing - so it's something in between a feature and a bugfix. At least the LockRectangle call instead of SurfaceLoader.SaveToStream already saves several ms. Also due to the clever locking mechanism inside Framegrabber between OnNewFrame and GetCurrentImage() this won't stall the rendering process not matter how long it takes. [/QUOTE]

Verification