Correct get_declaration_of not to skip first symbol

When lookig backward there for loop will never check symbols.cbegin()
as it is excluded by `i != symbols.cbegin()` condition.

Change normal iterators to `std::reverse_iterator`.

Author: filipsajdak

source/sema.h

@@ -236,19 +236,21 @@ class sema
             ++i;
         }
 
-        while (i == symbols.cend() || !i->start) {
-            --i;
+        auto ri = std::reverse_iterator(i);
+
+        while (ri == symbols.crbegin() || !ri->start) {
+            ++ri;
         }
 
-        auto depth = i->depth;
+        auto depth = ri->depth;
 
         //  Then look backward to find the first declaration of
         //  this name that is not deeper (in a nested scope)
-        for ( ; i != symbols.cbegin(); --i )
+        for (; ri != symbols.crend(); ++ri )
         {
-            if (i->sym.index() == symbol::active::declaration && i->depth <= depth)
+            if (ri->sym.index() == symbol::active::declaration && ri->depth <= depth)
             {
-                auto const& decl = std::get<symbol::active::declaration>(i->sym);
+                auto const& decl = std::get<symbol::active::declaration>(ri->sym);
 
                 //  Don't look beyond the current function
                 assert(decl.declaration);
@@ -260,7 +262,7 @@ class sema
                 if (decl.identifier && *decl.identifier == t) {
                     return &decl;
                 }
-                depth = i->depth;
+                depth = ri->depth;
             }
         }