[rlm_sql] Callback sql_fields

[Geaaru]

Thanks!

On Fri, Jun 15, 2018, 18:06 Alan DeKok <aland at deployingradius.com> wrote:

> On Jun 15, 2018, at 11:48 AM, Geaaru <geaaru at gmail.com> wrote:
> > after a diff of rlm_sql_ocioracle.c or rlm_sql_mysql.c between version
> > 3.0.8 and last 3.0.17 I see that on rlm_sql_module_t structure is
> > present sql_fields callback, but I don't see on rlm_sql.c code where it
> > is used.
> >
> > 1) It is related a new callback for next releases ?
>
>   It's historical.
>
> > 2) What is target of sql_fields callback ? The idea is store column
> > data pointer directly on rlm_sql strucuture to avoid duplicate code on
> > handle array of strings ?
>
>   It's historical code which isn't used.  It should be deleted.
>
> > 3) sql_store_result instead wil be removed ?
>
>   That will likely be removed, too.
>
>   Alan DeKok.
>
>