fix fedora-clang compilation issue

bnosac · Oct 11, 2020 · d527659 · d527659
1 parent 51f178f
commit d527659
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 2 deletions.
diff --git a/DESCRIPTION b/DESCRIPTION
@@ -1,7 +1,7 @@
 Package: BTM
 Type: Package
 Title: Biterm Topic Models for Short Text
-Version: 0.3.3
+Version: 0.3.4
 Maintainer: Jan Wijffels <[email protected]>
 Authors@R: c(
  person('Jan', 'Wijffels', role = c('aut', 'cre', 'cph'), email = '[email protected]', comment = "R wrapper"), 

diff --git a/NEWS.md b/NEWS.md
@@ -1,3 +1,7 @@
+### CHANGES IN BTM VERSION 0.3.4
+
+- Fix -Wself-assign issue on fedora-clang
+
 ### CHANGES IN BTM VERSION 0.3.3
 
 - Make example conditionally on availability of udpipe

diff --git a/src/rcpp_BTM.cpp b/src/rcpp_BTM.cpp
@@ -172,7 +172,7 @@ Rcpp::List btm_biterms_text(Rcpp::CharacterVector x, int W, int win = 15) {
  Rcpp::CharacterVector doc_ids = x.attr("names");
  std::string context_id;
 
- Rcpp::XPtr<Model> model(new Model(K = K, W = W, alpha = alpha, beta = beta, iter = iter, save_step = save_step, has_background = has_background), true);
+ Rcpp::XPtr<Model> model(new Model(K, W, alpha, beta, iter, save_step, has_background), true);
  std::string line;
 
  for (int idx = 0; idx < x.size(); idx++){