[dpdk-dev] [PATCH] no need to test for NULL when freeing

[David Marchand]

On Thu, Jan 21, 2016 at 12:30 PM, Thomas Monjalon
<thomas.monjalon at 6wind.com> wrote:
> 2016-01-21 12:23, David Marchand:
>> free() already handles NULL pointer.
>>
>> Signed-off-by: David Marchand <david.marchand at 6wind.com>
>> ---
>>  app/test/test_devargs.c                            |  3 +-
>>  app/test/test_link_bonding.c                       |  6 ++--
>>  app/test/test_pci.c                                |  3 +-
>>  app/test/test_ring.c                               | 36 ++++++++--------------
>>  drivers/net/xenvirt/rte_eth_xenvirt.c              |  6 ++--
>>  drivers/net/xenvirt/rte_mempool_gntalloc.c         |  9 ++----
>>  examples/ip_pipeline/cpu_core_map.c                |  3 +-
>>  .../pipeline/pipeline_flow_classification_be.c     |  6 ++--
>>  examples/vhost_xen/vhost_monitor.c                 |  3 +-
>>  examples/vhost_xen/xenstore_parse.c                | 33 +++++++-------------
>>  lib/librte_eal/common/eal_common_devargs.c         |  3 +-
>>  lib/librte_eal/linuxapp/eal/eal_memory.c           |  3 +-
>>  lib/librte_ether/rte_ethdev.c                      |  6 ++--
>>  lib/librte_kvargs/rte_kvargs.c                     |  4 +--
>>  14 files changed, 41 insertions(+), 83 deletions(-)
>
> Have you used a coccinelle script?

Nop, some shell script of mine with manual checks.


-- 
David Marchand