mbox series

[v4,0/3] Replace compilation time options with log level

Message ID 20190226230229.25450-1-dharmik.thakkar@arm.com (mailing list archive)
Headers
Series Replace compilation time options with log level |

Message

Dharmik Thakkar Feb. 26, 2019, 11:02 p.m. UTC 
  Currently, in test code, for printing extra information,
compilation time options are used. This approach does not compile the
code always. Macro needs to be set for the code to compile.

This patchset replaces compilation time option with log level based
approach, thus enabling compilation of code always. Log levels can be
set using eal parameters.

Dharmik Thakkar (3):
  test/hash: replace macro with log-level approach
  test/efd: enable unit test compilation always
  test/timer: enable unit test compilation always

 test/test/test_efd.c            | 24 +++++++++++-------------
 test/test/test_hash.c           | 24 ++++++++++++++----------
 test/test/test_timer_racecond.c | 24 +++++++++++++-----------
 3 files changed, 38 insertions(+), 34 deletions(-)

Comments

Thomas Monjalon March 28, 2019, 6:26 p.m. UTC | #1 
> Dharmik Thakkar (3):
>   test/hash: replace macro with log-level approach
>   test/efd: enable unit test compilation always
>   test/timer: enable unit test compilation always

Applied, thanks
Thomas Monjalon March 28, 2019, 6:35 p.m. UTC | #2 
28/03/2019 19:26, Thomas Monjalon:
> > Dharmik Thakkar (3):
> >   test/hash: replace macro with log-level approach
> >   test/efd: enable unit test compilation always
> >   test/timer: enable unit test compilation always
> 
> Applied, thanks

PS: for 32-bit build, I had to replace "%lu" by "%"PRIuPTR.
in third patch (app/test/test_timer_racecond.c).