Skip to content

Commit

Permalink
Remove commented-out code.
Browse files Browse the repository at this point in the history
  • Loading branch information
abellgithub committed Apr 19, 2019
1 parent 80ee1ac commit fce70c3
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 38 deletions.
9 changes: 0 additions & 9 deletions plugins/greyhound/io/GreyhoundCommon.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -64,15 +64,6 @@ static inline NL::json parse(const std::string& data)
return j;
}

//ABELL
/**
static inline std::string dense(const Json::Value& json)
{
Json::StreamWriterBuilder builder;
builder.settings_["indentation"] = "";
return Json::writeString(builder, json);
}
**/

static inline NL::json layoutToSchema(PointLayout& layout)
{
Expand Down
29 changes: 0 additions & 29 deletions test/unit/apps/pcpipelineTestJSON.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -71,10 +71,7 @@ void run_pipeline(std::string const& pipelineFile,
if (stat)
std::cerr << output << std::endl;
if (lookFor.size())
{
std::cerr << "Output = " << output << "!\n\n";
EXPECT_NE(output.find(lookFor), std::string::npos);
}
}

// most pipelines (those with a writer) will be invoked via `pdal pipeline`
Expand Down Expand Up @@ -315,32 +312,6 @@ TEST(json, issue_1417)
run_pipeline("pipeline/issue1417.json", options);
}

// Make sure we handle repeated options properly
/**
TEST(json, issue_1941)
{
PipelineManager manager;
std::string file;
file = Support::configuredpath("pipeline/range_multi_limits.json");
manager.readPipeline(file);
EXPECT_EQ(manager.execute(), (point_count_t)5);
const PointViewSet& s = manager.views();
EXPECT_EQ(s.size(), 1U);
PointViewPtr view = *s.begin();
EXPECT_EQ(view->getFieldAs<int>(Dimension::Id::X, 0), 3);
EXPECT_EQ(view->getFieldAs<int>(Dimension::Id::X, 1), 4);
EXPECT_EQ(view->getFieldAs<int>(Dimension::Id::X, 2), 5);
EXPECT_EQ(view->getFieldAs<int>(Dimension::Id::X, 3), 8);
EXPECT_EQ(view->getFieldAs<int>(Dimension::Id::X, 4), 9);
PipelineManager manager2;
file = Support::configuredpath("pipeline/range_bad_limits.json");
EXPECT_THROW(manager2.readPipeline(file), pdal_error);
}
**/

// Test that stage options passed via --stage.<tagname>.<option> work.
TEST(json, stagetags)
{
Expand Down

0 comments on commit fce70c3

Please sign in to comment.