On 08/19/15 19:20, Mike Holmes wrote:
The odp_format tool is not infallible, in this case it is better to omit
the macro from formatting

Signed-off-by: Mike Holmes <mike.hol...@linaro.org>
---
  platform/linux-generic/include/odp_buffer_internal.h | 3 ++-
  1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/platform/linux-generic/include/odp_buffer_internal.h 
b/platform/linux-generic/include/odp_buffer_internal.h
index a7638da..db9defa 100644
--- a/platform/linux-generic/include/odp_buffer_internal.h
+++ b/platform/linux-generic/include/odp_buffer_internal.h
@@ -30,7 +30,7 @@ extern "C" {
  #include <odp/thread.h>
  #include <odp/event.h>
-
+/* clang-format off */
  #define ODP_BITSIZE(x) \
        ((x) <=     2 ?  1 : \
        ((x) <=     4 ?  2 : \
@@ -49,6 +49,7 @@ extern "C" {
        ((x) <= 32768 ? 15 : \
        ((x) <= 65536 ? 16 : \
         (0/0)))))))))))))))))
+/* clang-format on */
that is ugly solution. We should not add to code comments specific to different tools. Can that be in some exception file?

Maxim.

  _ODP_STATIC_ASSERT(ODP_CONFIG_PACKET_SEG_LEN_MIN >= 256,
                   "ODP Segment size must be a minimum of 256 bytes");

_______________________________________________
lng-odp mailing list
lng-odp@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/lng-odp

Reply via email to