Skip to content

Commit

Permalink
tidy: Fix some class member naming. (libmythfreemheg/Engine)
Browse files Browse the repository at this point in the history
The clang-tidy "identifier naming" checker pointed out a few places
where class member function names don't use the convention of starting
with 'm_' and then using camelBack case for the rest of the variable
name.  Fix these for naming consistency across the code base.

https://clang.llvm.org/extra/clang-tidy/checks/readability-identifier-naming.html
  • Loading branch information
linuxdude42 committed Jul 27, 2020
1 parent 98486b1 commit 1e974f0
Show file tree
Hide file tree
Showing 8 changed files with 159 additions and 159 deletions.
10 changes: 5 additions & 5 deletions mythtv/libs/libmythfreemheg/DynamicLineArt.cpp
Expand Up @@ -185,20 +185,20 @@ void MHDrawPoly::Initialise(MHParseNode *p, MHEngine *engine)
for (int i = 0; i < args->GetSeqCount(); i++)
{
auto *pPoint = new MHPointArg;
m_Points.Append(pPoint);
m_points.Append(pPoint);
pPoint->Initialise(args->GetSeqN(i), engine);
}
}

void MHDrawPoly::Perform(MHEngine *engine)
{
int nPoints = m_Points.Size();
int nPoints = m_points.Size();
int *xArray = new int[nPoints];
int *yArray = new int[nPoints];

for (int i = 0; i < nPoints; i++)
{
MHPointArg *pPoint = m_Points[i];
MHPointArg *pPoint = m_points[i];
xArray[i] = pPoint->m_x.GetValue(engine);
yArray[i] = pPoint->m_y.GetValue(engine);
}
Expand All @@ -212,9 +212,9 @@ void MHDrawPoly::PrintArgs(FILE *fd, int /*nTabs*/) const
{
fprintf(fd, " ( ");

for (int i = 0; i < m_Points.Size(); i++)
for (int i = 0; i < m_points.Size(); i++)
{
m_Points[i]->PrintMe(fd, 0);
m_points[i]->PrintMe(fd, 0);
}

fprintf(fd, " )\n");
Expand Down
2 changes: 1 addition & 1 deletion mythtv/libs/libmythfreemheg/DynamicLineArt.h
Expand Up @@ -152,7 +152,7 @@ class MHDrawPoly: public MHElemAction {
protected:
void PrintArgs(FILE *fd, int nTabs) const override; // MHElemAction
bool m_fIsPolygon;
MHOwnPtrSequence<MHPointArg> m_Points; // List of points
MHOwnPtrSequence<MHPointArg> m_points; // List of points
};

#endif

0 comments on commit 1e974f0

Please sign in to comment.