On 11/07/2011 10:42 PM, Torvald Riegel wrote:
+               noex = tsubst_copy_and_build (noex, args, complain, in_decl,
+                                             /*function_p=*/false,
+                                             /*integral_const_expr_p=*/true);
+               noex = build_noexcept_spec (TREE_PURPOSE (noex),
+                                           tf_warning_or_error);

If you're going to pull the TREE_PURPOSE out, you might as well do that before tsubsting rather than after.

+      SET_EXPR_LOCATION(body, EXPR_LOCATION (TRANSACTION_EXPR_BODY (stmt)));

Space before (.

+      SET_EXPR_LOCATION(expr, loc);

Here too.

Otherwise, looks OK.

Jason

Reply via email to