From 7b72aa68be391f7a0f83a0beed38f5670650eedd Mon Sep 17 00:00:00 2001
From: Liana Husikyan <liana@ethdev.com>
Date: Mon, 4 May 2015 14:46:52 +0200
Subject: [PATCH] fixed the order of error msgs

---
 libsolidity/NameAndTypeResolver.cpp | 19 ++++++++++++++++---
 1 file changed, 16 insertions(+), 3 deletions(-)

diff --git a/libsolidity/NameAndTypeResolver.cpp b/libsolidity/NameAndTypeResolver.cpp
index 56e81e372..a81c82380 100644
--- a/libsolidity/NameAndTypeResolver.cpp
+++ b/libsolidity/NameAndTypeResolver.cpp
@@ -355,16 +355,29 @@ void DeclarationRegistrationHelper::registerDeclaration(Declaration& _declaratio
 {
 	if (!m_scopes[m_currentScope].registerDeclaration(_declaration, !_declaration.isVisibleInContract()))
 	{
+		SourceLocation firstDeclarationLocation;
+		SourceLocation secondDeclarationLocation;
+
+		if (_declaration.getLocation().start < m_scopes[m_currentScope].conflictingDeclaration(_declaration)->getLocation().start)
+		{
+			firstDeclarationLocation = _declaration.getLocation();
+			secondDeclarationLocation = m_scopes[m_currentScope].conflictingDeclaration(_declaration)->getLocation();
+		}
+		else
+		{
+			firstDeclarationLocation = m_scopes[m_currentScope].conflictingDeclaration(_declaration)->getLocation();
+			secondDeclarationLocation = _declaration.getLocation();
+		}
 		solAssert(m_scopes[m_currentScope].conflictingDeclaration(_declaration), "");
 		BOOST_THROW_EXCEPTION(DeclarationError()
-			<< errinfo_sourceLocation(_declaration.getLocation())
+			<< errinfo_sourceLocation(secondDeclarationLocation)
 			<< errinfo_comment("Identifier already declared.")
 			<< errinfo_secondarySourceLocation(SecondarySourceLocation().append(
 				"The previous declaration is here:",
-				m_scopes[m_currentScope].conflictingDeclaration(_declaration)->getLocation())));
+				firstDeclarationLocation
+			)));
 	}
 
-	//@todo the exception should also contain the location of the first declaration
 	_declaration.setScope(m_currentScope);
 	if (_opensScope)
 		enterNewSubScope(_declaration);