Patch from Julian Foad:

Initialise a "possibly uninitialised" variable (which was already
functionally correct).

[Note from DPM: this patch will be blown away if we ever update to a
newer version of Expat.]

diff --git a/simgear/xml/xmltok_impl.c b/simgear/xml/xmltok_impl.c
index f343b3fa0f769d2c95a9170e6f2cabeb0e4ece77..c52539be8afde81e7a6051af724408f47396960a 100644 (file)
--- a/simgear/xml/xmltok_impl.c
+++ b/simgear/xml/xmltok_impl.c
@@ -1391,7 +1391,7 @@ int PREFIX(getAtts)(const ENCODING *enc, const char *ptr,
 {
   enum { other, inName, inValue } state = inName;
   int nAtts = 0;
-  int open;
+  int open = 0;
 
   for (ptr += MINBPC(enc);; ptr += MINBPC(enc)) {
     switch (BYTE_TYPE(enc, ptr)) {