Hello, About commit 30e72a9a36750b7326803fa8fa1b8d40648e7e89 and, in particular, to:
- sal_Int32 nWindowWidth = ::std::max( nMinWidth, nBrowserWidth ); - nWindowWidth = ::std::min( nMaxWidth, nBrowserWidth ); + sal_Int32 nWindowWidth = ::std::min( nMaxWidth, nBrowserWidth ); Matteo Casalin thought (and I agree) that if my changes surely keep the previous behaviour, the original code might be wrong. Indeed, it seemed that its aim was to bound nWindowWidth between the maximum and minimum possible values, but it didn't. So perhaps it should be: sal_Int32 nWindowWidth = ::std::max( nMinWidth, nBrowserWidth ); nWindowWidth = ::std::min( nMaxWidth, nWindowWidth ); Any idea? Julien PS: Thank you again Matteo for your suggestion! -- View this message in context: http://nabble.documentfoundation.org/About-commit-cppcheck-some-cleanings-tp4075677.html Sent from the Dev mailing list archive at Nabble.com. _______________________________________________ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
