Ah, good catch. Thanks<br><br><div class="gmail_quote">On Mon, Feb 28, 2011 at 11:19 PM, Frits van Bommel <span dir="ltr"><<a href="mailto:fvbommel@gmail.com">fvbommel@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

On Tue, Mar 1, 2011 at 12:53 AM, Talin <<a href="mailto:viridia@gmail.com">viridia@gmail.com</a>> wrote:<br>
> +Constant* ConstantStruct::get(LLVMContext &Context, bool Packed,<br>
> +                              Constant * Val, ...) {<br>
> +  va_list ap;<br>
> +  std::vector<Constant*> Values;<br>
> +  va_start(ap, Val);<br>
> +  while (Val) {<br>
> +    Values.push_back(Val);<br>
> +    Val = va_arg(ap, llvm::Constant*);<br>
> +  }<br>
<br>
Shouldn't there technically be a va_end right about here?<br>
I'm not aware of any platform where that actually does anything, but<br>
it can't hurt either...<br>
<br>
> +  return get(Context, Values, Packed);<br>
> +}<br>
</blockquote></div><br><br clear="all"><br>-- <br>-- Talin<br>