[Freeipa-devel] [PATCH] First part of the ELAPI interface (resend)
Dmitri Pal
dpal at redhat.com
Tue Jul 14 15:55:09 UTC 2009
Simo Sorce wrote:
> On Mon, 2009-07-13 at 19:42 -0400, Dmitri Pal wrote:
>
>> Dmitri Pal wrote:
>>
>>> This is just a part of the interface,
>>> a beginning. Most likely some of the functions
>>> will be altered but it is a starting point.
>>> For example in future there will be
>>> a way to override some of the parts of the
>>> default template using the application configuration file.
>>>
>>>
>>> I just need to start somewhere. More to come during the week.
>>>
>
> Haven't read everything, but I saw a couple of things that are not ok.
> 1. the public header requires the private header, this is a nack.
> 2. Never, ever typedef to a pointer.
>
> typedef struct foo foo_t is ok if really necessary
> typedef struct foo * foo_t is an instant-nack
>
> Simo.
>
>
>
We had a phone conversation and decided that we will not try to hide the
"struct collection_item *" behind typedefs at all.
I will amend the patch accordingly.
--
Thank you,
Dmitri Pal
Engineering Manager IPA project,
Red Hat Inc.
-------------------------------
Looking to carve out IT costs?
www.redhat.com/carveoutcosts/
More information about the Freeipa-devel
mailing list