This is an archive of the discontinued LLVM Phabricator instance.

[analyzer] Fix 'Memory Error' bugtype capitalization.
ClosedPublic

Authored by NoQ on May 1 2017, 9:14 AM.

Details

Summary

It seems that we're not capitalizing every word in our bug type descriptions, however Memory Error is an exception from this rule. Additionally, one of the nullability checkers actually does spell it as Memory error. So i guess it's better to keep everything consistent.

Diff Detail

Event Timeline

NoQ created this revision.May 1 2017, 9:14 AM
xazax.hun accepted this revision.May 1 2017, 9:19 AM

LGTM!

This revision is now accepted and ready to land.May 1 2017, 9:19 AM
a.sidorin edited edge metadata.May 1 2017, 9:28 AM

Hi Artem. It is a good point but I think we should have a literal for this instead.

zaks.anna accepted this revision.May 1 2017, 9:31 AM

Thanks!

NoQ updated this revision to Diff 97574.May 3 2017, 2:08 AM

Added a reusable bug category.

a.sidorin accepted this revision.May 3 2017, 2:10 AM

Thank you!

xazax.hun accepted this revision.May 3 2017, 2:11 AM

Even better :)

This revision was automatically updated to reflect the committed changes.