Changeset 4339f09 in mainline for tools/mkext2.py

Timestamp:

2013-06-28T07:46:31Z (12 years ago)

Author:

Vojtech Horky <vojtechhorky@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

e52b4b5

Parents:

3de67b4c (diff), dba3e2c (diff)
Note: this is a merge changeset, the changes displayed below correspond to the merge itself.
Use the (diff) links above to see all the changes relative to each parent.

Message:

Mostly C style (obey clang suggestions)

Merge from lp:~vojtech-horky/helenos/misc.

Most changes driven by warnings/errors from clang, such as type
redefinitions (typedef struct …) or possibly uninitizalied
variables. Clang still emits some warnings but a lot less than
before . As a result, kernel can be built with clang/-Werror.

Also updated mkext2.py to work with Python 3 again.

File:

• tools/mkext2.py (modified) (3 diffs)

tools/mkext2.py

@@ -39 +39 @@
 import uuid
 from imgutil import *
+
+if sys.version >= '3':
+        xrange = range
 
 GDE_SIZE = 32
@@ -530 +533 @@
                 data.flags = 0
                 blockconv = lambda x: 0 if x == None else x
-                data.direct_blocks = map(blockconv, self.direct)
-                data.indirect_blocks = map(blockconv, self.indirect)
+                data.direct_blocks = list(map(blockconv, self.direct))
+                data.indirect_blocks = list(map(blockconv, self.indirect))
                 data.version = 0
                 data.file_acl = 0
@@ -567 +570 @@
                 head.inode_type = self.type
                 inode.write(head.pack())
-                inode.write(self.name+'\0')
+                inode.write(self.name+'\0'.encode())
                 inode.align_pos(4)