<div dir="ltr"><div dir="ltr">Yes, let's do that.<br><br><br><br></div><br><img width="0" height="0" class="mailtrack-img" alt="" style="display:flex" src="https://mailtrack.io/trace/mail/cf6be7eb05d862dfabd57274286de2fdc94bb37d.png?u=5575466"><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Mar 3, 2022 at 11:42 PM Albert Astals Cid <<a href="mailto:aacid@kde.org">aacid@kde.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">El divendres, 6 de novembre de 2020, a les 0:27:05 (CET), Albert Astals Cid va escriure:<br>
> El divendres, 11 de setembre de 2020, a les 22:00:30 CET, Albert Astals Cid va escriure:<br>
> > El divendres, 21 d’agost de 2020, a les 1:19:19 CEST, Albert Astals Cid va escriure:<br>
> > > Most of the guidelines suggest it so that you don't forget to add them when adding a new line in the "block".<br>
> > > <br>
> > > What do you think?<br>
> > > <br>
> > > <a href="https://invent.kde.org/graphics/okular/-/merge_requests/248" rel="noreferrer" target="_blank">https://invent.kde.org/graphics/okular/-/merge_requests/248</a><br>
> > > <br>
> > > We're going to need quite some changes to make it pass, so asking before starting to do the work :D<br>
> > <br>
> > We agreed on the Akademy Okular meeting that we will do this, *but* I'm going to postpone doing it after the 20.08.3 release.<br>
> > <br>
> > Rationale:<br>
> >  * Making this change even if mechanical (clang-tidy does it) can cause potential regressions if something goes wrong, hence is something that needs to be applied in master only<br>
> >  * If we apply it to master now, merging up from release/20.08 to master can cause master CI to stop compiling since it'd be requiring more things than release/20.08 CI<br>
> <br>
> Going to postpone this because it seems not clang-tidy 10 nor 11 nor 12 are able to format Okular codebase correctly with readability-braces-around-statements<br>
<br>
clang-tidy-13 seems to have succeeded.<br>
<br>
What do you say, should we give it a go?<br>
<br>
<a href="https://invent.kde.org/graphics/okular/-/merge_requests/578" rel="noreferrer" target="_blank">https://invent.kde.org/graphics/okular/-/merge_requests/578</a><br>
<br>
Cheers,<br>
  Albert<br>
<br>
> <br>
> Cheers,<br>
>   Albert<br>
> <br>
> > <br>
> > Cheers,<br>
> >   Albert<br>
> > <br>
> > > <br>
> > > Cheers,<br>
> > >   Albert<br>
> > > <br>
> > > <br>
> > > <br>
> > <br>
> > <br>
> > <br>
> > <br>
> > <br>
> <br>
> <br>
> <br>
> <br>
> <br>
<br>
<br>
<br>
<br>
</blockquote></div></div>