C99 initializers make it easier to see what's going on, and make cscope databases for example simpler to search. Can we use them in e2fsprogs, or are there still ancient-compiler-compatibility issues? Signed-off-by: Eric Sandeen <sandeen@xxxxxxxxxx> --- Index: e2fsprogs/lib/ext2fs/dosio.c =================================================================== --- e2fsprogs.orig/lib/ext2fs/dosio.c +++ e2fsprogs/lib/ext2fs/dosio.c @@ -58,14 +58,14 @@ static errcode_t dos_write_blk(io_channe static errcode_t dos_flush(io_channel channel); static struct struct_io_manager struct_dos_manager = { - EXT2_ET_MAGIC_IO_MANAGER, - "DOS I/O Manager", - dos_open, - dos_close, - dos_set_blksize, - dos_read_blk, - dos_write_blk, - dos_flush + .magic = EXT2_ET_MAGIC_IO_MANAGER, + .name = "DOS I/O Manager", + .open = dos_open, + .close = dos_close, + .set_blksize = dos_set_blksize, + .read_blk = dos_read_blk, + .write_blk = dos_write_blk, + .flush = dos_flush }; io_manager dos_io_manager = &struct_dos_manager; Index: e2fsprogs/lib/ext2fs/inode_io.c =================================================================== --- e2fsprogs.orig/lib/ext2fs/inode_io.c +++ e2fsprogs/lib/ext2fs/inode_io.c @@ -62,19 +62,19 @@ static errcode_t inode_write_blk64(io_ch unsigned long long block, int count, const void *data); static struct struct_io_manager struct_inode_manager = { - EXT2_ET_MAGIC_IO_MANAGER, - "Inode I/O Manager", - inode_open, - inode_close, - inode_set_blksize, - inode_read_blk, - inode_write_blk, - inode_flush, - inode_write_byte, - NULL, - NULL, - inode_read_blk64, - inode_write_blk64 + .magic = EXT2_ET_MAGIC_IO_MANAGER, + .name = "Inode I/O Manager", + .open = inode_open, + .close = inode_close, + .set_blksize = inode_set_blksize, + .read_blk = inode_read_blk, + .write_blk = inode_write_blk, + .flush = inode_flush, + .write_byte = inode_write_byte, + .set_option = NULL, + .get_stats = NULL, + .read_blk64 = inode_read_blk64, + .write_blk64 = inode_write_blk64 }; io_manager inode_io_manager = &struct_inode_manager; Index: e2fsprogs/lib/ext2fs/nt_io.c =================================================================== --- e2fsprogs.orig/lib/ext2fs/nt_io.c +++ e2fsprogs/lib/ext2fs/nt_io.c @@ -230,14 +230,14 @@ static errcode_t nt_write_blk(io_channel static errcode_t nt_flush(io_channel channel); static struct struct_io_manager struct_nt_manager = { - EXT2_ET_MAGIC_IO_MANAGER, - "NT I/O Manager", - nt_open, - nt_close, - nt_set_blksize, - nt_read_blk, - nt_write_blk, - nt_flush + .magic = EXT2_ET_MAGIC_IO_MANAGER, + .name = "NT I/O Manager", + .open = nt_open, + .close = nt_close, + .set_blksize = nt_set_blksize, + .read_blk = nt_read_blk, + .write_blk = nt_write_blk, + .flush = nt_flush }; Index: e2fsprogs/lib/ext2fs/test_io.c =================================================================== --- e2fsprogs.orig/lib/ext2fs/test_io.c +++ e2fsprogs/lib/ext2fs/test_io.c @@ -76,19 +76,19 @@ static errcode_t test_get_stats(io_chann static struct struct_io_manager struct_test_manager = { - EXT2_ET_MAGIC_IO_MANAGER, - "Test I/O Manager", - test_open, - test_close, - test_set_blksize, - test_read_blk, - test_write_blk, - test_flush, - test_write_byte, - test_set_option, - test_get_stats, - test_read_blk64, - test_write_blk64, + .magic = EXT2_ET_MAGIC_IO_MANAGER, + .name = "Test I/O Manager", + .open = test_open, + .close = test_close, + .set_blksize = test_set_blksize, + .read_blk = test_read_blk, + .write_blk = test_write_blk, + .flush = test_flush, + .write_byte = test_write_byte, + .set_option = test_set_option, + .get_stats = test_get_stats, + .read_blk64 = test_read_blk64, + .write_blk64 = test_write_blk64, }; io_manager test_io_manager = &struct_test_manager; Index: e2fsprogs/lib/ext2fs/undo_io.c =================================================================== --- e2fsprogs.orig/lib/ext2fs/undo_io.c +++ e2fsprogs/lib/ext2fs/undo_io.c @@ -84,16 +84,16 @@ static errcode_t undo_set_option(io_chan const char *arg); static struct struct_io_manager struct_undo_manager = { - EXT2_ET_MAGIC_IO_MANAGER, - "Undo I/O Manager", - undo_open, - undo_close, - undo_set_blksize, - undo_read_blk, - undo_write_blk, - undo_flush, - undo_write_byte, - undo_set_option + .magic = EXT2_ET_MAGIC_IO_MANAGER, + .name = "Undo I/O Manager", + .open = undo_open, + .close = undo_close, + .set_blksize = undo_set_blksize, + .read_blk = undo_read_blk, + .write_blk = undo_write_blk, + .flush = undo_flush, + .write_byte = undo_write_byte, + .set_option = undo_set_option }; io_manager undo_io_manager = &struct_undo_manager; Index: e2fsprogs/lib/ext2fs/unix_io.c =================================================================== --- e2fsprogs.orig/lib/ext2fs/unix_io.c +++ e2fsprogs/lib/ext2fs/unix_io.c @@ -113,23 +113,23 @@ static errcode_t unix_write_blk64(io_cha #endif static struct struct_io_manager struct_unix_manager = { - EXT2_ET_MAGIC_IO_MANAGER, - "Unix I/O Manager", - unix_open, - unix_close, - unix_set_blksize, - unix_read_blk, - unix_write_blk, - unix_flush, + .magic = EXT2_ET_MAGIC_IO_MANAGER, + .name = "Unix I/O Manager", + .open = unix_open, + .close = unix_close, + .set_blksize = unix_set_blksize, + .read_blk = unix_read_blk, + .write_blk = unix_write_blk, + .flush = unix_flush, #ifdef NEED_BOUNCE_BUFFER - 0, + .write_byte = 0, #else - unix_write_byte, + .write_byte = unix_write_byte, #endif - unix_set_option, - unix_get_stats, - unix_read_blk64, - unix_write_blk64, + .set_option = unix_set_option, + .get_stats = unix_get_stats, + .read_blk64 = unix_read_blk64, + .write_blk64 = unix_write_blk64, }; io_manager unix_io_manager = &struct_unix_manager; -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html