No subject


Wed Sep 29 11:15:30 PDT 2010


by tools like cppcheck was a goal.

Attached is a small patch that does nothing except eliminate one cppcheck
warning.  It is contributed under the LGPL3+.

As I said, it does nothing different and, depending on the compiler, it
could even be less efficient.  It does however make cppcheck happy.

Is this kind of thing useful?

--0022152d622da9d7e304916a7c92
Content-Type: text/html; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

<meta http-equiv=3D"content-type" content=3D"text/html; charset=3Dutf-8"><s=
pan class=3D"Apple-style-span" style=3D"font-family: arial, sans-serif; fon=
t-size: 13px; border-collapse: collapse; "><div><span class=3D"Apple-style-=
span" style=3D"font-family: arial, sans-serif; font-size: 13px; border-coll=
apse: collapse; ">[EDIT: Posting this again because the archive makes it lo=
ok like it got garbled.]</span></div>
<div><span class=3D"Apple-style-span" style=3D"font-family: arial, sans-ser=
if; font-size: 13px; border-collapse: collapse; "><br></span></div>From the=
 &#39;easy hacks&#39; page it seemed that getting rid of warnings and flags=
 by tools like cppcheck was a goal.<div>
<br></div><div>Attached is a small patch that does nothing except eliminate=
 one cppcheck warning. =C2=A0It is contributed under the LGPL3+.</div><div>=
<br></div><div>As I said, it does nothing different and, depending on the c=
ompiler, it could even be less efficient. =C2=A0It does however make cppche=
ck happy.</div>
<div><br></div><div>Is this kind of thing useful?</div></span>

--0022152d622da9d7e304916a7c92--
--0022152d622da9d7f404916a7c94
Content-Type: text/x-patch; charset=US-ASCII; name="javadep.diff"
Content-Disposition: attachment; filename="javadep.diff"
Content-Transfer-Encoding: base64
X-Attachment-Id: f_geok1pai0

ZGlmZiAtLWdpdCBhL3NvbHRvb2xzL2phdmFkZXAvamF2YWRlcC5jIGIvc29sdG9vbHMvamF2YWRl
cC9qYXZhZGVwLmMKaW5kZXggZDZkYzU4MC4uNjk1OGY4OCAxMDA2NDQKLS0tIGEvc29sdG9vbHMv
amF2YWRlcC9qYXZhZGVwLmMKKysrIGIvc29sdG9vbHMvamF2YWRlcC9qYXZhZGVwLmMKQEAgLTYw
Myw5ICs2MDMsMTEgQEAgeGNhbGxvYyhzaXplX3Qgbm1lbWIsIHNpemVfdCBzaXplKQogdm9pZCAq
CiB4cmVhbGxvYyh2b2lkICpwdHIsIHNpemVfdCBzaXplKQogewotICAgIHB0ciA9IHJlYWxsb2Mo
cHRyLCBzaXplKTsKKyAgICB2b2lkICpuZXdwdHIgPSByZWFsbG9jKHB0ciwgc2l6ZSk7CiAgICAg
Ci0gICAgaWYgKCAhcHRyICkKKyAgICBpZiAoIW5ld3B0cikKKwlwdHIgPSBuZXdwdHI7CisgICAg
ZWxzZQogICAgICAgICBlcnJfcXVpdCgib3V0IG9mIG1lbW9yeSIpOwogCiAgICAgcmV0dXJuIHB0
cjsK
--0022152d622da9d7f404916a7c94--


More information about the LibreOffice mailing list