[rlm_sql] Callback sql_fields

[Alan DeKok]

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.