[llvm-commits] [llvm] r160170 - /llvm/trunk/include/llvm/Support/CommandLine.h

[Duncan Sands]

Hi Alexander,
> Initializers for some fields were missing in Option::Option

did it matter?  If so, please add a testcase.  If not, why this change?

Ciao, Duncan.

>
> Modified:
>      llvm/trunk/include/llvm/Support/CommandLine.h
>
> Modified: llvm/trunk/include/llvm/Support/CommandLine.h
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Support/CommandLine.h?rev=160170&r1=160169&r2=160170&view=diff
> ==============================================================================
> --- llvm/trunk/include/llvm/Support/CommandLine.h (original)
> +++ llvm/trunk/include/llvm/Support/CommandLine.h Fri Jul 13 07:55:23 2012
> @@ -217,11 +217,11 @@
>     void setMiscFlag(enum MiscFlags M) { Misc |= M; }
>     void setPosition(unsigned pos) { Position = pos; }
>   protected:
> -  explicit Option(enum NumOccurrencesFlag OccurrencesFlag,
> +  explicit Option(enum NumOccurrencesFlag OccurrencesFlag,
>                     enum OptionHidden Hidden)
> -    : NumOccurrences(0), Occurrences(OccurrencesFlag), HiddenFlag(Hidden),
> -      Formatting(NormalFormatting), Position(0),
> -      AdditionalVals(0), NextRegistered(0),
> +    : NumOccurrences(0), Occurrences(OccurrencesFlag), Value(0),
> +      HiddenFlag(Hidden), Formatting(NormalFormatting), Misc(0),
> +      Position(0), AdditionalVals(0), NextRegistered(0),
>         ArgStr(""), HelpStr(""), ValueStr("") {
>     }
>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>