Re: [PATCH] Fix file find utils and add unit tests (PR driver/81829).

2017-08-30 Thread Martin Liška
On 08/25/2017 06:41 PM, Martin Sebor wrote: > On 08/18/2017 04:17 AM, Martin Liška wrote: >> On 08/15/2017 02:45 PM, Martin Liška wrote: >>> Hi. >>> >>> As shown in the PR, remove_prefix function is written wrongly. It does not >>> distinguish >>> in between a node in linked list and pprefix->plis

Re: [PATCH] Fix file find utils and add unit tests (PR driver/81829).

2017-08-25 Thread Martin Sebor
On 08/18/2017 04:17 AM, Martin Liška wrote: On 08/15/2017 02:45 PM, Martin Liška wrote: Hi. As shown in the PR, remove_prefix function is written wrongly. It does not distinguish in between a node in linked list and pprefix->plist. So I decide to rewrite it. Apart from that, I identified discr

Re: [PATCH] Fix file find utils and add unit tests (PR driver/81829).

2017-08-18 Thread Martin Liška
ry should be ignored from prefix. Can you please test attached patch in your scenario? Thanks, Martin >From f8029ed6d3dd444ee2608146118f2189cf9ef0d8 Mon Sep 17 00:00:00 2001 From: marxin Date: Mon, 14 Aug 2017 13:56:32 +0200 Subject: [PATCH] Fix file find utils and add unit tests (PR driv

[PATCH] Fix file find utils and add unit tests (PR driver/81829).

2017-08-15 Thread Martin Liška
Hi. As shown in the PR, remove_prefix function is written wrongly. It does not distinguish in between a node in linked list and pprefix->plist. So I decide to rewrite it. Apart from that, I identified discrepancy in between do_add_prefix and prefix_from_string where the later one appends always