[SSSD] [PATCH] Add a default section to a swtich-statement

Simo Sorce simo at redhat.com
Wed Jan 9 13:38:41 UTC 2013


On Wed, 2013-01-09 at 08:35 -0500, Simo Sorce wrote:
> On Wed, 2013-01-09 at 11:37 +0100, Jakub Hrozek wrote:
> > On Wed, Jan 09, 2013 at 11:31:53AM +0100, Jakub Hrozek wrote:
> > > On Wed, Jan 09, 2013 at 10:51:34AM +0100, Sumit Bose wrote:
> > > > Hi,
> > > > 
> > > > a switch statement which was introduced with "Remove unhelpful vtable
> > > > from sss_cache" has a missing default section which results in a
> > > > compiler warning about an uninitialized variable. The attached patch
> > > > should fix it.
> > > > 
> > > > bye,
> > > > Sumit
> > > 
> > > I don't see the warning even though I seem to be using very similar
> > > CFLAGS to Sumit's. But the patch is correct, Ack.
> > 
> > I fixed a typo in the commit message (s/swtich/switch) and pushed to
> > patch to master.
> 
> I will send a revert with the correct fix! :-(

Revert + fix case.

Simo.

-- 
Simo Sorce * Red Hat, Inc * New York
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Revert-Add-a-default-section-to-a-switch-statement.patch
Type: text/x-patch
Size: 1211 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20130109/1e8bded5/attachment.bin>


More information about the sssd-devel mailing list