> + /* This is the main thread still going through the shell, or, no > + watchpoint has been set yet. */ > + if (lwp->arch_private == NULL) > + return; Just a really minor nitpick: Would you consider putting the comment inside the if, instead of just before? I think it'd be slightly clearer that way. -- Joel