[fpc-devel] Attn Florian, commit 1b698210 10/10/2022 22:45:31 breaks DWARF info
lazarus at mfriebe.de
Wed Oct 19 23:42:48 CEST 2022
On 19/10/2022 22:56, Martin Frb via fpc-devel wrote:
> On 19/10/2022 21:12, Pierre Muller via fpc-devel wrote:
>> Hi Martin,
>> could you tell me if
>> fixes your troubles?
>> Thanks in advance,
> It looks like the right direction. But see my comment on the issue.
> It is correct as in that those are the 2 values that need to be
> adjusted (According to code review). Just the 2nd case should be
> adjusted by assigning the correct values to the variable "uppercodes"
> I will have to setup a quick test.
> I had my findings by code-review only.
Ok, I made a merge request (slightly diff from the patch, but to the
I tested this, and it produces valid dwarf again.
There is the question as to introducing a constant, instead of changing
several copies of the same number.
On the other hand it happens once every x years.
And I don't know what the preferred style is.
Could also be
const EntryLen = array[Boolean] of integer = (14, 17);
and then use a boolean variable instead of "uppercodes".
But that is not part of the issue itself.
More information about the fpc-devel