3121 Marko Mäkelä 2010-06-24
trx_undo_parse_page_header(): Silence a bogus warning.
modified:
storage/innobase/trx/trx0undo.c
3120 Marko Mäkelä 2010-06-24
After-review cleanup of Bug#57428: replace the dulint struct with a 64-bit int
trx_undo_build_roll_ptr(): Clarify the assertion on is_insert being 0 or 1.
TRX_SYS_FILE_FORMAT_TAG: Remove "extra" space after period in the comment.
See also http://desktoppub.about.com/cs/typespacing/a/onetwospaces.htm
modified:
storage/innobase/include/trx0sys.h
storage/innobase/include/trx0undo.ic
=== modified file 'storage/innobase/trx/trx0undo.c'
--- a/storage/innobase/trx/trx0undo.c revid:marko.makela@stripped624060929-z7dj4btnkyhqm229
+++ b/storage/innobase/trx/trx0undo.c revid:marko.makela@strippedg2u908nifuwi
@@ -707,6 +707,12 @@ trx_undo_parse_page_header(
mtr_t* mtr) /*!< in: mtr or NULL */
{
trx_id_t trx_id;
+ /* Silence a GCC warning about possibly uninitialized variable
+ when mach_ull_parse_compressed() is not inlined. */
+ ut_d(trx_id = 0);
+ /* Declare the variable uninitialized in Valgrind, so that the
+ above initialization will not mask any bugs. */
+ UNIV_MEM_INVALID(&trx_id, sizeof trx_id);
ptr = mach_ull_parse_compressed(ptr, end_ptr, &trx_id);
Attachment: [text/bzr-bundle] bzr/marko.makela@oracle.com-20100624072140-zp22g2u908nifuwi.bundle
| Thread |
|---|
| • bzr push into mysql-trunk-innodb branch (marko.makela:3120 to 3121) | marko.makela | 24 Jun |