In several places, we set gNextMessageViewIndexAfterDelete = -2 and check for that later to do $stuff (see URL field for an MXR link). This is pretty much incomprehensible. We should define a constant with a descriptive name for this to enhance code readability, in concordance with nsMsgViewIndex_None = 0xFFFFFFFF;