This is an archive of the discontinued LLVM Phabricator instance.

Fix covered-switch-default warning in FormatManager.
ClosedPublic

Authored by brucem on Sep 22 2015, 7:30 PM.

Details

Summary

The default case doesn't need to be here as the switch covers
all possible values. If there's a new "lazy bool" value added
in the future, the compiler would start to warn about the new
case not being covered.

Diff Detail

Repository
rL LLVM

Event Timeline

brucem updated this revision to Diff 35454.Sep 22 2015, 7:30 PM
brucem retitled this revision from to Fix covered-switch-default warning in FormatManager..
brucem updated this object.
brucem added reviewers: granata.enrico, clayborg.
brucem added a subscriber: lldb-commits.
granata.enrico accepted this revision.Sep 22 2015, 7:32 PM
granata.enrico edited edge metadata.

LGTM

This revision is now accepted and ready to land.Sep 22 2015, 7:32 PM
This revision was automatically updated to reflect the committed changes.