Skip to content

Commit

Permalink
Rename function.
Browse files Browse the repository at this point in the history
  • Loading branch information
abellgithub committed Sep 28, 2018
1 parent cfd12f7 commit f8b006b
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 10 deletions.
7 changes: 4 additions & 3 deletions pdal/Stage.hpp
Expand Up @@ -138,12 +138,13 @@ class PDAL_DLL Stage
{ return false; }

/**
Determine if a pipeline with this stage as a sink is streamable.
Return a pointer to a pipeline's first non-streamable stage,
if one exists.
\return Nullptr if the stage is streamable, a pointer to this stage
\return nullptr if the stage is streamable, a pointer to this stage
otherwise.
*/
virtual const Stage *checkStreamable() const
virtual const Stage *findNonstreamable() const
{ return this; }

/**
Expand Down
6 changes: 3 additions & 3 deletions pdal/Streamable.cpp
Expand Up @@ -53,13 +53,13 @@ bool Streamable::pipelineStreamable() const
}


const Stage *Streamable::checkStreamable() const
const Stage *Streamable::findNonstreamable() const
{
const Stage *nonstreamable;

for (const Stage *s : m_inputs)
{
nonstreamable = s->checkStreamable();
nonstreamable = s->findNonstreamable();
if (nonstreamable)
return nonstreamable;
}
Expand Down Expand Up @@ -114,7 +114,7 @@ void Streamable::execute(StreamPointTable& table)
}
};

const Stage *nonstreaming = checkStreamable();
const Stage *nonstreaming = findNonstreamable();
if (nonstreaming)
nonstreaming->throwError("Attempting to use stream mode with a "
"stage that doesn't support streaming.");
Expand Down
8 changes: 4 additions & 4 deletions pdal/Streamable.hpp
Expand Up @@ -109,12 +109,12 @@ class PDAL_DLL Streamable : public virtual Stage
{}

/**
Determine if a pipeline is streamable.
Find the first nonstreamable stage in a pipeline.
\return NULL if the pipeline is streamable, otherwise returns
a pointer to a stage that's not streamable.
\return NULL if the pipeline is streamable, otherwise return
a pointer to the first found stage that's not streamable.
*/
const Stage *checkStreamable() const;
const Stage *findNonstreamable() const;
};

} // namespace pdal

0 comments on commit f8b006b

Please sign in to comment.