[pacman-dev] [PATCH] Remove need to explicitly register the local DB

Xyne xyne at archlinux.ca
Sat Jan 29 04:09:30 EST 2011


Allan McRae wrote:

> The lazy initialisation means that nothing is actually loaded when 
> registering the local database.  The local database is only read in as 
> needed and only the parts that are needed are read in.
> 
> In fact, all that is done registering the local database is creating a 
> pmdb_t srtuct and assigning the local database operation struct to it. 
>   That is the "little to no fallout" if you do not use the local database.
> 
> Allan
> 

Thanks for the clarification.

Even if it is negligible, it's munging separate logic together. Given that the
purpose is presumably to save coders a few lines of code, I still think it
would make more sense to use a wrapper function to do that when getting the
handle.

Don't worry though, I'll leave it there as I have no intention of pestering you
about something that is effectively irrelevant. I just see it as lazy design in
a bad way.

Regards,
Xyne


More information about the pacman-dev mailing list