14 Nov
2016
14 Nov
'16
3:52 p.m.
On Nov 14, 2016, at 2:00 PM, Kamil Jońca <kjonca@o2.pl> wrote:
Naive question: shouldn't definition of sql_fall_through_t (in rlm_sql.h) be like this?
--8<---------------cut here---------------start------------->8--- typedef enum { FALL_THROUGH_DEFAULT = -1, FALL_THROUGH_NO, FALL_THROUGH_YES
} sql_fall_through_t; --8<---------------cut here---------------end--------------->8---
current seems to be inconsistent with internal dictionary. But it is only quick look of novice :)
Looks consistent to me. Though technically we should explicitly set FALL_THROUGH_NO = 0 and FALL_THROUGH_YES = 1. FALL_THROUGH_DEFAULT is not intended to be set by the user. -Arran