Page MenuHomePhabricator

[FileCheck][NFC] Remove redundant DumpInputDefault
ClosedPublic

Authored by jdenny on Wed, Jun 24, 9:57 AM.

Details

Summary

I cannot find a reason to keep DumpInputDefault. Anyone else?

Diff Detail

Event Timeline

jdenny created this revision.Wed, Jun 24, 9:57 AM
Herald added a project: Restricted Project. · View Herald TranscriptWed, Jun 24, 9:57 AM
mehdi_amini accepted this revision.Wed, Jun 24, 8:30 PM
This revision is now accepted and ready to land.Wed, Jun 24, 8:30 PM

Any change of getting this with the context included? I'm guessing it's fine, but it's a little hard to check without.

jdenny updated this revision to Diff 273337.Thu, Jun 25, 6:39 AM

Ah, sorry I forgot to include context. Fixed.

jhenderson accepted this revision.Thu, Jun 25, 6:58 AM

LGTM too.

This revision was automatically updated to reflect the committed changes.