From fa6d4c63dcefabd6a4a63dfe857959544c583ab4 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Pawe=C5=82=20Bylica?= <pawel.bylica@imapp.pl>
Date: Wed, 29 Oct 2014 16:19:13 +0100
Subject: [PATCH] Improve Stack code formatting

---
 libevmjit/Stack.cpp | 86 +++++++++++++++++++++------------------------
 libevmjit/Stack.h   |  2 --
 2 files changed, 41 insertions(+), 47 deletions(-)

diff --git a/libevmjit/Stack.cpp b/libevmjit/Stack.cpp
index 144cd5d54..d6538a22d 100644
--- a/libevmjit/Stack.cpp
+++ b/libevmjit/Stack.cpp
@@ -14,8 +14,8 @@ namespace eth
 namespace jit
 {
 
-Stack::Stack(llvm::IRBuilder<>& _builder, RuntimeManager& _runtimeManager)
-	: CompilerHelper(_builder),
+Stack::Stack(llvm::IRBuilder<>& _builder, RuntimeManager& _runtimeManager):
+	CompilerHelper(_builder),
 	m_runtimeManager(_runtimeManager)
 {
 	m_arg = m_builder.CreateAlloca(Type::i256, nullptr, "stack.arg");
@@ -36,9 +36,6 @@ Stack::Stack(llvm::IRBuilder<>& _builder, RuntimeManager& _runtimeManager)
 	m_set = Function::Create(FunctionType::get(Type::Void, getSetArgTypes, false), Linkage::ExternalLinkage, "stack_set", module);
 }
 
-Stack::~Stack()
-{}
-
 llvm::Value* Stack::get(size_t _index)
 {
 	m_builder.CreateCall3(m_get, m_runtimeManager.getRuntimePtr(), llvm::ConstantInt::get(Type::Size, _index, false), m_arg);
@@ -71,46 +68,45 @@ size_t Stack::maxStackSize = 0;
 
 extern "C"
 {
-
-using namespace dev::eth::jit;
-
-EXPORT void stack_pop(Runtime* _rt, uint64_t _count)
-{
-	auto& stack = _rt->getStack();
-	if (stack.size() < _count)
-		longjmp(_rt->getJmpBuf(), static_cast<uint64_t>(ReturnCode::StackTooSmall));
-
-	stack.erase(stack.end() - _count, stack.end());
-}
-
-EXPORT void stack_push(Runtime* _rt, i256* _word)
-{
-	auto& stack = _rt->getStack();
-	stack.push_back(*_word);
-
-	if (stack.size() > Stack::maxStackSize)
-		Stack::maxStackSize = stack.size();
-}
-
-EXPORT void stack_get(Runtime* _rt, uint64_t _index, i256* _ret)
-{
-	auto& stack = _rt->getStack();
-	// TODO: encode _index and stack size in the return code
-	if (stack.size() <= _index)
-		longjmp(_rt->getJmpBuf(), static_cast<uint64_t>(ReturnCode::StackTooSmall));
-
-	*_ret = *(stack.rbegin() + _index);
-}
-
-EXPORT void stack_set(Runtime* _rt, uint64_t _index, i256* _word)
-{
-	auto& stack = _rt->getStack();
-	// TODO: encode _index and stack size in the return code
-	if (stack.size() <= _index)
-		longjmp(_rt->getJmpBuf(), static_cast<uint64_t>(ReturnCode::StackTooSmall));
-
-	*(stack.rbegin() + _index) = *_word;
-}
+	using namespace dev::eth::jit;
+
+	EXPORT void stack_pop(Runtime* _rt, uint64_t _count)
+	{
+		auto& stack = _rt->getStack();
+		if (stack.size() < _count)
+			longjmp(_rt->getJmpBuf(), static_cast<uint64_t>(ReturnCode::StackTooSmall));
+
+		stack.erase(stack.end() - _count, stack.end());
+	}
+
+	EXPORT void stack_push(Runtime* _rt, i256* _word)
+	{
+		auto& stack = _rt->getStack();
+		stack.push_back(*_word);
+
+		if (stack.size() > Stack::maxStackSize)
+			Stack::maxStackSize = stack.size();
+	}
+
+	EXPORT void stack_get(Runtime* _rt, uint64_t _index, i256* _ret)
+	{
+		auto& stack = _rt->getStack();
+		// TODO: encode _index and stack size in the return code
+		if (stack.size() <= _index)
+			longjmp(_rt->getJmpBuf(), static_cast<uint64_t>(ReturnCode::StackTooSmall));
+
+		*_ret = *(stack.rbegin() + _index);
+	}
+
+	EXPORT void stack_set(Runtime* _rt, uint64_t _index, i256* _word)
+	{
+		auto& stack = _rt->getStack();
+		// TODO: encode _index and stack size in the return code
+		if (stack.size() <= _index)
+			longjmp(_rt->getJmpBuf(), static_cast<uint64_t>(ReturnCode::StackTooSmall));
+
+		*(stack.rbegin() + _index) = *_word;
+	}
 
 } // extern "C"
 
diff --git a/libevmjit/Stack.h b/libevmjit/Stack.h
index db9032ec4..3e8881e4f 100644
--- a/libevmjit/Stack.h
+++ b/libevmjit/Stack.h
@@ -15,9 +15,7 @@ class RuntimeManager;
 class Stack : public CompilerHelper
 {
 public:
-
 	Stack(llvm::IRBuilder<>& builder, RuntimeManager& runtimeManager);
-	virtual ~Stack();
 
 	llvm::Value* get(size_t _index);
 	void set(size_t _index, llvm::Value* _value);