[prev in list] [next in list] [prev in thread] [next in thread] 

List:       cfe-commits
Subject:    Re: [cfe-commits] r72673 - in /cfe/trunk: lib/Sema/SemaExprCXX.cpp
From:       Sebastian Redl <sebastian.redl () getdesigned ! at>
Date:       2009-05-31 21:52:21
Message-ID: 4A22FC15.6080901 () getdesigned ! at
[Download RAW message or body]

Anders Carlsson wrote:
> ==============================================================================
> --- cfe/trunk/lib/Sema/SemaExprCXX.cpp (original)
> +++ cfe/trunk/lib/Sema/SemaExprCXX.cpp Sun May 31 15:26:12 2009
> @@ -508,6 +508,11 @@
>        return true;
>    }
>  
> +  // FindAllocationOverload can change the passed in arguments, so we need to
> +  // copy them back.
> +  if (NumPlaceArgs > 0)
> +    std::copy(&AllocArgs[1], AllocArgs.end(), PlaceArgs);
> +  
>   
Why, so it can! I never thought about this.
Hmm, now I wonder how many times I've made this mistake ...

Sebastian
_______________________________________________
cfe-commits mailing list
cfe-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

[prev in list] [next in list] [prev in thread] [next in thread]