Inline `ensure_list_params!/1`

elixir-ecto / ecto_sql

SQL-based adapters for Ecto and database migrations

https://hexdocs.pm/ecto_sql

Apache License 2.0

578 stars 312 forks source link

Inline `ensure_list_params!/1` #616

Closed greg-rychlewski closed 5 months ago

greg-rychlewski commented 5 months ago

Would this be worth it since it will be run every time someone executes a query?

josevalim commented 5 months ago

I don't think it would make any measurable difference, it helps a bit more when there is a loop or similar, but it doesn't hurt either. It is your call.