summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDylan Baker <baker.dylan.c@gmail.com>2016-03-30 16:10:54 -0700
committerDylan Baker <baker.dylan.c@gmail.com>2016-03-31 15:52:22 -0700
commit0e66448334c88db352767df35f4335073b40ac06 (patch)
treeae351652da1c0685fffc2d856b5d33bcb966d00f
parent84a2cd0955770c4f7975832cd623bff7380223a2 (diff)
glapi: use xrange instead of range in python
In python 2 range returns a new list, which is okay for very small values, but can get quite expensive for larger lists. xrange on the other hand is an iterator, which requires less memory and processing power. Signed-off-by: Dylan Baker <dylanx.c.baker@intel.com>
-rw-r--r--src/mapi/glapi/gen/glX_proto_size.py2
-rw-r--r--src/mapi/glapi/gen/glX_server_table.py16
-rw-r--r--src/mapi/glapi/gen/gl_XML.py5
-rw-r--r--src/mapi/glapi/gen/gl_apitemp.py2
4 files changed, 12 insertions, 13 deletions
diff --git a/src/mapi/glapi/gen/glX_proto_size.py b/src/mapi/glapi/gen/glX_proto_size.py
index d8b9c21b3d..0c0b10689c 100644
--- a/src/mapi/glapi/gen/glX_proto_size.py
+++ b/src/mapi/glapi/gen/glX_proto_size.py
@@ -151,7 +151,7 @@ class glx_enum_function(object):
masked_enums = {}
masked_count = {}
- for i in range(0, mask + 1):
+ for i in xrange(0, mask + 1):
masked_enums[i] = "0"
masked_count[i] = 0
diff --git a/src/mapi/glapi/gen/glX_server_table.py b/src/mapi/glapi/gen/glX_server_table.py
index d1a2bb9fe7..6443a99ddf 100644
--- a/src/mapi/glapi/gen/glX_server_table.py
+++ b/src/mapi/glapi/gen/glX_server_table.py
@@ -35,7 +35,7 @@ import license
def log2(value):
- for i in range(0, 30):
+ for i in xrange(0, 30):
p = 1 << i
if p >= value:
return i
@@ -46,7 +46,7 @@ def log2(value):
def round_down_to_power_of_two(n):
"""Returns the nearest power-of-two less than or equal to n."""
- for i in range(30, 0, -1):
+ for i in xrange(30, 0, -1):
p = 1 << i
if p <= n:
return p
@@ -95,17 +95,17 @@ class function_table:
next_opcode = min_opcode + (1 << remaining_bits)
empty_children = 0
- for M in range(0, remaining_bits):
+ for M in xrange(0, remaining_bits):
op_count = 1 << (remaining_bits - M)
child_count = 1 << M
empty_children = 0
full_children = 0
- for i in range(min_opcode, next_opcode, op_count):
+ for i in xrange(min_opcode, next_opcode, op_count):
used = 0
empty = 0
- for j in range(i, i + op_count):
+ for j in xrange(i, i + op_count):
if j in self.functions:
used += 1
else:
@@ -130,7 +130,7 @@ class function_table:
count = 1
depth = 1
all_children_are_nonempty_leaf_nodes = 1
- for i in range(min_opcode, next_opcode, op_count):
+ for i in xrange(min_opcode, next_opcode, op_count):
n = self.divide_group(i, total + M)
if not (n[1] == [] and not self.is_empty_leaf(i, n[0])):
@@ -151,7 +151,7 @@ class function_table:
return [M, children, count, depth]
def is_empty_leaf(self, base_opcode, M):
- for op in range(base_opcode, base_opcode + (1 << M)):
+ for op in xrange(base_opcode, base_opcode + (1 << M)):
if op in self.functions:
return 0
break
@@ -187,7 +187,7 @@ class function_table:
print ' LEAF(%u),' % (len(self.lookup_table))
- for op in range(child_base_opcode, child_base_opcode + (1 << child_M)):
+ for op in xrange(child_base_opcode, child_base_opcode + (1 << child_M)):
if op in self.functions:
func = self.functions[op]
size = func.command_fixed_length()
diff --git a/src/mapi/glapi/gen/gl_XML.py b/src/mapi/glapi/gen/gl_XML.py
index de1001980a..695ef7f07b 100644
--- a/src/mapi/glapi/gen/gl_XML.py
+++ b/src/mapi/glapi/gen/gl_XML.py
@@ -685,7 +685,7 @@ class gl_function(gl_item):
if len(parameters) != len(self.parameters):
raise RuntimeError("Parameter count mismatch in %s. Was %d, now %d." % (name, len(self.parameters), len(parameters)))
- for j in range(0, len(parameters)):
+ for j in xrange(0, len(parameters)):
p1 = parameters[j]
p2 = self.parameters[j]
if not p1.compatible(p2):
@@ -975,9 +975,8 @@ class gl_api(object):
classify_category. Each iterated value is a tuple of the
name and number (which may be None) of the category.
"""
-
list = []
- for cat_type in range(0,4):
+ for cat_type in xrange(0,4):
keys = self.categories[cat_type].keys()
keys.sort()
diff --git a/src/mapi/glapi/gen/gl_apitemp.py b/src/mapi/glapi/gen/gl_apitemp.py
index feb5f8ed61..5143ae3ba9 100644
--- a/src/mapi/glapi/gen/gl_apitemp.py
+++ b/src/mapi/glapi/gen/gl_apitemp.py
@@ -196,7 +196,7 @@ class PrintGlOffsets(gl_XML.gl_print_base):
print ' * when someone tries to call a dynamically-registered'
print ' * extension function without a current rendering context.'
print ' */'
- for i in range(1, 100):
+ for i in xrange(1, 100):
print ' TABLE_ENTRY(Unused),'
print '};'