Fix precedence rules of decorator options.

Cython/Compiler/ParseTreeTransforms.py

@@ -330,9 +330,6 @@ class ResolveOptions(CythonTransform):
             return None
 
     def visit_with_options(self, node, options):
-        if not options:
-            return self.visit_Node(node)
-        else:
         oldoptions = self.options
         newoptions = copy.copy(oldoptions)
         newoptions.update(options)
@@ -343,7 +340,7 @@ class ResolveOptions(CythonTransform):
  
     # Handle decorators
     def visit_DefNode(self, node):
-        options = {}
+        options = []
         
         if node.decorators:
             # Split the decorators into two lists -- real decorators and options
@@ -351,13 +348,20 @@ class ResolveOptions(CythonTransform):
             for dec in node.decorators:
                 option = self.try_to_parse_option(dec.decorator)
                 if option is not None:
-                    name, value = option
-                    options[name] = value
+                    options.append(option)
                 else:
                     realdecs.append(dec)
             node.decorators = realdecs
 
+        if options:
+            optdict = {}
+            options.reverse() # Decorators coming first take precedence
+            for option in options:
+                name, value = option
+                optdict[name] = value
             return self.visit_with_options(node, options)
+        else:
+            return self.visit_Node(node)
                                    
     # Handle with statements
     def visit_WithStatNode(self, node):

tests/run/bufaccess.pyx

@@ -543,6 +543,7 @@ def safe_get(object[int] buf, int idx):
 
 @testcase
 @cython.boundscheck(False)
+@cython.boundscheck(True)
 def unsafe_get(object[int] buf, int idx):
     """
     Access outside of the area the buffer publishes.