[SOLVED] Re: Cppcheck : Variable 'nMode' is reassigned in fstat.cxx

2013-01-21 Thread julien2412
Thank you Stephan, I pushed the fix on master (see http://cgit.freedesktop.org/libreoffice/core/commit/?id=87f9d7da00857c649784a7d9eca046bf6e71ae3c) Julien -- View this message in context: http://nabble.documentfoundation.org/Cppcheck-Variable-nMode-is-reassigned-in-fstat-cxx-tp4030405p4030913

Re: Cppcheck : Variable 'nMode' is reassigned in fstat.cxx

2013-01-21 Thread Stephan Bergmann
On 01/19/2013 05:35 PM, julien2412 wrote: Cppcheck reported this: Indeed, we can see this: 109 if (bRO) 110 { 111 nMode = aBuf.st_mode & ~S_IWUSR; 112 nMode = aBuf.st_mode & ~S_IWGRP; 113 nMode = aBuf.st_mode & ~S_IWOTH; 114 }

Cppcheck : Variable 'nMode' is reassigned in fstat.cxx

2013-01-19 Thread julien2412
Hello, Cppcheck reported this: Indeed, we can see this: 109 if (bRO) 110 { 111 nMode = aBuf.st_mode & ~S_IWUSR; 112 nMode = aBuf.st_mode & ~S_IWGRP; 113 nMode = aBuf.st_mode & ~S_IWOTH; 114 } see http://opengrok.libreoffice.org/xref/