File tree Expand file tree Collapse file tree 2 files changed +14
-13
lines changed
Expand file tree Collapse file tree 2 files changed +14
-13
lines changed Original file line number Diff line number Diff line change @@ -55,9 +55,9 @@ class FExpr_FillNA : public FExpr_Func {
5555 out +=" , value=" ;
5656 out += value_->repr ();
5757 } else {
58- out += " , reverse=" ;
59- out += reverse_? " True" : " False" ;
60- }
58+ out += " , reverse=" ;
59+ out += reverse_? " True" : " False" ;
60+ }
6161 out += ' )' ;
6262 return out;
6363 }
@@ -83,12 +83,12 @@ class FExpr_FillNA : public FExpr_Func {
8383 indices[i] = static_cast <int32_t >(fill_id);
8484 }
8585 } else {
86- for (size_t i = i1; i < i2; ++i) {
87- size_t is_valid = col.get_element_isvalid (i);
88- fill_id = is_valid? i : fill_id;
89- indices[i] = static_cast <int32_t >(fill_id);
86+ for (size_t i = i1; i < i2; ++i) {
87+ size_t is_valid = col.get_element_isvalid (i);
88+ fill_id = is_valid? i : fill_id;
89+ indices[i] = static_cast <int32_t >(fill_id);
90+ }
9091 }
91- }
9292
9393 }
9494 );
@@ -142,8 +142,8 @@ class FExpr_FillNA : public FExpr_Func {
142142 if (!gby) {
143143 gby = Groupby::single_group (wf.nrows ());
144144 } else {
145- wf.increase_grouping_mode (Grouping::GtoALL);
146- }
145+ wf.increase_grouping_mode (Grouping::GtoALL);
146+ }
147147
148148 for (size_t i = 0 ; i < wf.ncols (); ++i) {
149149 bool is_grouped = ctx.has_group_column (
Original file line number Diff line number Diff line change @@ -58,9 +58,10 @@ Column FExpr_RowCount::apply_function(colvec&& columns,
5858 }
5959 for (size_t i = 0 ; i < columns.size (); ++i) {
6060 xassert (columns[i].nrows () == nrows);
61- bool is_void_column = columns[i].stype () == SType::VOID;
62- columns[i] = is_void_column? Const_ColumnImpl::make_bool_column (columns[i].nrows (), true )
63- : Column (new Isna_ColumnImpl (std::move (columns[i])));
61+ Column coli = columns[i];
62+ bool is_void_column = coli.stype () == SType::VOID;
63+ columns[i] = is_void_column? Const_ColumnImpl::make_bool_column (coli.nrows (), true )
64+ : Column (new Isna_ColumnImpl (std::move (coli)));
6465 }
6566 return Column (new FuncNary_ColumnImpl<int32_t >(
6667 std::move (columns), op_rowcount, nrows, SType::INT32));
You can’t perform that action at this time.
0 commit comments