[RFA] Submit process record and replay fourth time, 7/8
Pedro Alves
pedro@codesourcery.com
Sun Mar 22 15:37:00 GMT 2009
The way this is written looks like there's a bunch
of unneeded duplication.
+#define I386_LINUX_RECORD_SIZE__old_kernel_stat 32
+ /* Initialize the i386_linux_record_tdep. */
+ i386_linux_record_tdep.size__old_kernel_stat =
+ I386_LINUX_RECORD_SIZE__old_kernel_stat;
Are the I386_LINUX_RECORD_SIZE__XXXXX defines used for
anything else other than initializing the corresponding
i386_linux_record_tdep.size_XXXXX field? If not, why not
drop the defines and just write
i386_linux_record_tdep.size__old_kernel_stat = 32;
?
Personally, I think that stamping everything with "record" in
its name is a mistake. This is about the linux syscall abi,
could be reused for something else...
--
Pedro Alves
More information about the Gdb-patches
mailing list