On Mon, Aug 25, 2008 at 7:36 PM, Nagy Gabor <[EMAIL PROTECTED]> wrote:
>> This is an automated email from the git hooks/post-receive script. It
>> was generated because a ref change was pushed to the repository
>> containing the project "The official pacman repository".
>>
>> The branch, master has been updated
>> via d05882db9e417244fa580c4697b45333faffcc79 (commit)
>> via 070135626023caa0186a77f9f846c078e3cf1be8 (commit)
>> via 9dbe5c9d1ee5fb00f25515dfb5a2feb3ae6ebd59 (commit)
>> via a06d0de104d2f03c19b917ea47394d8d6be89a9a (commit)
>> from a4100b384737d6b422a31eb0f6614b4f5f6070cb (commit)
>>
>> Those revisions listed above that are new to this repository have
>> not appeared on any other notification email; so we list those
>> revisions in full, below.
>>
>> - Log
>> -----------------------------------------------------------------
>> commit d05882db9e417244fa580c4697b45333faffcc79 Author: Xavier
>> Chantry <[EMAIL PROTECTED]> Date: Sun Aug 17 19:36:28 2008 +0200
>>
>> Rename alpm_db_get{pkg,grp}cache to alpm_db_get_{pkg,grp}cache
>>
>> This is more consistent with the private functions :
>> _alpm_db_get_{pkg,grp}cache
>>
>> Signed-off-by: Xavier Chantry <[EMAIL PROTECTED]>
>> Signed-off-by: Dan McGee <[EMAIL PROTECTED]>
>>
>
> I am not a big fan of public function renaming, but if this was
> accepted, I bring up an other annoying function name convention (imho):
> alpm_dep_get_string()
> alpm_get_md5sum()
> The return values of these functions must be freed unlike other
> alpm_*_get_ functions. (The different behaviour can be suspected from
> "const char*" vs "char*" only). I may missed some other examples.
>
> IMHO in these cases the use of "compute" instead of "get" would be more
> suggestive, see alpm_pkg_compute_requiredby().
I think you are right in this case- if you want to submit a patch I'd
welcome it.
-Dan
_______________________________________________
pacman-dev mailing list
[email protected]
http://archlinux.org/mailman/listinfo/pacman-dev
