thrift: fail compilation if an unexpected token is ----------------------------------------------------
Key: THRIFT-1274 URL: https://issues.apache.org/jira/browse/THRIFT-1274 Project: Thrift Issue Type: Improvement Components: C++ - Library Reporter: Dave Watson Priority: Minor Attachments: 0006-thrift-fail-compilation-if-an-unexpected-token-is-en.patch.diff >From e52b015b1c3d3b25ca1c905ea28cab80783a14bc Mon Sep 17 00:00:00 2001 From: Adam Simpkins <simpk...@fb.com> Date: Fri, 26 Feb 2010 04:48:55 +0000 Subject: [PATCH 06/33] thrift: fail compilation if an unexpected token is encountered Summary: Currently, if the thrift lexer encounters a token it does not expect, it prints the token it to stdout and continues. (This is the default behavior of flex when a token is unmatched.) This updates thriftl.ll to fail with an error message when it sees an unexpected character. Test Plan: [internal fb thing].thrift contains a stray single-quote. Previously it was getting printed to stdout whenever you compile this file, or anything that includes it. (Furthermore, it is printed out multiple times, since we scan the file twice each time it is included. Compiling [internal fb thing].thrift scans this file 8 times!) Now compiling this file fails with an error, as desired. I'll submit a separate diff to fix [internal fb thing].thrift. --- compiler/cpp/src/thriftl.ll | 9 +++++++++ 1 files changed, 9 insertions(+), 0 deletions(-) -- This message is automatically generated by JIRA. For more information on JIRA, see: http://www.atlassian.com/software/jira