Hi, Samer

This enum is from UEFI spec and the I think we'd better align with spec's 
definition.

Siyuan

-----Original Message-----
From: El-Haj-Mahmoud, Samer [mailto:samer.el-haj-mahm...@hpe.com] 
Sent: Thursday, September 17, 2015 12:29 AM
To: edk2-devel@lists.01.org
Cc: Fu, Siyuan <siyuan...@intel.com>; El-Haj-Mahmoud, Samer 
<samer.el-haj-mahm...@hpe.com>
Subject: RE: [PATCH] MdePkg: Http.h - Add HttpMethodMax to EFI_HTTP_METHOD

Can someone review this change please?

-----Original Message-----
From: El-Haj-Mahmoud, Samer 
Sent: Monday, September 14, 2015 3:26 PM
To: edk2-devel@lists.01.org
Cc: Siyuan Fu <siyuan...@intel.com>; El-Haj-Mahmoud, Samer 
<samer.el-haj-mahm...@hpe.com>; El-Haj-Mahmoud, Samer 
<samer.el-haj-mahm...@hpe.com>
Subject: [PATCH] MdePkg: Http.h - Add HttpMethodMax to EFI_HTTP_METHOD

Add HttpMethodMax enum value to EFI_HTTP_METHOD to make it easier to iterate 
through the HTTP methods using a loop

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Samer El-Haj-Mahmoud <samer.el-haj-mahm...@hpe.com>
---
 MdePkg/Include/Protocol/Http.h | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/MdePkg/Include/Protocol/Http.h b/MdePkg/Include/Protocol/Http.h 
index b1f8c51..c7a89b4 100644
--- a/MdePkg/Include/Protocol/Http.h
+++ b/MdePkg/Include/Protocol/Http.h
@@ -5,6 +5,7 @@
   HTTP Protocol (HTTP)
 
   Copyright (c) 2015, Intel Corporation. All rights reserved.<BR>
+  (C) Copyright 2015 Hewlett Packard Enterprise Development LP<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 @@ -54,7 +55,8 @@ typedef enum {
   HttpMethodHead,
   HttpMethodPut,
   HttpMethodDelete,
-  HttpMethodTrace
+  HttpMethodTrace,
+  HttpMethodMax
 } EFI_HTTP_METHOD;
 
 ///
--
1.9.5.msysgit.1


_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel

Reply via email to