Skip to content

Commit 3818ff2

Browse files
authored
Merge pull request #1422 from N-Dekker/Declare-Stream-data-private
style: Declare data of Stream classes `private`, instead of `protected`
2 parents 787e753 + 0fb1638 commit 3818ff2

File tree

5 files changed

+8
-7
lines changed

5 files changed

+8
-7
lines changed

include/itkInputBinaryStream.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,7 @@ class WebAssemblyInterface_EXPORT InputBinaryStream
8989
}
9090
}
9191

92-
protected:
92+
private:
9393
std::istream * m_IStream{ nullptr };
9494
bool m_DeleteIStream{ false };
9595

include/itkInputTextStream.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@ class WebAssemblyInterface_EXPORT InputTextStream
9191
}
9292
}
9393

94-
protected:
94+
private:
9595
std::istream * m_IStream{ nullptr };
9696
bool m_DeleteIStream{ false };
9797

include/itkOutputBinaryStream.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,7 @@ class WebAssemblyInterface_EXPORT OutputBinaryStream
8989
return this->m_Identifier;
9090
}
9191

92-
protected:
92+
private:
9393
std::ostream * m_OStream{ nullptr };
9494
bool m_DeleteOStream{ false };
9595

include/itkOutputTextStream.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,7 @@ class WebAssemblyInterface_EXPORT OutputTextStream
8989
return this->m_Identifier;
9090
}
9191

92-
protected:
92+
private:
9393
std::ostream * m_OStream{ nullptr };
9494
bool m_DeleteOStream{ false };
9595

include/itkWasmStringStream.h

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -123,11 +123,12 @@ class WebAssemblyInterface_EXPORT WasmStringStream : public WasmDataObject
123123
this->m_JSON = jsonStream.str();
124124
}
125125

126-
std::stringstream m_StringStream;
127-
std::string m_String;
128-
129126
void
130127
PrintSelf(std::ostream & os, Indent indent) const override;
128+
129+
private:
130+
std::stringstream m_StringStream;
131+
std::string m_String;
131132
};
132133

133134
} // namespace itk

0 commit comments

Comments
 (0)