some more style fixes

5 files changed, 8 insertions, 21 deletions

diff --git a/libsolidity/NameAndTypeResolver.h b/libsolidity/NameAndTypeResolver.h
index fd939de8..4fe8dd8c 100644
--- a/libsolidity/NameAndTypeResolver.h
+++ b/libsolidity/NameAndTypeResolver.h
@@ -44,14 +44,14 @@ class NameAndTypeResolver: private boost::noncopyable
 public:
 	NameAndTypeResolver(std::vector<Declaration const*> const& _globals, ErrorList& _errors);
 	/// Registers all declarations found in the source unit.
-	/// @returns false in case of type error.
+	/// @returns false in case of error.
 	bool registerDeclarations(SourceUnit& _sourceUnit);
 	/// Resolves all names and types referenced from the given contract.
-	/// @returns false in case of type error.
+	/// @returns false in case of error.
 	bool resolveNamesAndTypes(ContractDefinition& _contract);
 	/// Updates the given global declaration (used for "this"). Not to be used with declarations
 	/// that create their own scope.
-	/// @returns false in case of type error.
+	/// @returns false in case of error.
 	bool updateDeclaration(Declaration const& _declaration);
 
 	/// Resolves the given @a _name inside the scope @a _scope. If @a _scope is omitted,
diff --git a/libsolidity/Parser.cpp b/libsolidity/Parser.cpp
index 58632b85..47d1fb2f 100644
--- a/libsolidity/Parser.cpp
+++ b/libsolidity/Parser.cpp
@@ -66,7 +66,8 @@ private:
 
 ASTPointer<SourceUnit> Parser::parse(shared_ptr<Scanner> const& _scanner)
 {
-	try{
+	try
+	{
 		m_scanner = _scanner;
 		ASTNodeFactory nodeFactory(*this);
 		vector<ASTPointer<ASTNode>> nodes;
diff --git a/libsolidity/Parser.h b/libsolidity/Parser.h
index 3fc3768e..ee5be51c 100644
--- a/libsolidity/Parser.h
+++ b/libsolidity/Parser.h
@@ -34,8 +34,7 @@ class Scanner;
 class Parser
 {
 public:
-	Parser(ErrorList& errors):
-		m_errors(errors){};
+	Parser(ErrorList& errors): m_errors(errors){};
 
 	ASTPointer<SourceUnit> parse(std::shared_ptr<Scanner> const& _scanner);
 	std::shared_ptr<std::string const> const& sourceName() const;
diff --git a/libsolidity/TypeChecker.cpp b/libsolidity/TypeChecker.cpp
index 9ceb8203..ca5b1eb7 100644
--- a/libsolidity/TypeChecker.cpp
+++ b/libsolidity/TypeChecker.cpp
@@ -43,19 +43,7 @@ bool TypeChecker::checkTypeRequirements(const ContractDefinition& _contract)
 		if (m_errors.empty())
 			throw; // Something is weird here, rather throw again.
 	}
-
-return	Error::containsOnlyWarnings(m_errors);
-//	bool success = true;
-//	for (auto const& it: m_errors)
-//	{
-//		auto e = dynamic_cast<Error const*>(it.get());
-//		if (e->type() != Error::Type::Warning)
-//		{
-//			success = false;
-//			break;
-//		}
-//	}
-//	return success;
+	return	Error::containsOnlyWarnings(m_errors);
 }
 
 TypePointer const& TypeChecker::type(Expression const& _expression) const
@@ -72,7 +60,6 @@ TypePointer const& TypeChecker::type(VariableDeclaration const& _variable) const
 
 bool TypeChecker::visit(ContractDefinition const& _contract)
 {
-
 	// We force our own visiting order here.
 	ASTNode::listAccept(_contract.definedStructs(), *this);
 	ASTNode::listAccept(_contract.baseContracts(), *this);
diff --git a/libsolidity/TypeChecker.h b/libsolidity/TypeChecker.h
index de095e3b..d9cb39ae 100644
--- a/libsolidity/TypeChecker.h
+++ b/libsolidity/TypeChecker.h
@@ -42,7 +42,7 @@ namespace solidity
 class TypeChecker: private ASTConstVisitor
 {
 public:
-	/// @_errors the reference to the list of errors and warnings to add them found during type checking.
+	/// @param _errors the reference to the list of errors and warnings to add them found during type checking.
 	TypeChecker(ErrorList& _errors): m_errors(_errors) {}
 
 	/// Performs type checking on the given contract and all of its sub-nodes.