Ludovic Courtès writes: > Christopher Baines skribis: > >> These names should be more descriptive. >> >> * guix/store/database.scm (path-id): Rename to select-valid-path-id. >> (sqlite-register): Rename to register-valid-path. >> (register-items): Update accordingly. >> >> Change-Id: I6d4a14d4cde9d71ab34d6ffdbfbfde51b2c0e1db > > OK for ‘register-valid-path’. > > For ‘path-id’ my preference would be ‘valid-path-id’ or keeping > ‘path-id’; ‘select’ looks odd as nothing’s being “selected” here in the > non-SQL sense of the word. The main thing I'm trying to do here is make the procedures that directly interact with the database stand out (similar to the ! suffix convention). Maybe the procedure taking the db as an argument is enough, but to me path-id/valid-path-id look too much like record accessors.