More cleanup related to malformed and malbounded types.

More cleanup related to malformed and malbounded types.
Remove support for warnings in parser and related flags.
Rename flag --error-on-malformed-type to --error-on-bad-type, since it covers
both malformed and malbounded types (also closer to --error-on-bad-override).

R=hausner@google.com

Committed: https://code.google.com/p/dart/source/detail?r=26677

[hausner, 2013-08-26 22:17:33]

LGTM but see comment. I think we should keep Warning().

https://codereview.chromium.org/23465004/diff/1/runtime/vm/parser.cc
File runtime/vm/parser.cc (left):

https://codereview.chromium.org/23465004/diff/1/runtime/vm/parser.cc#oldcode7015
runtime/vm/parser.cc:7015: void Parser::Warning(intptr_t token_pos, const char*
format, ...) {
Can you please leave Warning() and friends in the Parser? It comes in handy when
we make language changes and want to issue a message when we see deprecated
code.

[regis, 2013-08-26 23:23:38]

Thanks!

https://codereview.chromium.org/23465004/diff/1/runtime/vm/parser.cc
File runtime/vm/parser.cc (left):

https://codereview.chromium.org/23465004/diff/1/runtime/vm/parser.cc#oldcode7015
runtime/vm/parser.cc:7015: void Parser::Warning(intptr_t token_pos, const char*
format, ...) {
On 2013/08/26 22:17:33, hausner wrote:
> Can you please leave Warning() and friends in the Parser? It comes in handy
when
> we make language changes and want to issue a message when we see deprecated
> code.

Done.

[regis, 2013-08-26 23:24:06]

Committed patchset #2 manually as r26677 (presubmit successful).