Re: [patch 2/8] FatPkg\EnhancedFatDxe: Initialize variable after declaration

["Bi, Dandan" <dandan.bi@intel.com>]

Got it ! I will submit a new patch. Thanks!

Regards,
Dandan

-----Original Message-----
From: Ni, Ruiyu 
Sent: Friday, December 9, 2016 8:58 AM
To: Bi, Dandan <dandan.bi@intel.com>; edk2-devel@lists.01.org
Subject: RE: [patch 2/8] FatPkg\EnhancedFatDxe: Initialize variable after declaration

Dandan,
Could you please create a global array mMonthDays?
UINT8  mMonthDays[] = { 31, 28, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31 };

Regards,
Ray

>-----Original Message-----
>From: Bi, Dandan
>Sent: Thursday, December 8, 2016 6:54 PM
>To: edk2-devel@lists.01.org
>Cc: Ni, Ruiyu <ruiyu.ni@intel.com>
>Subject: [patch 2/8] FatPkg\EnhancedFatDxe: Initialize variable after 
>declaration
>
>Cc: Ruiyu Ni <ruiyu.ni@intel.com>
>Contributed-under: TianoCore Contribution Agreement 1.0
>Signed-off-by: Dandan Bi <dandan.bi@intel.com>
>---
> FatPkg/EnhancedFatDxe/Misc.c | 14 +++++++++++++-
> 1 file changed, 13 insertions(+), 1 deletion(-)
>
>diff --git a/FatPkg/EnhancedFatDxe/Misc.c 
>b/FatPkg/EnhancedFatDxe/Misc.c index f91759c..6ad688c 100644
>--- a/FatPkg/EnhancedFatDxe/Misc.c
>+++ b/FatPkg/EnhancedFatDxe/Misc.c
>@@ -696,15 +696,27 @@ Returns:
>   TRUE                  - The time is valid.
>   FALSE                 - The time is not valid.
>
> --*/
> {
>-  static UINT8  MonthDays[] = { 31, 28, 31, 30, 31, 30, 31, 31, 30, 
>31, 30, 31 };
>+  STATIC UINT8  MonthDays[12];
>   UINTN         Day;
>   BOOLEAN       ValidTime;
>
>   ValidTime = TRUE;
>+  MonthDays[0] = 31;
>+  MonthDays[1] = 28;
>+  MonthDays[2] = 31;
>+  MonthDays[3] = 30;
>+  MonthDays[4] = 31;
>+  MonthDays[5] = 30;
>+  MonthDays[6] = 31;
>+  MonthDays[7] = 31;
>+  MonthDays[8] = 30;
>+  MonthDays[9] = 31;
>+  MonthDays[10] = 30;
>+  MonthDays[11] = 31;
>
>   //
>   // Check the fields for range problems
>   // Fat can only support from 1980
>   //
>--
>1.9.5.msysgit.1