On Fri, Feb 2, 2018 at 9:26 AM, Oliver Ford <ojf...@gmail.com> wrote:

> On Thu, Feb 1, 2018 at 1:46 AM, David G. Johnston
> <david.g.johns...@gmail.com> wrote:
>
> > The three callers of WinGetFuncArgInFrame don't use the isout argument;
> they
> > probably need to read that and a new isexcluded argument.  Start at the
> > head, loop until isout = true || isexcluded = false.
>
> The patch takes a slightly different approach and puts the logic in
> WinGetFuncArgInFrame.
> The "row_is_in_frame" function now returns a specific return code for
> when an Exclude
> clause was matched.


​I would suggest adding constants for the 4 possible results from
row_is_in_frame.

David J.

Reply via email to