[ros-dev] [ros-diffs] [ion] 24595: - Implement DbgkpQueueMessage. This is the main bi-directional communication routine for the newer non-LPC Debug Object. - Implement DbgkPostFakeProcessCreateMessages (and stub DbgkpPostFakeThreadMessages and DbgkpPostFakeModuleMessages). These are required when attaching to a process after threads have been created and modules loaded, so that the debugger can have a valid state. - Still missing the two functions to Set/Clear the Debug Object, will do these next.

Alex Ionescu ionucu at videotron.ca
Sun Oct 22 07:50:33 CEST 2006


Thomas Weidenmueller wrote:
>>-        RtlCopyMemory(StateChange,
>>+        /* Return our wait state change structure */
>>+        RtlMoveMemory(StateChange,
>>                       &WaitStateChange,
>>                       sizeof(DBGUI_WAIT_STATE_CHANGE));
> 
> 
> RtlMoveMemory doesn't really make sense as the destination and source
> are never overlapping, resulting in unnecessary overhead.
> 
> - Thomas
> _______________________________________________
> Ros-dev mailing list
> Ros-dev at reactos.org
> http://www.reactos.org/mailman/listinfo/ros-dev
> 

Exactly, isn't RtlCopy *slower* because it handles overlapping cases,
and Move is faster because it doesn't?

-- 
Best regards,
Alex Ionescu
Project Lead, TinyKRNL
Kernel-Mode Software Design Engineer, ReactOS


More information about the Ros-dev mailing list