On 02.01.2015 23:04, Branko Čibej wrote:
> On 02.01.2015 13:31, Stefan Fuhrmann wrote:
>> Now there are only two points left
>>
>> * Be explicit in our coding style guide about the mandatory
>> svn_ prefix in all non-static identifiers.
> If this is not already stated explicitly in HACKING, it should be. You
> just volunteered to check and fix. :)
>
>> * Don't shorten library names. No need to fix that for older
>> FS but get it right in FSX.
> Ack.
On second thoughts, consider what we're doing on the authzperf branch:
* "local" symbols that are not exported (struct and enum names and
typedefs) are simply called authz_something_t;
* "local" function names are svn_authz__something, even though there's
no "libsvn_authz" or svn_authz.h. The symbol name should indicate
grouping by logical module rather than by library name; we do this
all over the place in libsvn_subr, and it should be just fine in
other libraries (IMO, as long as module implementations aren't split
amongst several libraries).
-- Brane
Received on 2015-01-04 20:13:17 CET