[Development] Qt 5.4.0 header diff

[Jason McDonald]

On Wed, Nov 19, 2014 at 10:25 AM, Thiago Macieira <thiago.macieira at intel.com
> wrote:

> On Tuesday 18 November 2014 19:24:13 Giuseppe D'Angelo wrote:
> > Erk... would it be possible to resend the diff EXCLUDING the license
> > header changes? The Gui/Widgets header diffs are huge...
>
> Done.
>
> The old header was actually 41 lines long. I had the threshold for "this is
> the header" at 40 lines.
>
> >From now on, it's 37 lines long.
>

When I was looking after license headers, I kept the number of lines fixed,
which often resulted in there being blank lines at the bottom of the
header.

One of the problems I found if the number of lines changed was that a bunch
of translation autotests broke, as the line numbers were important in those
tests.  It was a lot less work to keep the length of the headers constant.

That was in the Qt 4.x days though, where breaking those tests would have
caused an immediate CI failure in the monolithic repo.

Did someone fix those tests when license header length changed, or are
those tests more clever about line numbers now?

--
Jason
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.qt-project.org/pipermail/development/attachments/20141119/38d7fae1/attachment.html>