Update plugin to handle new style rules for virtual annotations.

tools/clang/plugins/tests/test.sh

 #!/bin/bash
 #
 # Copyright (c) 2011 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 #
 # Hacky, primitive testing: This runs the style plugin for a set of input files
 # and compares the output with golden result files.
 
 E_BADARGS=65
@@ skipping 15 matching lines @@
 do_testcase() {
   local flags=""
   if [ -e "${3}" ]; then
     flags="$(cat "${3}")"
   fi
 
   if [ "$(uname -s)" = "Darwin" ]; then
     flags="${flags} -isysroot $(xcrun --show-sdk-path) -stdlib=libstdc++"
   fi
 
-  local output="$("${CLANG_PATH}" -c -Wno-c++11-extensions \
+  local output="$("${CLANG_PATH}" -fsyntax-only -c -Wno-c++11-extensions \

[hans, 2014/09/26 01:29:29]

can probably drop the -c now that we pass -fsyntax-only

[dcheng, 2014/09/26 07:07:14]

Done.

       -Xclang -load -Xclang "${PLUGIN_PATH}" \
       -Xclang -add-plugin -Xclang find-bad-constructs ${flags} ${1} 2>&1)"
   local diffout="$(echo "${output}" | diff - "${2}")"
   if [ "${diffout}" = "" ]; then
     echo "PASS: ${1}"
   else
     failed_any_test=yes
     echo "FAIL: ${1}"
     echo "Output of compiler:"
     echo "${output}"
+    cat > ${2}-actual << EOF

[hans, 2014/09/26 01:29:29]

Won't this leave a file in the source tree? It seems handy for testing, though.

[dcheng, 2014/09/26 07:07:14]

Only if there's a diff. It makes it much easier to rebase things.

[hans, 2014/09/27 00:36:09]

Oh, right. That makes sense.

+${output}
+EOF
+
     echo "Expected output:"
     cat "${2}"
     echo
   fi
 }
 
 # Validate input to the script.
 if [[ -z "${1}" ]]; then
   usage
   exit ${E_BADARGS}
@@ skipping 23 matching lines @@
   do_testcase "${input}" "${input%cpp}txt" "${input%cpp}flags"
 done
 
 for input in *.c; do
   do_testcase "${input}" "${input%c}txt" "${input%c}flags"
 done
 
 if [[ "${failed_any_test}" ]]; then
   exit ${E_FAILEDTEST}
 fi