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 <jma...@loplof.de> 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 (g...@alum.mit.edu): >> 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 <jma...@loplof.de> > 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@wireshark.org> > Archives: https://www.wireshark.org/lists/wireshark-dev > Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev > mailto:wireshark-dev-requ...@wireshark.org?subject=unsubscribe ___________________________________________________________________________ Sent via: Wireshark-dev mailing list <wireshark-dev@wireshark.org> Archives: https://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev mailto:wireshark-dev-requ...@wireshark.org?subject=unsubscribe
