1963529 - It might be better that a constructor of `AutoRestore` takes new value too

Status: NEW

(Core :: XPCOM, enhancement)

Description

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)(sickish, maybe slower respose)]

Currently, you need to write:

  AutoRestore<bool> restoreFoo(mFoo);
  mFoo = false;

It might be nicer if this can be rewritten as:

  AutoRestore<bool> restoreFoo(mFoo, false);

So, the constructor like this:

AutoRestore(T& aValue, T aNewValue) : mLocation(aValue), mValue(aValue) {
  mLocation = aNewValue;
}