These members are later properly set in EnsureInitialized which is called before any method that reads them, but there's no reason to leave them uninitialized in the constructor considering the annoying warning noise that it generates, and if a bug is introduced it'll be much easier to debug if we dereference nullptr than some random address.
Assignee: nobody → nical.bugzilla
Whiteboard: [CID 749989]
Previously uploaded the wrong version of the patch.
Whiteboard: [CID 749989] → [CID 749989][CID 1123714]
Attachment #8709421 - Flags: review?(matt.woodrow) → review+
Attachment #8709458 - Flags: review?(matt.woodrow) → review+
You need to log in before you can comment on or make changes to this bug.