Subject : Help regarding Bug 90341 - Clean up excessive const_cast'ing

Ilmari Lauhakangas ilmari.lauhakangas at libreoffice.org
Sat Dec 24 14:45:49 UTC 2022


On 24.12.2022 13.42, Siddharth Khattar wrote:
> Hello,
> 
> So I did all the recommended changes that were commented in the code
> in the file "impastpl.cxx" but Jenkins is still giving me a build
> error. Can someone please help me with it, the link to it on Gerrit
> is: https://gerrit.libreoffice.org/c/core/+/144537
> I tried a further commit and investigated a few things by changing
> some static_cast back to const_cast but couldn't figure it out,
> unfortunately. For reference, The easy hack I'm referring to is:
> https://bugs.documentfoundation.org/show_bug.cgi?id=90341

Without commenting on the content of the changes, your problem is that 
you have created a series of patches. The expectation is that for a task 
like this, you keep updating the single patch. In your latest patch I 
also commented about this: https://gerrit.libreoffice.org/c/core/+/144790

You should look at 'git log', see how many of your own commits you have 
and then say

git reset --soft HEAD~x

where x is the number of your own commits minus one. Now you have all 
your modifications ready to commit with 'git commit --amend' and they 
will go toward your first patch which should be this: 
https://gerrit.libreoffice.org/c/core/+/144374

When editing the commit message, double-check that the Change-Id is

Change-Id: I8a3d7c6695dea16536f1a0ebbbbd2c27fbedeb8d

Finally './logerrit submit master' to submit the new patchset.

Ilmari


More information about the LibreOffice mailing list