Commit d638e304f13a introduced a bullet list, but parse-support-md
choke on it as it doesn't know what to do about it.

Introduce ri_BulletList() so that r_content() will find this new
function and call it instead of calling process_unknown().

Reported-by: Julien Grall <julien@xen.org>
Fixes: d638e304f13a ("SUPPORT.MD: Fix matrix generation after 43c416d0d819 and 77c39a53cf5b")
Signed-off-by: Anthony PERARD <anthony.perard@citrix.com>
---
 docs/parse-support-md | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/docs/parse-support-md b/docs/parse-support-md
index a397637639..b04f62da37 100755
--- a/docs/parse-support-md
+++ b/docs/parse-support-md
@@ -218,6 +218,13 @@ sub ri_DefinitionList {
     }
 }
 
+sub ri_BulletList {
+    # Assume a paragraph introduce this bullet list, which would mean that
+    # ri_Para() has already been called, and there's nothing else to do about
+    # the caveat.
+    return;
+}
+
 sub process_unknown {
     my ($c, $e) = @_;
     $had_unknown = Dumper($e);
-- 
Anthony PERARD

Hi Anthony,

On 19/03/2024 15:15, Anthony PERARD wrote:
> Commit d638e304f13a introduced a bullet list, but parse-support-md
> choke on it as it doesn't know what to do about it.
> 
> Introduce ri_BulletList() so that r_content() will find this new
> function and call it instead of calling process_unknown().
> 
> Reported-by: Julien Grall <julien@xen.org>
> Fixes: d638e304f13a ("SUPPORT.MD: Fix matrix generation after 43c416d0d819 and 77c39a53cf5b")
> Signed-off-by: Anthony PERARD <anthony.perard@citrix.com>

My Perl knowledge is rubbish. But I tested the fix. So:

Tested-by: Julien Grall <jgrall@amazon.com>
Acked-by: Julien Grall <jgrall@amazon.com>

Cheers,

> ---
>   docs/parse-support-md | 7 +++++++
>   1 file changed, 7 insertions(+)
> 
> diff --git a/docs/parse-support-md b/docs/parse-support-md
> index a397637639..b04f62da37 100755
> --- a/docs/parse-support-md
> +++ b/docs/parse-support-md
> @@ -218,6 +218,13 @@ sub ri_DefinitionList {
>       }
>   }
>   
> +sub ri_BulletList {
> +    # Assume a paragraph introduce this bullet list, which would mean that
> +    # ri_Para() has already been called, and there's nothing else to do about
> +    # the caveat.
> +    return;
> +}
> +
>   sub process_unknown {
>       my ($c, $e) = @_;
>       $had_unknown = Dumper($e);

-- 
Julien Grall

On 19/03/2024 17:45, Julien Grall wrote:
> Hi Anthony,
> 
> On 19/03/2024 15:15, Anthony PERARD wrote:
>> Commit d638e304f13a introduced a bullet list, but parse-support-md
>> choke on it as it doesn't know what to do about it.
>>
>> Introduce ri_BulletList() so that r_content() will find this new
>> function and call it instead of calling process_unknown().
>>
>> Reported-by: Julien Grall <julien@xen.org>
>> Fixes: d638e304f13a ("SUPPORT.MD: Fix matrix generation after 
>> 43c416d0d819 and 77c39a53cf5b")
>> Signed-off-by: Anthony PERARD <anthony.perard@citrix.com>
> 
> My Perl knowledge is rubbish. But I tested the fix. So:
> 
> Tested-by: Julien Grall <jgrall@amazon.com>
> Acked-by: Julien Grall <jgrall@amazon.com>

And committed.

Cheers,

> 
> Cheers,
> 
>> ---
>>   docs/parse-support-md | 7 +++++++
>>   1 file changed, 7 insertions(+)
>>
>> diff --git a/docs/parse-support-md b/docs/parse-support-md
>> index a397637639..b04f62da37 100755
>> --- a/docs/parse-support-md
>> +++ b/docs/parse-support-md
>> @@ -218,6 +218,13 @@ sub ri_DefinitionList {
>>       }
>>   }
>> +sub ri_BulletList {
>> +    # Assume a paragraph introduce this bullet list, which would mean 
>> that
>> +    # ri_Para() has already been called, and there's nothing else to 
>> do about
>> +    # the caveat.
>> +    return;
>> +}
>> +
>>   sub process_unknown {
>>       my ($c, $e) = @_;
>>       $had_unknown = Dumper($e);
> 

-- 
Julien Grall