elixir-sqlite / exqlite

An SQLite3 driver for Elixir
https://hexdocs.pm/exqlite
MIT License
217 stars 48 forks source link

Pre-define atoms #299

Closed ruslandoga closed 1 month ago

ruslandoga commented 1 month ago

This PR predefines atoms like Rustler does. I think this improves readability and kind of makes it safer against typos. Once or if #298 is merged, many of the currently used atoms could be replaced with raise_badarg() function. I'll also leave some notes on atoms that I think shouldn't be atoms and rather SQLite errors.