Inconsistent declarations

[pjljvandelaar]

Hi Gnatcoll developers,

As you can see e.g. in

gnatcoll-core/src/gnatcoll-boyer_moore.adb

Lines 136 to 138 in 12821a4

	Prefix : Offset_Array (1 .. From_String'Length);
	Reverse_Prefix : Offset_Array (1 .. From_String'Length);
	K, K2 : Natural := 0;

some declarations are combined,
while others are kept apart.

Do you want to be consistent in this aspect?
So either

 Prefix          : Offset_Array (1 .. From_String'Length); 
 Reverse_Prefix  : Offset_Array (1 .. From_String'Length); 
 K               : Natural := 0; 
 K2              : Natural := 0; 

or

 Prefix, Reverse_Prefix  : Offset_Array (1 .. From_String'Length); 
 K, K2                   : Natural := 0; 

If so, using the rejuvenation library we can rewrite the code to the desired format.

Greetings,
Pierre

Problem detected and solvable using Rejuvenation-Ada crate

[Jicquel]

Hello pjljvandelaar, thank you for opening this issue. You can open a PR with the modification you suggested, and I will take care of it. I would prefer for the more compact version, so the second one in your example. Please be aware that we would like to keep our current coding style, as already discussed in the other PR you opened.