Auto merge of #440 - servo:macfix, r=Manishearth

Work around inconsistent padding from different LLVM versions.

This allows building with LLVM 4 or 6 and unbreaks my mac cross-compilation.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/rust-mozjs/440)
<!-- Reviewable:end -->

Author: bors-servo

Cargo.toml

@@ -2,7 +2,7 @@
 name = "mozjs"
 description = "Rust bindings to the Mozilla SpiderMonkey JavaScript engine."
 repository = "https://github.com/servo/rust-mozjs"
-version = "0.9.1"
+version = "0.9.2"
 authors = ["The Servo Project Developers"]
 build = "build.rs"
 license = "MPL-2.0"

src/conversions.rs

@@ -44,6 +44,7 @@ use rust::maybe_wrap_value;
 use libc;
 use num_traits::{Bounded, Zero};
 use std::borrow::Cow;
+use std::mem;
 use std::rc::Rc;
 use std::{ptr, slice};
 
@@ -601,11 +602,18 @@ impl<C: Clone, T: FromJSValConvertible<Config=C>> FromJSValConvertible for Vec<T
             return Ok(ConversionResult::Failure("Value is not an object".into()));
         }
 
+        // Depending on the version of LLVM in use, bindgen can end up including
+        // a padding field in the ForOfIterator. To support multiple versions of
+        // LLVM that may not have the same fields as a result, we create an empty
+        // iterator instance and initialize a non-empty instance using the empty
+        // instance as a base value.
+        let zero = mem::zeroed();
         let mut iterator = ForOfIterator {
             cx_: cx,
             iterator: RootedObject::new_unrooted(),
             nextMethod: RootedValue::new_unrooted(),
             index: ::std::u32::MAX, // NOT_ARRAY
+            ..zero
         };
         let iterator = ForOfIteratorGuard::new(cx, &mut iterator);
         let iterator = &mut *iterator.root;