From 1c3af740109bb3e626448d6900bc4f7b9a4784c8 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Kirill=20M=C3=BCller?= <krlmlr+r@mailbox.org>
Date: Sat, 18 Aug 2018 09:38:25 +0200
Subject: [PATCH] Revert "Call constructor instead of assignment"

This reverts commit 6e77a951e0d5b65e272f9007ef1fbc7a7e4f4970.

rchk results with https://github.com/krlmlr/Rcpp/commit/5dbfca11da325a8e72a49b90ccdd1e8714f4089c.
---
 RcppNoInitTest.so.bcheck | 2 +-
 src/test.cpp             | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/RcppNoInitTest.so.bcheck b/RcppNoInitTest.so.bcheck
index ec2a706..b14e1e7 100644
--- a/RcppNoInitTest.so.bcheck
+++ b/RcppNoInitTest.so.bcheck
@@ -26,4 +26,4 @@ Function Rcpp::internal::is_Rcpp_eval_call(SEXPREC*)
 Function _RcppNoInitTest_test
   [UP] ignoring variable <unnamed var:   %14 = alloca %struct.SEXPREC*, align 8> as it has address taken, results will be incomplete 
   [PB] has possible protection stack imbalance /home/kirill/git/R/RcppNoInitTest/src/RcppExports.cpp:17
-Analyzed 152 functions, traversed 956 states.
+Analyzed 171 functions, traversed 1017 states.
diff --git a/src/test.cpp b/src/test.cpp
index 38bc8f7..cc9f499 100644
--- a/src/test.cpp
+++ b/src/test.cpp
@@ -6,6 +6,6 @@ using namespace Rcpp;
 //' @useDynLib RcppNoInitTest, .registration = TRUE
 // [[Rcpp::export]]
 IntegerVector test() {
-  IntegerVector ret(no_init(1));
+  IntegerVector ret = no_init(1);
   return ret;
 }