Thanks for removing stuff that exists in the NT kernel as sanity
checks -- the entire MS dev team must be wrong, thanks for correcting
them all :)

Make sure not to ask "anyone can explain these checks? they seem
useless to me" when removing stuff like this.
Best regards,
Alex Ionescu


On Sat, Sep 24, 2016 at 2:30 AM,  <dchapys...@svn.reactos.org> wrote:
> Author: dchapyshev
> Date: Sat Sep 24 09:30:06 2016
> New Revision: 72787
>
> URL: http://svn.reactos.org/svn/reactos?rev=72787&view=rev
> Log:
> [NTOSKRNL] Remove unneeded sanity checks
>
> Modified:
>     trunk/reactos/ntoskrnl/include/internal/ke_x.h
>
> Modified: trunk/reactos/ntoskrnl/include/internal/ke_x.h
> URL: 
> http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/include/internal/ke_x.h?rev=72787&r1=72786&r2=72787&view=diff
> ==============================================================================
> --- trunk/reactos/ntoskrnl/include/internal/ke_x.h      [iso-8859-1] 
> (original)
> +++ trunk/reactos/ntoskrnl/include/internal/ke_x.h      [iso-8859-1] Sat Sep 
> 24 09:30:06 2016
> @@ -25,7 +25,6 @@
>                                                                              \
>      /* Sanity checks */                                                     \
>      ASSERT(KeGetCurrentIrql() <= APC_LEVEL);                                \
> -    ASSERT(_Thread == KeGetCurrentThread());                                \
>      ASSERT((_Thread->SpecialApcDisable <= 0) &&                             \
>             (_Thread->SpecialApcDisable != -32768));                         \
>                                                                              \
> @@ -42,7 +41,6 @@
>                                                                              \
>      /* Sanity checks */                                                     \
>      ASSERT(KeGetCurrentIrql() <= APC_LEVEL);                                \
> -    ASSERT(_Thread == KeGetCurrentThread());                                \
>      ASSERT(_Thread->SpecialApcDisable < 0);                                 \
>                                                                              \
>      /* Leave region and check if APCs are OK now */                         \
> @@ -66,7 +64,6 @@
>      PKTHREAD _Thread = KeGetCurrentThread();                                \
>                                                                              \
>      /* Sanity checks */                                                     \
> -    ASSERT(_Thread == KeGetCurrentThread());                                \
>      ASSERT((_Thread->KernelApcDisable <= 0) &&                              \
>             (_Thread->KernelApcDisable != -32768));                          \
>                                                                              \
> @@ -82,7 +79,6 @@
>      PKTHREAD _Thread = KeGetCurrentThread();                                \
>                                                                              \
>      /* Sanity checks */                                                     \
> -    ASSERT(_Thread == KeGetCurrentThread());                                \
>      ASSERT(_Thread->KernelApcDisable < 0);                                  \
>                                                                              \
>      /* Enable Kernel APCs */                                                \
>
>

_______________________________________________
Ros-dev mailing list
Ros-dev@reactos.org
http://www.reactos.org/mailman/listinfo/ros-dev

Reply via email to