Hi,
ini_config.c got too big. It became to hard to change things and add new functionality. This patch splits it into several files. See the patch comment for more details. No functional changes though.
Dmitri Pal wrote:
Hi,
ini_config.c got too big. It became to hard to change things and add new functionality. This patch splits it into several files. See the patch comment for more details. No functional changes though.
Is it Ok if I continue my work on top of this patch or I should stay away until it is committed? This is a drastic change and my next step would pretty intrusive too so it would be hard to get back and address comments re this patch if any if I get too deep.
sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://fedorahosted.org/mailman/listinfo/sssd-devel
Go ahead. Any nack will only come from Makefile.am
On Apr 16, 2010, at 4:43 PM, Dmitri Pal dpal@redhat.com wrote:
Dmitri Pal wrote:
Hi,
ini_config.c got too big. It became to hard to change things and add new functionality. This patch splits it into several files. See the patch comment for more details. No functional changes though.
Is it Ok if I continue my work on top of this patch or I should stay away until it is committed? This is a drastic change and my next step would pretty intrusive too so it would be hard to get back and address comments re this patch if any if I get too deep.
sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://fedorahosted.org/mailman/listinfo/sssd-devel
-- Thank you, Dmitri Pal
Engineering Manager IPA project, Red Hat Inc.
Looking to carve out IT costs? www.redhat.com/carveoutcosts/
sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://fedorahosted.org/mailman/listinfo/sssd-devel
On 04/15/2010 05:53 PM, Dmitri Pal wrote:
Hi,
ini_config.c got too big. It became to hard to change things and add new functionality. This patch splits it into several files. See the patch comment for more details. No functional changes though.
Ack.
On 04/21/2010 04:12 PM, Stephen Gallagher wrote:
On 04/15/2010 05:53 PM, Dmitri Pal wrote:
Hi,
ini_config.c got too big. It became to hard to change things and add new functionality. This patch splits it into several files. See the patch comment for more details. No functional changes though.
Ack.
Pushed to master.
sssd-devel@lists.fedorahosted.org