[PATCH 0/3] DP: Handle "/" separator in debug path for GCC

[PATCH 0/3] DP: Handle "/" separator in debug path for GCC

[Star Zeng]

Star Zeng (3):
  PerformancePkg Dp_App: Handle "/" separator in debug path for GCC
    build
  ShellPkg UefiDpLib: Handle "/" separator in debug path for GCC build
  MdeModulePkg BmPerf: Handle "/" separator in debug path for GCC build

 MdeModulePkg/Library/UefiBootManagerLib/BmPerformance.c | 4 ++--
 PerformancePkg/Dp_App/DpUtilities.c                     | 2 +-
 ShellPkg/Library/UefiDpLib/DpUtilities.c                | 2 +-
 3 files changed, 4 insertions(+), 4 deletions(-)

-- 
2.7.0.windows.1

[PATCH 1/3] PerformancePkg Dp_App: Handle "/" separator in debug path for GCC build

[Star Zeng]

Cc: Liming Gao <liming.gao@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Star Zeng <star.zeng@intel.com>
---
 PerformancePkg/Dp_App/DpUtilities.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/PerformancePkg/Dp_App/DpUtilities.c b/PerformancePkg/Dp_App/DpUtilities.c
index d5840e8d6f83..d3a9b6eab3fa 100644
--- a/PerformancePkg/Dp_App/DpUtilities.c
+++ b/PerformancePkg/Dp_App/DpUtilities.c
@@ -169,7 +169,7 @@ GetShortPdbFileName (
     for (EndIndex = 0; PdbFileName[EndIndex] != 0; EndIndex++)
       ;
     for (IndexA = 0; PdbFileName[IndexA] != 0; IndexA++) {
-      if (PdbFileName[IndexA] == '\\') {
+      if ((PdbFileName[IndexA] == '\\') || (PdbFileName[IndexA] == '/')) {
         StartIndex = IndexA + 1;
       }
 
-- 
2.7.0.windows.1

[PATCH 2/3] ShellPkg UefiDpLib: Handle "/" separator in debug path for GCC build

[Star Zeng]

Cc: Liming Gao <liming.gao@intel.com>
Cc: Ruiyu Ni <ruiyu.ni@intel.com>
Cc: Jaben Carsey <jaben.carsey@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Star Zeng <star.zeng@intel.com>
---
 ShellPkg/Library/UefiDpLib/DpUtilities.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ShellPkg/Library/UefiDpLib/DpUtilities.c b/ShellPkg/Library/UefiDpLib/DpUtilities.c
index 64f1830fcc92..c60d27d8613f 100644
--- a/ShellPkg/Library/UefiDpLib/DpUtilities.c
+++ b/ShellPkg/Library/UefiDpLib/DpUtilities.c
@@ -154,7 +154,7 @@ DpGetShortPdbFileName (
     for (EndIndex = 0; PdbFileName[EndIndex] != 0; EndIndex++)
       ;
     for (IndexA = 0; PdbFileName[IndexA] != 0; IndexA++) {
-      if (PdbFileName[IndexA] == '\\') {
+      if ((PdbFileName[IndexA] == '\\') || (PdbFileName[IndexA] == '/')) {
         StartIndex = IndexA + 1;
       }
 
-- 
2.7.0.windows.1

[PATCH 3/3] MdeModulePkg BmPerf: Handle "/" separator in debug path for GCC build

[Star Zeng]

Cc: Liming Gao <liming.gao@intel.com>
Cc: Ruiyu Ni <ruiyu.ni@intel.com>
Cc: Feng Tian <feng.tian@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Star Zeng <star.zeng@intel.com>
---
 MdeModulePkg/Library/UefiBootManagerLib/BmPerformance.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/MdeModulePkg/Library/UefiBootManagerLib/BmPerformance.c b/MdeModulePkg/Library/UefiBootManagerLib/BmPerformance.c
index 0abd0194409c..4d4495b2a826 100644
--- a/MdeModulePkg/Library/UefiBootManagerLib/BmPerformance.c
+++ b/MdeModulePkg/Library/UefiBootManagerLib/BmPerformance.c
@@ -3,7 +3,7 @@
   performance, all the function will only include if the performance
   switch is set.
 
-Copyright (c) 2004 - 2015, Intel Corporation. All rights reserved.<BR>
+Copyright (c) 2004 - 2017, Intel Corporation. All rights reserved.<BR>
 This program and the accompanying materials
 are licensed and made available under the terms and conditions of the BSD License
 which accompanies this distribution.  The full text of the license may be found at
@@ -49,7 +49,7 @@ BmGetShortPdbFileName (
       ;
 
     for (Index = 0; PdbFileName[Index] != 0; Index++) {
-      if (PdbFileName[Index] == '\\') {
+      if ((PdbFileName[Index] == '\\') || (PdbFileName[Index] == '/')) {
         StartIndex = Index + 1;
       }
 
-- 
2.7.0.windows.1

Re: [PATCH 3/3] MdeModulePkg BmPerf: Handle "/" separator in debug path for GCC build

[Tian, Feng]

Reviewed-by: Feng Tian <feng.tian@Intel.com>

Thanks
Feng

-----Original Message-----
From: Zeng, Star 
Sent: Wednesday, March 15, 2017 1:51 PM
To: edk2-devel@lists.01.org
Cc: Zeng, Star <star.zeng@intel.com>; Gao, Liming <liming.gao@intel.com>; Ni, Ruiyu <ruiyu.ni@intel.com>; Tian, Feng <feng.tian@intel.com>
Subject: [PATCH 3/3] MdeModulePkg BmPerf: Handle "/" separator in debug path for GCC build

Cc: Liming Gao <liming.gao@intel.com>
Cc: Ruiyu Ni <ruiyu.ni@intel.com>
Cc: Feng Tian <feng.tian@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Star Zeng <star.zeng@intel.com>
---
 MdeModulePkg/Library/UefiBootManagerLib/BmPerformance.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/MdeModulePkg/Library/UefiBootManagerLib/BmPerformance.c b/MdeModulePkg/Library/UefiBootManagerLib/BmPerformance.c
index 0abd0194409c..4d4495b2a826 100644
--- a/MdeModulePkg/Library/UefiBootManagerLib/BmPerformance.c
+++ b/MdeModulePkg/Library/UefiBootManagerLib/BmPerformance.c
@@ -3,7 +3,7 @@
   performance, all the function will only include if the performance
   switch is set.
 
-Copyright (c) 2004 - 2015, Intel Corporation. All rights reserved.<BR>
+Copyright (c) 2004 - 2017, Intel Corporation. All rights reserved.<BR>
 This program and the accompanying materials  are licensed and made available under the terms and conditions of the BSD License  which accompanies this distribution.  The full text of the license may be found at @@ -49,7 +49,7 @@ BmGetShortPdbFileName (
       ;
 
     for (Index = 0; PdbFileName[Index] != 0; Index++) {
-      if (PdbFileName[Index] == '\\') {
+      if ((PdbFileName[Index] == '\\') || (PdbFileName[Index] == '/')) 
+ {
         StartIndex = Index + 1;
       }
 
--
2.7.0.windows.1

Re: [PATCH 0/3] DP: Handle "/" separator in debug path for GCC

[Gao, Liming]

Reviewed-by: Liming Gao <liming.gao@intel.com>

> -----Original Message-----
> From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Star Zeng
> Sent: Wednesday, March 15, 2017 1:50 PM
> To: edk2-devel@lists.01.org
> Cc: Zeng, Star <star.zeng@intel.com>
> Subject: [edk2] [PATCH 0/3] DP: Handle "/" separator in debug path for GCC
> 
> Star Zeng (3):
>   PerformancePkg Dp_App: Handle "/" separator in debug path for GCC
>     build
>   ShellPkg UefiDpLib: Handle "/" separator in debug path for GCC build
>   MdeModulePkg BmPerf: Handle "/" separator in debug path for GCC build
> 
>  MdeModulePkg/Library/UefiBootManagerLib/BmPerformance.c | 4 ++--
>  PerformancePkg/Dp_App/DpUtilities.c                     | 2 +-
>  ShellPkg/Library/UefiDpLib/DpUtilities.c                | 2 +-
>  3 files changed, 4 insertions(+), 4 deletions(-)
> 
> --
> 2.7.0.windows.1
> 
> _______________________________________________
> edk2-devel mailing list
> edk2-devel@lists.01.org
> https://lists.01.org/mailman/listinfo/edk2-devel

Re: [PATCH 3/3] MdeModulePkg BmPerf: Handle "/" separator in debug path for GCC build

[Ni, Ruiyu]

Reviewed-by: Ruiyu Ni <ruiyu.ni@intel.com>

Thanks/Ray

> -----Original Message-----
> From: Tian, Feng
> Sent: Wednesday, March 15, 2017 1:52 PM
> To: Zeng, Star <star.zeng@intel.com>; edk2-devel@lists.01.org
> Cc: Gao, Liming <liming.gao@intel.com>; Ni, Ruiyu <ruiyu.ni@intel.com>;
> Tian, Feng <feng.tian@intel.com>
> Subject: RE: [PATCH 3/3] MdeModulePkg BmPerf: Handle "/" separator in
> debug path for GCC build
> 
> Reviewed-by: Feng Tian <feng.tian@Intel.com>
> 
> Thanks
> Feng
> 
> -----Original Message-----
> From: Zeng, Star
> Sent: Wednesday, March 15, 2017 1:51 PM
> To: edk2-devel@lists.01.org
> Cc: Zeng, Star <star.zeng@intel.com>; Gao, Liming <liming.gao@intel.com>;
> Ni, Ruiyu <ruiyu.ni@intel.com>; Tian, Feng <feng.tian@intel.com>
> Subject: [PATCH 3/3] MdeModulePkg BmPerf: Handle "/" separator in debug
> path for GCC build
> 
> Cc: Liming Gao <liming.gao@intel.com>
> Cc: Ruiyu Ni <ruiyu.ni@intel.com>
> Cc: Feng Tian <feng.tian@intel.com>
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Star Zeng <star.zeng@intel.com>
> ---
>  MdeModulePkg/Library/UefiBootManagerLib/BmPerformance.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/MdeModulePkg/Library/UefiBootManagerLib/BmPerformance.c
> b/MdeModulePkg/Library/UefiBootManagerLib/BmPerformance.c
> index 0abd0194409c..4d4495b2a826 100644
> --- a/MdeModulePkg/Library/UefiBootManagerLib/BmPerformance.c
> +++ b/MdeModulePkg/Library/UefiBootManagerLib/BmPerformance.c
> @@ -3,7 +3,7 @@
>    performance, all the function will only include if the performance
>    switch is set.
> 
> -Copyright (c) 2004 - 2015, Intel Corporation. All rights reserved.<BR>
> +Copyright (c) 2004 - 2017, Intel Corporation. All rights reserved.<BR>
>  This program and the accompanying materials  are licensed and made
> available under the terms and conditions of the BSD License  which
> accompanies this distribution.  The full text of the license may be found at
> @@ -49,7 +49,7 @@ BmGetShortPdbFileName (
>        ;
> 
>      for (Index = 0; PdbFileName[Index] != 0; Index++) {
> -      if (PdbFileName[Index] == '\\') {
> +      if ((PdbFileName[Index] == '\\') || (PdbFileName[Index] == '/'))
> + {
>          StartIndex = Index + 1;
>        }
> 
> --
> 2.7.0.windows.1

Re: [PATCH 2/3] ShellPkg UefiDpLib: Handle "/" separator in debug path for GCC build

[Ni, Ruiyu]

Reviewed-by: Ruiyu Ni <ruiyu.ni@intel.com>

Thanks/Ray

> -----Original Message-----
> From: Zeng, Star
> Sent: Wednesday, March 15, 2017 1:51 PM
> To: edk2-devel@lists.01.org
> Cc: Zeng, Star <star.zeng@intel.com>; Gao, Liming <liming.gao@intel.com>;
> Ni, Ruiyu <ruiyu.ni@intel.com>; Carsey, Jaben <jaben.carsey@intel.com>
> Subject: [PATCH 2/3] ShellPkg UefiDpLib: Handle "/" separator in debug path
> for GCC build
> 
> Cc: Liming Gao <liming.gao@intel.com>
> Cc: Ruiyu Ni <ruiyu.ni@intel.com>
> Cc: Jaben Carsey <jaben.carsey@intel.com>
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Star Zeng <star.zeng@intel.com>
> ---
>  ShellPkg/Library/UefiDpLib/DpUtilities.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/ShellPkg/Library/UefiDpLib/DpUtilities.c
> b/ShellPkg/Library/UefiDpLib/DpUtilities.c
> index 64f1830fcc92..c60d27d8613f 100644
> --- a/ShellPkg/Library/UefiDpLib/DpUtilities.c
> +++ b/ShellPkg/Library/UefiDpLib/DpUtilities.c
> @@ -154,7 +154,7 @@ DpGetShortPdbFileName (
>      for (EndIndex = 0; PdbFileName[EndIndex] != 0; EndIndex++)
>        ;
>      for (IndexA = 0; PdbFileName[IndexA] != 0; IndexA++) {
> -      if (PdbFileName[IndexA] == '\\') {
> +      if ((PdbFileName[IndexA] == '\\') || (PdbFileName[IndexA] == '/')) {
>          StartIndex = IndexA + 1;
>        }
> 
> --
> 2.7.0.windows.1