Skip to content

Commit

Permalink
Removed unused variables. Misc cleanup.
Browse files Browse the repository at this point in the history
  • Loading branch information
nvdbaranec committed May 17, 2021
1 parent cc26de7 commit d58938d
Show file tree
Hide file tree
Showing 4 changed files with 7 additions and 9 deletions.
2 changes: 1 addition & 1 deletion cpp/src/column/column_factories.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -172,4 +172,4 @@ std::unique_ptr<column> make_dictionary_from_scalar(scalar const& s,
0);
}

} // namespace cudf
} // namespace cudf
2 changes: 1 addition & 1 deletion cpp/src/scalar/scalar.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
* limitations under the License.
*/

#include "structs/utilities.hpp"
#include <structs/utilities.hpp>

#include <cudf/column/column.hpp>
#include <cudf/detail/utilities/vector_factories.hpp>
Expand Down
2 changes: 1 addition & 1 deletion cpp/src/structs/structs_column_factories.cu
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
* limitations under the License.
*/

#include "structs/utilities.hpp"
#include <structs/utilities.hpp>

#include <cudf/column/column_factories.hpp>
#include <cudf/types.hpp>
Expand Down
10 changes: 4 additions & 6 deletions cpp/tests/scalar/factories_test.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -177,18 +177,16 @@ TEST_F(StructScalarFactory, Basic)

// table_view constructor
{
auto sc = cudf::make_struct_scalar(cudf::table_view{children});
auto s = static_cast<cudf::scalar_type_t<cudf::struct_view>*>(sc.get());
auto sview = s->view();
auto sc = cudf::make_struct_scalar(cudf::table_view{children});
auto s = static_cast<cudf::scalar_type_t<cudf::struct_view>*>(sc.get());
EXPECT_TRUE(s->is_valid());
CUDF_TEST_EXPECT_TABLES_EQUIVALENT(cudf::table_view{children}, s->view());
}

// host_span constructor
{
auto sc = cudf::make_struct_scalar(cudf::host_span<cudf::column_view const>{children});
auto s = static_cast<cudf::scalar_type_t<cudf::struct_view>*>(sc.get());
auto sview = s->view();
auto sc = cudf::make_struct_scalar(cudf::host_span<cudf::column_view const>{children});
auto s = static_cast<cudf::scalar_type_t<cudf::struct_view>*>(sc.get());
EXPECT_TRUE(s->is_valid());
CUDF_TEST_EXPECT_TABLES_EQUIVALENT(cudf::table_view{children}, s->view());
}
Expand Down

0 comments on commit d58938d

Please sign in to comment.