----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120047/#review65838 -----------------------------------------------------------
protocols/wlm/wlmchatsession.cpp <https://git.reviewboard.kde.org/r/120047/#comment45994> Shouldn't you check if `GIFLIB_MINOR` is defined too? protocols/wlm/wlmchatsession.cpp <https://git.reviewboard.kde.org/r/120047/#comment45993> There shouldn't be a space before ");" - Raphael Kubo da Costa On Sept. 5, 2014, 4:52 p.m., Raymond Wooninck wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/120047/ > ----------------------------------------------------------- > > (Updated Sept. 5, 2014, 4:52 p.m.) > > > Review request for Kopete and Pali Rohár. > > > Bugs: 335519 > http://bugs.kde.org/show_bug.cgi?id=335519 > > > Repository: kopete > > > Description > ------- > > Giflib 5.1.0 introduced some API changes with regards to EGifCloseFile and > takes now an extra argument. > > > Diffs > ----- > > protocols/wlm/wlmchatsession.cpp 4843791 > > Diff: https://git.reviewboard.kde.org/r/120047/diff/ > > > Testing > ------- > > Builds fine with both giflib < 5.1.0 and with giflib 5.1.0 > > > Thanks, > > Raymond Wooninck > >
_______________________________________________ kopete-devel mailing list kopete-devel@kde.org https://mail.kde.org/mailman/listinfo/kopete-devel
