Re: [lng-odp] [PATCH 1/6] linux-generic: imp.c: remove header defines from c file

2015-02-16 Thread Taras Kondratiuk
On 02/11/2015 09:10 AM, Maxim Uvarov wrote:
 Signed-off-by: Maxim Uvarov maxim.uva...@linaro.org
 ---
  platform/linux-generic/odp_impl.c | 5 -
  1 file changed, 5 deletions(-)
 
 diff --git a/platform/linux-generic/odp_impl.c 
 b/platform/linux-generic/odp_impl.c
 index ca3224d..f0a582e 100644
 --- a/platform/linux-generic/odp_impl.c
 +++ b/platform/linux-generic/odp_impl.c
 @@ -11,9 +11,6 @@
   * ODP Implementation information
   */
  
 -#ifndef ODP_IMPL_H_
 -#define ODP_IMPL_H_
 -
  #ifdef __cplusplus
  extern C {
  #endif

extern C should be also removed.

 @@ -31,5 +28,3 @@ const char *odp_version_impl_str(void)
  #ifdef __cplusplus
  }
  #endif
 -
 -#endif
 


-- 
Taras Kondratiuk

___
lng-odp mailing list
lng-odp@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/lng-odp


Re: [lng-odp] [PATCH 1/6] linux-generic: imp.c: remove header defines from c file

2015-02-16 Thread Maxim Uvarov

On 02/16/2015 01:59 PM, Taras Kondratiuk wrote:

On 02/11/2015 09:10 AM, Maxim Uvarov wrote:

Signed-off-by: Maxim Uvarov maxim.uva...@linaro.org
---
  platform/linux-generic/odp_impl.c | 5 -
  1 file changed, 5 deletions(-)

diff --git a/platform/linux-generic/odp_impl.c 
b/platform/linux-generic/odp_impl.c
index ca3224d..f0a582e 100644
--- a/platform/linux-generic/odp_impl.c
+++ b/platform/linux-generic/odp_impl.c
@@ -11,9 +11,6 @@
   * ODP Implementation information
   */
  
-#ifndef ODP_IMPL_H_

-#define ODP_IMPL_H_
-
  #ifdef __cplusplus
  extern C {
  #endif

extern C should be also removed.


obviously, yes.  It was removed in fist patch which includes everything. 
Will do it here. Thanks.


Maxim.




@@ -31,5 +28,3 @@ const char *odp_version_impl_str(void)
  #ifdef __cplusplus
  }
  #endif
-
-#endif






___
lng-odp mailing list
lng-odp@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/lng-odp