Skip to content

Commit

Permalink
Delete move constructor from TaggedStringStream (pytorch#28604)
Browse files Browse the repository at this point in the history
Summary:
Pull Request resolved: pytorch#28604

This isn't used anywhere, and it doesn't work with older libstdc++
because std::ostringstream is not copyable or movable.

Test Plan: Internal android build.

Reviewed By: jamesr66a

Differential Revision: D18099511

fbshipit-source-id: 1ffb49303aa5d7890ca7f057b21886f88c04ce20
  • Loading branch information
dreiss authored and facebook-github-bot committed Oct 24, 2019
1 parent 52e0a94 commit df81cb2
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion torch/csrc/jit/passes/python_print.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,6 @@ struct PythonPrintImpl {
class TaggedStringStream {
public:
TaggedStringStream(const SourceRangeStack* srs) : srs_(srs) {}
TaggedStringStream(TaggedStringStream&& rhs) = default;

TaggedStringStream& operator<<(const std::string& s) {
// This prevents having redundant entries at the same offset,
Expand Down

0 comments on commit df81cb2

Please sign in to comment.