Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(143)

Unified Diff: src/trusted/service_runtime/nacl_syscall_handlers_gen.py

Issue 12209042: Ensure syscall functions are consistently prefixed with "NaClSys" (Closed) Base URL: svn://svn.chromium.org/native_client/trunk/src/native_client
Patch Set: Created 7 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « src/trusted/service_runtime/nacl_syscall_common.c ('k') | src/trusted/service_runtime/nacl_text.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/trusted/service_runtime/nacl_syscall_handlers_gen.py
diff --git a/src/trusted/service_runtime/nacl_syscall_handlers_gen.py b/src/trusted/service_runtime/nacl_syscall_handlers_gen.py
index 7ab5be3a26486039544b83bae667dbbf7098fc49..141b51bbad4692a7f072ea49b62951f51a9cfa65 100755
--- a/src/trusted/service_runtime/nacl_syscall_handlers_gen.py
+++ b/src/trusted/service_runtime/nacl_syscall_handlers_gen.py
@@ -117,7 +117,7 @@ SYSCALL_LIST = [
['const char *path', 'struct nacl_abi_stat *nasp']),
('NACL_sys_getdents', 'NaClSysGetdents',
['int d', 'void *buf', 'size_t count']),
- ('NACL_sys_sysbrk', 'NaClSetBreak', ['uintptr_t new_break']),
+ ('NACL_sys_brk', 'NaClSysBrk', ['uintptr_t new_break']),
('NACL_sys_mmap', 'NaClSysMmap',
['void *start', 'size_t length', 'int prot',
'int flags', 'int d', 'nacl_abi_off_t *offp']),
@@ -179,11 +179,11 @@ SYSCALL_LIST = [
['int32_t sem_handle']),
('NACL_sys_sched_yield', 'NaClSysSchedYield', []),
('NACL_sys_sysconf', 'NaClSysSysconf', ['int32_t name', 'int32_t *result']),
- ('NACL_sys_dyncode_create', 'NaClTextSysDyncodeCreate',
+ ('NACL_sys_dyncode_create', 'NaClSysDyncodeCreate',
['uint32_t dest', 'uint32_t src', 'uint32_t size']),
- ('NACL_sys_dyncode_modify', 'NaClTextSysDyncodeModify',
+ ('NACL_sys_dyncode_modify', 'NaClSysDyncodeModify',
['uint32_t dest', 'uint32_t src', 'uint32_t size']),
- ('NACL_sys_dyncode_delete', 'NaClTextSysDyncodeDelete',
+ ('NACL_sys_dyncode_delete', 'NaClSysDyncodeDelete',
['uint32_t dest', 'uint32_t size']),
('NACL_sys_second_tls_set', 'NaClSysSecondTlsSet',
['uint32_t new_value']),
@@ -329,6 +329,14 @@ def main(argv):
if not ARG_REGISTERS.has_key(arch):
raise Exception()
+ # Check naming consistency.
+ for syscall_number, func_name, alist in SYSCALL_LIST:
+ assert syscall_number.startswith('NACL_sys_'), syscall_number
+ name1 = syscall_number[len('NACL_sys_'):].replace('_', '')
+ assert func_name.startswith('NaClSys'), func_name
+ name2 = func_name[len('NaClSys'):].lower()
+ assert name1 == name2, '%r != %r' % (name1, name2)
+
data = input_src.read()
protos = SYSCALL_LIST
print >>output_dst, data
« no previous file with comments | « src/trusted/service_runtime/nacl_syscall_common.c ('k') | src/trusted/service_runtime/nacl_text.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698