On Fri, Feb 2, 2018 at 9:26 AM, Oliver Ford <ojford@gmail.com> wrote:
On Thu, Feb 1, 2018 at 1:46 AM, David G. Johnston <david.g.johnston@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.
