It was only failing on windows builds (it was inside a #ifdef Q_OS_WIN), so unless you built for windows, you wouldn’t have seen it.
-hadriel
> On Jul 8, 2015, at 2:12 AM, Joerg Mayer <jmayer@xxxxxxxxx> wrote:
>
> On Wed, Jul 08, 2015 at 05:18:06AM +0000, Wireshark code review wrote:
>> URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=d7f0118a74d540d3c8b07296c2c7b248ed13b464
>> Commits:
>> d7f0118 by Guy Harris (guy@xxxxxxxxxxxx):
>> Put back closing } accidentally removed in previous commit.
>
> Thanks Guy! The strange thing is, that I definitely compiled before committing
> (last time and this time). In this particular case, I even did a complete recompile
> (delete build directory, create build dir, run cmake etc) for some other reasons.
>
> Thanks
> Joerg
> --
> Joerg Mayer <jmayer@xxxxxxxxx>
> We are stuck with technology when what we really want is just stuff that
> works. Some say that should read Microsoft instead of technology.
> ___________________________________________________________________________
> Sent via: Wireshark-dev mailing list <wireshark-dev@xxxxxxxxxxxxx>
> Archives: https://www.wireshark.org/lists/wireshark-dev
> Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
> mailto:wireshark-dev-request@xxxxxxxxxxxxx?subject=unsubscribe