Page MenuHomePhabricator

Fix unused variable warning when compiling with asserts disabled.
ClosedPublic

Authored by akuegel on Oct 15 2020, 3:50 AM.

Diff Detail

Event Timeline

akuegel created this revision.Oct 15 2020, 3:50 AM
Herald added a project: Restricted Project. · View Herald TranscriptOct 15 2020, 3:50 AM
akuegel requested review of this revision.Oct 15 2020, 3:50 AM
bkramer accepted this revision.Oct 15 2020, 3:53 AM

The code now essentially says

if (OldMO.isDef())
  assert(OldMO.isDef())

Which seems redundant. But I guess getting rid of the warning is fine.

This revision is now accepted and ready to land.Oct 15 2020, 3:53 AM
This revision was landed with ongoing or failed builds.Oct 15 2020, 3:56 AM
This revision was automatically updated to reflect the committed changes.
jmorse added a subscriber: jmorse.Oct 15 2020, 3:59 AM

Hrrmmm, that smells fishy; thanks for the warning sqelching, I'll follow it up.