Skip to content

Commit

Permalink
Remove redundant virtual fn implementations from the compiler. (#2268)
Browse files Browse the repository at this point in the history
  • Loading branch information
InsertCreativityHere authored Jun 4, 2024
1 parent 3f2005d commit ae7e172
Show file tree
Hide file tree
Showing 8 changed files with 0 additions and 92 deletions.
20 changes: 0 additions & 20 deletions cpp/src/slice2cpp/Gen.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -993,11 +993,6 @@ Slice::Gen::MetaDataVisitor::visitModuleStart(const ModulePtr& p)
return true;
}

void
Slice::Gen::MetaDataVisitor::visitModuleEnd(const ModulePtr&)
{
}

void
Slice::Gen::MetaDataVisitor::visitClassDecl(const ClassDeclPtr& p)
{
Expand All @@ -1013,11 +1008,6 @@ Slice::Gen::MetaDataVisitor::visitClassDefStart(const ClassDefPtr& p)
return true;
}

void
Slice::Gen::MetaDataVisitor::visitClassDefEnd(const ClassDefPtr&)
{
}

bool
Slice::Gen::MetaDataVisitor::visitExceptionStart(const ExceptionPtr& p)
{
Expand All @@ -1026,11 +1016,6 @@ Slice::Gen::MetaDataVisitor::visitExceptionStart(const ExceptionPtr& p)
return true;
}

void
Slice::Gen::MetaDataVisitor::visitExceptionEnd(const ExceptionPtr&)
{
}

bool
Slice::Gen::MetaDataVisitor::visitStructStart(const StructPtr& p)
{
Expand All @@ -1039,11 +1024,6 @@ Slice::Gen::MetaDataVisitor::visitStructStart(const StructPtr& p)
return true;
}

void
Slice::Gen::MetaDataVisitor::visitStructEnd(const StructPtr&)
{
}

void
Slice::Gen::MetaDataVisitor::visitOperation(const OperationPtr& p)
{
Expand Down
4 changes: 0 additions & 4 deletions cpp/src/slice2cpp/Gen.h
Original file line number Diff line number Diff line change
Expand Up @@ -210,14 +210,10 @@ namespace Slice
public:
bool visitUnitStart(const UnitPtr&) final;
bool visitModuleStart(const ModulePtr&) final;
void visitModuleEnd(const ModulePtr&) final;
void visitClassDecl(const ClassDeclPtr&) final;
bool visitClassDefStart(const ClassDefPtr&) final;
void visitClassDefEnd(const ClassDefPtr&) final;
bool visitExceptionStart(const ExceptionPtr&) final;
void visitExceptionEnd(const ExceptionPtr&) final;
bool visitStructStart(const StructPtr&) final;
void visitStructEnd(const StructPtr&) final;
void visitOperation(const OperationPtr&) final;
void visitDataMember(const DataMemberPtr&) final;
void visitSequence(const SequencePtr&) final;
Expand Down
20 changes: 0 additions & 20 deletions cpp/src/slice2cs/CsUtil.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -2051,11 +2051,6 @@ Slice::CsGenerator::MetaDataVisitor::visitModuleStart(const ModulePtr& p)
return true;
}

void
Slice::CsGenerator::MetaDataVisitor::visitModuleEnd(const ModulePtr&)
{
}

void
Slice::CsGenerator::MetaDataVisitor::visitClassDecl(const ClassDeclPtr& p)
{
Expand All @@ -2069,35 +2064,20 @@ Slice::CsGenerator::MetaDataVisitor::visitClassDefStart(const ClassDefPtr& p)
return true;
}

void
Slice::CsGenerator::MetaDataVisitor::visitClassDefEnd(const ClassDefPtr&)
{
}

bool
Slice::CsGenerator::MetaDataVisitor::visitExceptionStart(const ExceptionPtr& p)
{
validate(p);
return true;
}

void
Slice::CsGenerator::MetaDataVisitor::visitExceptionEnd(const ExceptionPtr&)
{
}

bool
Slice::CsGenerator::MetaDataVisitor::visitStructStart(const StructPtr& p)
{
validate(p);
return true;
}

void
Slice::CsGenerator::MetaDataVisitor::visitStructEnd(const StructPtr&)
{
}

void
Slice::CsGenerator::MetaDataVisitor::visitOperation(const OperationPtr& p)
{
Expand Down
4 changes: 0 additions & 4 deletions cpp/src/slice2cs/CsUtil.h
Original file line number Diff line number Diff line change
Expand Up @@ -117,14 +117,10 @@ namespace Slice
public:
virtual bool visitUnitStart(const UnitPtr&);
virtual bool visitModuleStart(const ModulePtr&);
virtual void visitModuleEnd(const ModulePtr&);
virtual void visitClassDecl(const ClassDeclPtr&);
virtual bool visitClassDefStart(const ClassDefPtr&);
virtual void visitClassDefEnd(const ClassDefPtr&);
virtual bool visitExceptionStart(const ExceptionPtr&);
virtual void visitExceptionEnd(const ExceptionPtr&);
virtual bool visitStructStart(const StructPtr&);
virtual void visitStructEnd(const StructPtr&);
virtual void visitOperation(const OperationPtr&);
virtual void visitParamDecl(const ParamDeclPtr&);
virtual void visitDataMember(const DataMemberPtr&);
Expand Down
5 changes: 0 additions & 5 deletions cpp/src/slice2js/Gen.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1264,11 +1264,6 @@ Slice::Gen::TypesVisitor::visitModuleStart(const ModulePtr& p)
return true;
}

void
Slice::Gen::TypesVisitor::visitModuleEnd(const ModulePtr&)
{
}

bool
Slice::Gen::TypesVisitor::visitClassDefStart(const ClassDefPtr& p)
{
Expand Down
1 change: 0 additions & 1 deletion cpp/src/slice2js/Gen.h
Original file line number Diff line number Diff line change
Expand Up @@ -97,7 +97,6 @@ namespace Slice
TypesVisitor(::IceUtilInternal::Output&, std::vector<std::string>, bool);

virtual bool visitModuleStart(const ModulePtr&);
virtual void visitModuleEnd(const ModulePtr&);
virtual bool visitClassDefStart(const ClassDefPtr&);
virtual bool visitInterfaceDefStart(const InterfaceDefPtr&);
virtual bool visitExceptionStart(const ExceptionPtr&);
Expand Down
32 changes: 0 additions & 32 deletions cpp/src/slice2swift/Gen.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1088,11 +1088,6 @@ Gen::ProxyVisitor::visitModuleStart(const ModulePtr& p)
return p->hasInterfaceDefs();
}

void
Gen::ProxyVisitor::visitModuleEnd(const ModulePtr&)
{
}

bool
Gen::ProxyVisitor::visitInterfaceDefStart(const InterfaceDefPtr& p)
{
Expand Down Expand Up @@ -1426,24 +1421,8 @@ Gen::ValueVisitor::visitClassDefEnd(const ClassDefPtr&)
out << eb;
}

void
Gen::ValueVisitor::visitOperation(const OperationPtr&)
{
}

Gen::ObjectVisitor::ObjectVisitor(::IceUtilInternal::Output& o) : out(o) {}

bool
Gen::ObjectVisitor::visitModuleStart(const ModulePtr&)
{
return true;
}

void
Gen::ObjectVisitor::visitModuleEnd(const ModulePtr&)
{
}

bool
Gen::ObjectVisitor::visitInterfaceDefStart(const InterfaceDefPtr& p)
{
Expand Down Expand Up @@ -1617,17 +1596,6 @@ Gen::ObjectVisitor::visitOperation(const OperationPtr& op)

Gen::ObjectExtVisitor::ObjectExtVisitor(::IceUtilInternal::Output& o) : out(o) {}

bool
Gen::ObjectExtVisitor::visitModuleStart(const ModulePtr&)
{
return true;
}

void
Gen::ObjectExtVisitor::visitModuleEnd(const ModulePtr&)
{
}

bool
Gen::ObjectExtVisitor::visitInterfaceDefStart(const InterfaceDefPtr& p)
{
Expand Down
6 changes: 0 additions & 6 deletions cpp/src/slice2swift/Gen.h
Original file line number Diff line number Diff line change
Expand Up @@ -78,7 +78,6 @@ namespace Slice
ProxyVisitor(::IceUtilInternal::Output&);

virtual bool visitModuleStart(const ModulePtr&);
virtual void visitModuleEnd(const ModulePtr&);
virtual bool visitInterfaceDefStart(const InterfaceDefPtr&);
virtual void visitInterfaceDefEnd(const InterfaceDefPtr&);
virtual void visitOperation(const OperationPtr&);
Expand All @@ -94,7 +93,6 @@ namespace Slice

virtual bool visitClassDefStart(const ClassDefPtr&);
virtual void visitClassDefEnd(const ClassDefPtr&);
virtual void visitOperation(const OperationPtr&);

private:
IceUtilInternal::Output& out;
Expand All @@ -105,8 +103,6 @@ namespace Slice
public:
ObjectVisitor(::IceUtilInternal::Output&);

virtual bool visitModuleStart(const ModulePtr&);
virtual void visitModuleEnd(const ModulePtr&);
virtual bool visitInterfaceDefStart(const InterfaceDefPtr&);
virtual void visitInterfaceDefEnd(const InterfaceDefPtr&);
virtual void visitOperation(const OperationPtr&);
Expand All @@ -120,8 +116,6 @@ namespace Slice
public:
ObjectExtVisitor(::IceUtilInternal::Output&);

virtual bool visitModuleStart(const ModulePtr&);
virtual void visitModuleEnd(const ModulePtr&);
virtual bool visitInterfaceDefStart(const InterfaceDefPtr&);
virtual void visitInterfaceDefEnd(const InterfaceDefPtr&);
virtual void visitOperation(const OperationPtr&);
Expand Down

0 comments on commit ae7e172

Please sign in to comment.