Merge "Remove ashmemd and related artifacts"
diff --git a/core/fuzz_test.mk b/core/fuzz_test.mk
index 71801ba..4a0fcfa2 100644
--- a/core/fuzz_test.mk
+++ b/core/fuzz_test.mk
@@ -67,12 +67,6 @@
LOCAL_MODULE_PATH_64 := $(TARGET_OUT_DATA_NATIVE_TESTS)/fuzzers/$(my_fuzzer)/$(LOCAL_MODULE)
LOCAL_MODULE_PATH_32 := $($(TARGET_2ND_ARCH_VAR_PREFIX)TARGET_OUT_DATA_NATIVE_TESTS)/fuzzers/$(my_fuzzer)/$(LOCAL_MODULE)
-ifndef LOCAL_MULTILIB
-ifndef LOCAL_32_BIT_ONLY
-LOCAL_MULTILIB := 64
-endif
-endif
-
ifndef LOCAL_STRIP_MODULE
LOCAL_STRIP_MODULE := keep_symbols
endif
diff --git a/core/main.mk b/core/main.mk
index 618594b..e20f669 100644
--- a/core/main.mk
+++ b/core/main.mk
@@ -1353,7 +1353,9 @@
$(eval whitelist_patterns := $(call resolve-product-relative-paths,$(whitelist))) \
$(eval files := $(call product-installed-files, $(makefile))) \
$(eval offending_files := $(filter-out $(path_patterns) $(whitelist_patterns) $(static_whitelist_patterns),$(files))) \
- $(call maybe-print-list-and-error,$(offending_files),$(makefile) produces files outside its artifact path requirement.) \
+ $(call maybe-print-list-and-error,$(offending_files),\
+ $(makefile) produces files outside its artifact path requirement. \
+ Allowed paths are $(subst $(space),$(comma)$(space),$(addsuffix *,$(requirements)))) \
$(eval unused_whitelist := $(filter-out $(files),$(whitelist_patterns))) \
$(call maybe-print-list-and-error,$(unused_whitelist),$(makefile) includes redundant whitelist entries in its artifact path requirement.) \
$(eval ### Optionally verify that nothing else produces files inside this artifact path requirement.) \
diff --git a/core/tasks/vndk.mk b/core/tasks/vndk.mk
index b487f53..3c4d942 100644
--- a/core/tasks/vndk.mk
+++ b/core/tasks/vndk.mk
@@ -48,7 +48,6 @@
vndk_snapshot_top := $(call intermediates-dir-for,PACKAGING,vndk-snapshot)
vndk_snapshot_out := $(vndk_snapshot_top)/vndk-snapshot
vndk_snapshot_soong_dir := $(call intermediates-dir-for,PACKAGING,vndk-snapshot-soong)
-vndk_snapshot_configs_out := $(vndk_snapshot_top)/configs
#######################################
# vndk_snapshot_zip
@@ -105,7 +104,6 @@
vndk_snapshot_out :=
vndk_snapshot_soong_dir :=
vndk_snapshot_soong_files :=
-vndk_snapshot_configs_out :=
vndk_snapshot_variant :=
else # BOARD_VNDK_RUNTIME_DISABLE is set to 'true'
diff --git a/envsetup.sh b/envsetup.sh
index 40f7705..92dad9a 100644
--- a/envsetup.sh
+++ b/envsetup.sh
@@ -575,6 +575,7 @@
function print_lunch_menu()
{
local uname=$(uname)
+ local choices=$(TARGET_BUILD_APPS= get_build_var COMMON_LUNCH_CHOICES)
echo
echo "You're building on" $uname
echo
@@ -582,7 +583,7 @@
local i=1
local choice
- for choice in $(TARGET_BUILD_APPS= get_build_var COMMON_LUNCH_CHOICES)
+ for choice in $choices
do
echo " $i. $choice"
i=$(($i+1))
diff --git a/target/board/generic_arm64/BoardConfig.mk b/target/board/generic_arm64/BoardConfig.mk
index ecc547f..f5caf70 100644
--- a/target/board/generic_arm64/BoardConfig.mk
+++ b/target/board/generic_arm64/BoardConfig.mk
@@ -67,7 +67,7 @@
# TODO(b/36764215): remove this setting when the generic system image
# no longer has QCOM-specific directories under /.
-BOARD_SEPOLICY_DIRS += build/make/target/board/generic_arm64_ab/sepolicy
+BOARD_SEPOLICY_DIRS += build/make/target/board/generic_arm64/sepolicy
# Wifi.
BOARD_WLAN_DEVICE := emulator
diff --git a/target/board/generic_arm64_ab/sepolicy/OWNERS b/target/board/generic_arm64/sepolicy/OWNERS
similarity index 100%
rename from target/board/generic_arm64_ab/sepolicy/OWNERS
rename to target/board/generic_arm64/sepolicy/OWNERS
diff --git a/target/board/generic_arm64_ab/sepolicy/file.te b/target/board/generic_arm64/sepolicy/file.te
similarity index 100%
rename from target/board/generic_arm64_ab/sepolicy/file.te
rename to target/board/generic_arm64/sepolicy/file.te
diff --git a/target/board/generic_arm64_ab/sepolicy/file_contexts b/target/board/generic_arm64/sepolicy/file_contexts
similarity index 100%
rename from target/board/generic_arm64_ab/sepolicy/file_contexts
rename to target/board/generic_arm64/sepolicy/file_contexts
diff --git a/target/board/generic_arm64_ab/BoardConfig.mk b/target/board/generic_arm64_ab/BoardConfig.mk
deleted file mode 100644
index 28140ce..0000000
--- a/target/board/generic_arm64_ab/BoardConfig.mk
+++ /dev/null
@@ -1,39 +0,0 @@
-#
-# Copyright (C) 2017 The Android Open-Source Project
-#
-# Licensed under the Apache License, Version 2.0 (the "License");
-# you may not use this file except in compliance with the License.
-# You may obtain a copy of the License at
-#
-# http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing, software
-# distributed under the License is distributed on an "AS IS" BASIS,
-# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-# See the License for the specific language governing permissions and
-# limitations under the License.
-#
-
-include build/make/target/board/BoardConfigGsiCommon.mk
-
-TARGET_ARCH := arm64
-TARGET_ARCH_VARIANT := armv8-a
-TARGET_CPU_ABI := arm64-v8a
-TARGET_CPU_ABI2 :=
-TARGET_CPU_VARIANT := generic
-
-TARGET_2ND_ARCH := arm
-TARGET_2ND_ARCH_VARIANT := armv8-a
-TARGET_2ND_CPU_ABI := armeabi-v7a
-TARGET_2ND_CPU_ABI2 := armeabi
-TARGET_2ND_CPU_VARIANT := generic
-
-# TODO(jiyong) These might be SoC specific.
-BOARD_ROOT_EXTRA_FOLDERS += firmware firmware/radio persist
-BOARD_ROOT_EXTRA_SYMLINKS += /vendor/lib/dsp:/dsp
-BOARD_ROOT_EXTRA_SYMLINKS += /vendor/firmware_mnt/image:/firmware/image
-BOARD_ROOT_EXTRA_SYMLINKS += /vendor/firmware_mnt/verinfo:/firmware/verinfo
-
-# TODO(b/36764215): remove this setting when the generic system image
-# no longer has QCOM-specific directories under /.
-BOARD_SEPOLICY_DIRS += build/make/target/board/generic_arm64_ab/sepolicy
diff --git a/target/board/generic_arm_ab/BoardConfig.mk b/target/board/generic_arm_ab/BoardConfig.mk
deleted file mode 100644
index bcb4cc5..0000000
--- a/target/board/generic_arm_ab/BoardConfig.mk
+++ /dev/null
@@ -1,36 +0,0 @@
-#
-# Copyright (C) 2017 The Android Open-Source Project
-#
-# Licensed under the Apache License, Version 2.0 (the "License");
-# you may not use this file except in compliance with the License.
-# You may obtain a copy of the License at
-#
-# http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing, software
-# distributed under the License is distributed on an "AS IS" BASIS,
-# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-# See the License for the specific language governing permissions and
-# limitations under the License.
-#
-
-include build/make/target/board/BoardConfigGsiCommon.mk
-
-TARGET_ARCH := arm
-TARGET_ARCH_VARIANT := armv7-a-neon
-TARGET_CPU_ABI := armeabi-v7a
-TARGET_CPU_ABI2 := armeabi
-TARGET_CPU_VARIANT := generic
-
-# Legacy GSI keeps 32 bits binder for 32 bits CPU Arch
-TARGET_USES_64_BIT_BINDER := false
-
-# TODO(jiyong) These might be SoC specific.
-BOARD_ROOT_EXTRA_FOLDERS += firmware firmware/radio persist
-BOARD_ROOT_EXTRA_SYMLINKS += /vendor/lib/dsp:/dsp
-BOARD_ROOT_EXTRA_SYMLINKS += /vendor/firmware_mnt/image:/firmware/image
-BOARD_ROOT_EXTRA_SYMLINKS += /vendor/firmware_mnt/verinfo:/firmware/verinfo
-
-# TODO(b/36764215): remove this setting when the generic system image
-# no longer has QCOM-specific directories under /.
-BOARD_SEPOLICY_DIRS += build/make/target/board/generic_arm64_ab/sepolicy
diff --git a/target/board/generic_x86_64_arm64/BoardConfig.mk b/target/board/generic_x86_64_arm64/BoardConfig.mk
new file mode 100755
index 0000000..f528294
--- /dev/null
+++ b/target/board/generic_x86_64_arm64/BoardConfig.mk
@@ -0,0 +1,59 @@
+# Copyright (C) 2018 The Android Open Source Project
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+
+# x86_64 emulator specific definitions
+TARGET_CPU_ABI := x86_64
+TARGET_ARCH := x86_64
+TARGET_ARCH_VARIANT := x86_64
+
+TARGET_2ND_CPU_ABI := x86
+TARGET_2ND_ARCH := x86
+TARGET_2ND_ARCH_VARIANT := x86_64
+
+TARGET_NATIVE_BRIDGE_ARCH := arm64
+TARGET_NATIVE_BRIDGE_ARCH_VARIANT := armv8-a
+TARGET_NATIVE_BRIDGE_CPU_VARIANT := generic
+TARGET_NATIVE_BRIDGE_ABI := arm64-v8a
+
+TARGET_NATIVE_BRIDGE_2ND_ARCH := arm
+TARGET_NATIVE_BRIDGE_2ND_ARCH_VARIANT := armv7-a-neon
+TARGET_NATIVE_BRIDGE_2ND_CPU_VARIANT := generic
+TARGET_NATIVE_BRIDGE_2ND_ABI := armeabi-v7a armeabi
+
+BUILD_BROKEN_DUP_RULES := true
+
+TARGET_PRELINK_MODULE := false
+
+include build/make/target/board/BoardConfigMainlineCommon.mk
+include build/make/target/board/BoardConfigEmuCommon.mk
+
+# the settings differ from BoardConfigMainlineCommon.mk
+BOARD_USES_SYSTEM_OTHER_ODEX :=
+
+# Resize to 4G to accommodate ASAN and CTS
+BOARD_USERDATAIMAGE_PARTITION_SIZE := 4294967296
+
+BOARD_SEPOLICY_DIRS += device/generic/goldfish/sepolicy/x86
+
+# Wifi.
+BOARD_WLAN_DEVICE := emulator
+BOARD_HOSTAPD_DRIVER := NL80211
+BOARD_WPA_SUPPLICANT_DRIVER := NL80211
+BOARD_HOSTAPD_PRIVATE_LIB := lib_driver_cmd_simulated
+BOARD_WPA_SUPPLICANT_PRIVATE_LIB := lib_driver_cmd_simulated
+WPA_SUPPLICANT_VERSION := VER_0_8_X
+WIFI_DRIVER_FW_PATH_PARAM := "/dev/null"
+WIFI_DRIVER_FW_PATH_STA := "/dev/null"
+WIFI_DRIVER_FW_PATH_AP := "/dev/null"
diff --git a/target/board/generic_x86_64_arm64/README.txt b/target/board/generic_x86_64_arm64/README.txt
new file mode 100644
index 0000000..48ee319
--- /dev/null
+++ b/target/board/generic_x86_64_arm64/README.txt
@@ -0,0 +1,10 @@
+The "generic_x86_64" product defines a non-hardware-specific IA target
+without a kernel or bootloader.
+
+It can be used to build the entire user-level system, and
+will work with the IA version of the emulator,
+
+It is not a product "base class"; no other products inherit
+from it or use it in any way.
+
+Third party arm64 to x86_64 translator has to be installed as well
diff --git a/target/board/generic_x86_64_ab/BoardConfig.mk b/target/board/generic_x86_64_arm64/device.mk
old mode 100644
new mode 100755
similarity index 66%
rename from target/board/generic_x86_64_ab/BoardConfig.mk
rename to target/board/generic_x86_64_arm64/device.mk
index 1dd5e48..fa1eb67
--- a/target/board/generic_x86_64_ab/BoardConfig.mk
+++ b/target/board/generic_x86_64_arm64/device.mk
@@ -1,5 +1,5 @@
#
-# Copyright (C) 2017 The Android Open-Source Project
+# Copyright (C) 2009 The Android Open Source Project
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
@@ -13,13 +13,3 @@
# See the License for the specific language governing permissions and
# limitations under the License.
#
-
-include build/make/target/board/BoardConfigGsiCommon.mk
-
-TARGET_CPU_ABI := x86_64
-TARGET_ARCH := x86_64
-TARGET_ARCH_VARIANT := x86_64
-
-TARGET_2ND_CPU_ABI := x86
-TARGET_2ND_ARCH := x86
-TARGET_2ND_ARCH_VARIANT := x86_64
diff --git a/target/board/generic_x86_64_arm64/system_ext.prop b/target/board/generic_x86_64_arm64/system_ext.prop
new file mode 100644
index 0000000..ed9d173
--- /dev/null
+++ b/target/board/generic_x86_64_arm64/system_ext.prop
@@ -0,0 +1,5 @@
+#
+# system.prop for generic sdk
+#
+
+rild.libpath=/vendor/lib64/libreference-ril.so
diff --git a/target/board/generic_x86_ab/BoardConfig.mk b/target/board/generic_x86_ab/BoardConfig.mk
deleted file mode 100644
index 53acffd..0000000
--- a/target/board/generic_x86_ab/BoardConfig.mk
+++ /dev/null
@@ -1,24 +0,0 @@
-#
-# Copyright (C) 2017 The Android Open-Source Project
-#
-# Licensed under the Apache License, Version 2.0 (the "License");
-# you may not use this file except in compliance with the License.
-# You may obtain a copy of the License at
-#
-# http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing, software
-# distributed under the License is distributed on an "AS IS" BASIS,
-# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-# See the License for the specific language governing permissions and
-# limitations under the License.
-#
-
-include build/make/target/board/BoardConfigGsiCommon.mk
-
-TARGET_CPU_ABI := x86
-TARGET_ARCH := x86
-TARGET_ARCH_VARIANT := x86
-
-# Legacy GSI keeps 32 bits binder for 32 bits CPU Arch
-TARGET_USES_64_BIT_BINDER := false
diff --git a/target/board/generic_x86_arm/BoardConfig.mk b/target/board/generic_x86_arm/BoardConfig.mk
index 3fbf79b..f6589b0 100644
--- a/target/board/generic_x86_arm/BoardConfig.mk
+++ b/target/board/generic_x86_arm/BoardConfig.mk
@@ -34,8 +34,6 @@
# the settings differ from BoardConfigMainlineCommon.mk
BOARD_USES_SYSTEM_OTHER_ODEX :=
-BOARD_CACHEIMAGE_FILE_SYSTEM_TYPE := ext4
-BOARD_CACHEIMAGE_PARTITION_SIZE := 16777216
# Resize to 4G to accomodate ASAN and CTS
BOARD_USERDATAIMAGE_PARTITION_SIZE := 4294967296
diff --git a/target/board/gsi_arm64/BoardConfig.mk b/target/board/gsi_arm64/BoardConfig.mk
index 571d623..db6f3f0 100644
--- a/target/board/gsi_arm64/BoardConfig.mk
+++ b/target/board/gsi_arm64/BoardConfig.mk
@@ -34,4 +34,4 @@
# TODO(b/36764215): remove this setting when the generic system image
# no longer has QCOM-specific directories under /.
-BOARD_SEPOLICY_DIRS += build/make/target/board/generic_arm64_ab/sepolicy
+BOARD_SEPOLICY_DIRS += build/make/target/board/generic_arm64/sepolicy
diff --git a/target/product/AndroidProducts.mk b/target/product/AndroidProducts.mk
index cfb8930..174916d 100644
--- a/target/product/AndroidProducts.mk
+++ b/target/product/AndroidProducts.mk
@@ -43,13 +43,9 @@
else
PRODUCT_MAKEFILES := \
- $(LOCAL_DIR)/aosp_arm64_ab.mk \
$(LOCAL_DIR)/aosp_arm64.mk \
- $(LOCAL_DIR)/aosp_arm_ab.mk \
$(LOCAL_DIR)/aosp_arm.mk \
- $(LOCAL_DIR)/aosp_x86_64_ab.mk \
$(LOCAL_DIR)/aosp_x86_64.mk \
- $(LOCAL_DIR)/aosp_x86_ab.mk \
$(LOCAL_DIR)/aosp_x86_arm.mk \
$(LOCAL_DIR)/aosp_x86.mk \
$(LOCAL_DIR)/full.mk \
diff --git a/target/product/aosp_arm64_ab.mk b/target/product/aosp_arm64_ab.mk
deleted file mode 100644
index f707a39..0000000
--- a/target/product/aosp_arm64_ab.mk
+++ /dev/null
@@ -1,50 +0,0 @@
-#
-# Copyright (C) 2017 The Android Open-Source Project
-#
-# Licensed under the Apache License, Version 2.0 (the "License");
-# you may not use this file except in compliance with the License.
-# You may obtain a copy of the License at
-#
-# http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing, software
-# distributed under the License is distributed on an "AS IS" BASIS,
-# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-# See the License for the specific language governing permissions and
-# limitations under the License.
-#
-
-# PRODUCT_PROPERTY_OVERRIDES cannot be used here because sysprops will be at
-# /vendor/[build|default].prop when build split is on. In order to have sysprops
-# on the generic system image, place them in build/make/target/board/
-# gsi_system.prop.
-
-# aosp_arm64_ab-userdebug is a Legacy GSI for the devices with:
-# - ARM 64 bits user space
-# - 64 bits binder interface
-# - system-as-root
-
-$(call inherit-product, $(SRC_TARGET_DIR)/product/core_64_bit.mk)
-$(call inherit-product, $(SRC_TARGET_DIR)/product/legacy_gsi_common.mk)
-
-# Enable mainline checking for excat this product name
-ifeq (aosp_arm64_ab,$(TARGET_PRODUCT))
-PRODUCT_ENFORCE_ARTIFACT_PATH_REQUIREMENTS := relaxed
-endif
-
-PRODUCT_ARTIFACT_PATH_REQUIREMENT_WHITELIST += \
- root/init.zygote32_64.rc \
- root/init.zygote64_32.rc \
-
-# Copy different zygote settings for vendor.img to select by setting property
-# ro.zygote=zygote64_32 or ro.zygote=zygote32_64:
-# 1. 64-bit primary, 32-bit secondary OR
-# 2. 32-bit primary, 64-bit secondary
-# init.zygote64_32.rc is in the core_64_bit.mk below
-PRODUCT_COPY_FILES += \
- system/core/rootdir/init.zygote32_64.rc:root/init.zygote32_64.rc
-
-PRODUCT_NAME := aosp_arm64_ab
-PRODUCT_DEVICE := generic_arm64_ab
-PRODUCT_BRAND := Android
-PRODUCT_MODEL := AOSP on ARM64
diff --git a/target/product/aosp_arm_ab.mk b/target/product/aosp_arm_ab.mk
deleted file mode 100644
index e760932..0000000
--- a/target/product/aosp_arm_ab.mk
+++ /dev/null
@@ -1,37 +0,0 @@
-#
-# Copyright (C) 2017 The Android Open-Source Project
-#
-# Licensed under the Apache License, Version 2.0 (the "License");
-# you may not use this file except in compliance with the License.
-# You may obtain a copy of the License at
-#
-# http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing, software
-# distributed under the License is distributed on an "AS IS" BASIS,
-# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-# See the License for the specific language governing permissions and
-# limitations under the License.
-#
-
-# PRODUCT_PROPERTY_OVERRIDES cannot be used here because sysprops will be at
-# /vendor/[build|default].prop when build split is on. In order to have sysprops
-# on the generic system image, place them in build/make/target/board/
-# gsi_system.prop.
-
-# aosp_arm_ab-userdebug is a Legacy GSI for the devices with:
-# - ARM 32 bits user space
-# - 32 bits binder interface
-# - system-as-root
-
-$(call inherit-product, $(SRC_TARGET_DIR)/product/legacy_gsi_common.mk)
-
-# Enable mainline checking for excat this product name
-ifeq (aosp_arm_ab,$(TARGET_PRODUCT))
-PRODUCT_ENFORCE_ARTIFACT_PATH_REQUIREMENTS := relaxed
-endif
-
-PRODUCT_NAME := aosp_arm_ab
-PRODUCT_DEVICE := generic_arm_ab
-PRODUCT_BRAND := Android
-PRODUCT_MODEL := AOSP on ARM32
diff --git a/target/product/aosp_product.mk b/target/product/aosp_product.mk
index cda977b..8c87983 100644
--- a/target/product/aosp_product.mk
+++ b/target/product/aosp_product.mk
@@ -40,6 +40,7 @@
PRODUCT_PRODUCT_PROPERTIES += \
ro.config.ringtone=Ring_Synth_04.ogg \
ro.config.notification_sound=pixiedust.ogg \
+ ro.com.android.dataroaming=true \
# More AOSP packages
PRODUCT_PACKAGES += \
diff --git a/target/product/aosp_x86_64_ab.mk b/target/product/aosp_x86_64_ab.mk
deleted file mode 100644
index dfb7b49..0000000
--- a/target/product/aosp_x86_64_ab.mk
+++ /dev/null
@@ -1,50 +0,0 @@
-#
-# Copyright (C) 2017 The Android Open-Source Project
-#
-# Licensed under the Apache License, Version 2.0 (the "License");
-# you may not use this file except in compliance with the License.
-# You may obtain a copy of the License at
-#
-# http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing, software
-# distributed under the License is distributed on an "AS IS" BASIS,
-# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-# See the License for the specific language governing permissions and
-# limitations under the License.
-#
-
-# PRODUCT_PROPERTY_OVERRIDES cannot be used here because sysprops will be at
-# /vendor/[build|default].prop when build split is on. In order to have sysprops
-# on the generic system image, place them in build/make/target/board/
-# gsi_system.prop.
-
-# aosp_x86_64_ab-userdebug is a Legacy GSI for the devices with:
-# - x86 64 bits user space
-# - 64 bits binder interface
-# - system-as-root
-
-$(call inherit-product, $(SRC_TARGET_DIR)/product/core_64_bit.mk)
-$(call inherit-product, $(SRC_TARGET_DIR)/product/legacy_gsi_common.mk)
-
-# Enable mainline checking for excat this product name
-ifeq (aosp_x86_64_ab,$(TARGET_PRODUCT))
-PRODUCT_ENFORCE_ARTIFACT_PATH_REQUIREMENTS := relaxed
-endif
-
-PRODUCT_ARTIFACT_PATH_REQUIREMENT_WHITELIST += \
- root/init.zygote32_64.rc \
- root/init.zygote64_32.rc \
-
-# Copy different zygote settings for vendor.img to select by setting property
-# ro.zygote=zygote64_32 or ro.zygote=zygote32_64:
-# 1. 64-bit primary, 32-bit secondary OR
-# 2. 32-bit primary, 64-bit secondary
-# init.zygote64_32.rc is in the core_64_bit.mk below
-PRODUCT_COPY_FILES += \
- system/core/rootdir/init.zygote32_64.rc:root/init.zygote32_64.rc
-
-PRODUCT_NAME := aosp_x86_64_ab
-PRODUCT_DEVICE := generic_x86_64_ab
-PRODUCT_BRAND := Android
-PRODUCT_MODEL := AOSP on x86_64
diff --git a/target/product/aosp_x86_ab.mk b/target/product/aosp_x86_ab.mk
deleted file mode 100644
index d07351c..0000000
--- a/target/product/aosp_x86_ab.mk
+++ /dev/null
@@ -1,37 +0,0 @@
-#
-# Copyright (C) 2017 The Android Open-Source Project
-#
-# Licensed under the Apache License, Version 2.0 (the "License");
-# you may not use this file except in compliance with the License.
-# You may obtain a copy of the License at
-#
-# http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing, software
-# distributed under the License is distributed on an "AS IS" BASIS,
-# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-# See the License for the specific language governing permissions and
-# limitations under the License.
-#
-
-# PRODUCT_PROPERTY_OVERRIDES cannot be used here because sysprops will be at
-# /vendor/[build|default].prop when build split is on. In order to have sysprops
-# on the generic system image, place them in build/make/target/board/
-# gsi_system.prop.
-
-# aosp_x86_ab-userdebug is a Legacy GSI for the devices with:
-# - x86 32 bits user space
-# - 32 bits binder interface
-# - system-as-root
-
-$(call inherit-product, $(SRC_TARGET_DIR)/product/legacy_gsi_common.mk)
-
-# Enable mainline checking for excat this product name
-ifeq (aosp_x86_ab,$(TARGET_PRODUCT))
-PRODUCT_ENFORCE_ARTIFACT_PATH_REQUIREMENTS := relaxed
-endif
-
-PRODUCT_NAME := aosp_x86_ab
-PRODUCT_DEVICE := generic_x86_ab
-PRODUCT_BRAND := Android
-PRODUCT_MODEL := AOSP on x86
diff --git a/target/product/base_system.mk b/target/product/base_system.mk
index 721835c..4e33d23 100644
--- a/target/product/base_system.mk
+++ b/target/product/base_system.mk
@@ -19,9 +19,6 @@
abb \
adbd \
am \
- android.hardware.neuralnetworks@1.0 \
- android.hardware.neuralnetworks@1.1 \
- android.hardware.neuralnetworks@1.2 \
android.hidl.allocator@1.0-service \
android.hidl.base-V1.0-java \
android.hidl.manager-V1.0-java \
@@ -159,6 +156,7 @@
libnetd_client \
libnetlink \
libnetutils \
+ libneuralnetworks_packageinfo \
libOpenMAXAL \
libOpenSLES \
libpdfium \
@@ -186,7 +184,6 @@
libui \
libusbhost \
libutils \
- libvorbisidec \
libvulkan \
libwifi-service \
libwilhelm \
@@ -248,6 +245,7 @@
Shell \
shell_and_utilities_system \
sm \
+ snapshotctl \
statsd \
storaged \
surfaceflinger \
diff --git a/target/product/emulator.mk b/target/product/emulator.mk
index 322eab4..7ff01cd 100644
--- a/target/product/emulator.mk
+++ b/target/product/emulator.mk
@@ -53,7 +53,9 @@
#config.disable_location=true
# Enable Perfetto traced
-PRODUCT_SYSTEM_EXT_PROPERTIES += \
+# There is a stable property API for this prop so we can move it to /product.
+# https://android-review.googlesource.com/c/platform/system/libsysprop/+/952375
+PRODUCT_PRODUCT_PROPERTIES += \
persist.traced.enable=1
# enable Google-specific location features,
diff --git a/target/product/emulator_vendor.mk b/target/product/emulator_vendor.mk
index 9c3be8e..e67124e 100644
--- a/target/product/emulator_vendor.mk
+++ b/target/product/emulator_vendor.mk
@@ -58,7 +58,9 @@
#config.disable_location=true
# Enable Perfetto traced
-PRODUCT_SYSTEM_EXT_PROPERTIES += \
+# There is a stable property API for this prop so we can move it to /product.
+# https://android-review.googlesource.com/c/platform/system/libsysprop/+/952375
+PRODUCT_PRODUCT_PROPERTIES += \
persist.traced.enable=1
# enable Google-specific location features,
diff --git a/target/product/gsi_common.mk b/target/product/gsi_common.mk
deleted file mode 100644
index bfecc95..0000000
--- a/target/product/gsi_common.mk
+++ /dev/null
@@ -1,91 +0,0 @@
-#
-# Copyright (C) 2019 The Android Open-Source Project
-#
-# Licensed under the Apache License, Version 2.0 (the "License");
-# you may not use this file except in compliance with the License.
-# You may obtain a copy of the License at
-#
-# http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing, software
-# distributed under the License is distributed on an "AS IS" BASIS,
-# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-# See the License for the specific language governing permissions and
-# limitations under the License.
-#
-
-$(call inherit-product, $(SRC_TARGET_DIR)/product/mainline_system.mk)
-
-# GSI includes all AOSP product packages and placed under /system/product
-$(call inherit-product, $(SRC_TARGET_DIR)/product/handheld_product.mk)
-$(call inherit-product, $(SRC_TARGET_DIR)/product/telephony_product.mk)
-
-# Default AOSP sounds
-$(call inherit-product-if-exists, frameworks/base/data/sounds/AllAudio.mk)
-
-# GSI doesn't support apex for now.
-# Properties set in product take precedence over those in vendor.
-PRODUCT_PRODUCT_PROPERTIES += \
- ro.apex.updatable=false
-
-# Additional settings used in all AOSP builds
-PRODUCT_PRODUCT_PROPERTIES += \
- ro.config.ringtone=Ring_Synth_04.ogg \
- ro.config.notification_sound=pixiedust.ogg \
-
-# The mainline checking whitelist, should be clean up
-PRODUCT_ARTIFACT_PATH_REQUIREMENT_WHITELIST += \
- system/app/messaging/messaging.apk \
- system/app/WAPPushManager/WAPPushManager.apk \
- system/bin/healthd \
- system/etc/init/healthd.rc \
- system/etc/seccomp_policy/crash_dump.%.policy \
- system/etc/seccomp_policy/mediacodec.policy \
- system/etc/vintf/manifest/manifest_healthd.xml \
- system/lib/libframesequence.so \
- system/lib/libgiftranscode.so \
- system/lib64/libframesequence.so \
- system/lib64/libgiftranscode.so \
-
-# Some GSI builds enable dexpreopt, whitelist these preopt files
-PRODUCT_ARTIFACT_PATH_REQUIREMENT_WHITELIST += %.odex %.vdex %.art
-
-# Exclude all files under system/product and system/system_ext
-PRODUCT_ARTIFACT_PATH_REQUIREMENT_WHITELIST += \
- system/product/% \
- system/system_ext/%
-
-
-# Split selinux policy
-PRODUCT_FULL_TREBLE_OVERRIDE := true
-
-# Enable dynamic partition size
-PRODUCT_USE_DYNAMIC_PARTITION_SIZE := true
-
-# Needed by Pi newly launched device to pass VtsTrebleSysProp on GSI
-PRODUCT_COMPATIBLE_PROPERTY_OVERRIDE := true
-
-# GSI specific tasks on boot
-PRODUCT_PACKAGES += \
- gsi_skip_mount.cfg \
- init.gsi.rc
-
-# Support addtional P vendor interface
-PRODUCT_EXTRA_VNDK_VERSIONS := 28
-
-# More AOSP packages
-PRODUCT_PACKAGES += \
- messaging \
- PhotoTable \
- WAPPushManager \
- WallpaperPicker \
-
-# Telephony:
-# Provide a APN configuration to GSI product
-PRODUCT_COPY_FILES += \
- device/sample/etc/apns-full-conf.xml:$(TARGET_COPY_OUT_PRODUCT)/etc/apns-conf.xml
-
-# NFC:
-# Provide a libnfc-nci.conf to GSI product
-PRODUCT_COPY_FILES += \
- device/generic/common/nfc/libnfc-nci.conf:$(TARGET_COPY_OUT_PRODUCT)/etc/libnfc-nci.conf
diff --git a/target/product/legacy_gsi_common.mk b/target/product/legacy_gsi_common.mk
deleted file mode 100644
index fdae6eb..0000000
--- a/target/product/legacy_gsi_common.mk
+++ /dev/null
@@ -1,37 +0,0 @@
-#
-# Copyright (C) 2019 The Android Open-Source Project
-#
-# Licensed under the Apache License, Version 2.0 (the "License");
-# you may not use this file except in compliance with the License.
-# You may obtain a copy of the License at
-#
-# http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing, software
-# distributed under the License is distributed on an "AS IS" BASIS,
-# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-# See the License for the specific language governing permissions and
-# limitations under the License.
-#
-
-include $(SRC_TARGET_DIR)/product/gsi_common.mk
-
-PRODUCT_ARTIFACT_PATH_REQUIREMENT_WHITELIST += \
- system/etc/init/init.legacy-gsi.rc \
- system/etc/init/gsi/init.vndk-27.rc \
- system/etc/ld.config.vndk_lite.txt \
-
-# Legacy GSI support addtional O-MR1 interface
-PRODUCT_EXTRA_VNDK_VERSIONS += 27
-
-# Support for the O-MR1 devices
-PRODUCT_COPY_FILES += \
- build/make/target/product/gsi/init.legacy-gsi.rc:system/etc/init/init.legacy-gsi.rc \
- build/make/target/product/gsi/init.vndk-27.rc:system/etc/init/gsi/init.vndk-27.rc
-
-# Name space configuration file for non-enforcing VNDK
-PRODUCT_PACKAGES += \
- ld.config.vndk_lite.txt
-
-# Legacy GSI relax the compatible property checking
-PRODUCT_COMPATIBLE_PROPERTY_OVERRIDE := false
diff --git a/tools/releasetools/apex_utils.py b/tools/releasetools/apex_utils.py
index 18ad8ce..ee3c463 100644
--- a/tools/releasetools/apex_utils.py
+++ b/tools/releasetools/apex_utils.py
@@ -42,7 +42,7 @@
def SignApexPayload(avbtool, payload_file, payload_key_path, payload_key_name,
- algorithm, salt, signing_args=None):
+ algorithm, salt, no_hashtree, signing_args=None):
"""Signs a given payload_file with the payload key."""
# Add the new footer. Old footer, if any, will be replaced by avbtool.
cmd = [avbtool, 'add_hashtree_footer',
@@ -52,6 +52,8 @@
'--prop', 'apex.key:{}'.format(payload_key_name),
'--image', payload_file,
'--salt', salt]
+ if no_hashtree:
+ cmd.append('--no_hashtree')
if signing_args:
cmd.extend(shlex.split(signing_args))
@@ -64,13 +66,15 @@
# Verify the signed payload image with specified public key.
logger.info('Verifying %s', payload_file)
- VerifyApexPayload(avbtool, payload_file, payload_key_path)
+ VerifyApexPayload(avbtool, payload_file, payload_key_path, no_hashtree)
-def VerifyApexPayload(avbtool, payload_file, payload_key):
+def VerifyApexPayload(avbtool, payload_file, payload_key, no_hashtree=False):
"""Verifies the APEX payload signature with the given key."""
cmd = [avbtool, 'verify_image', '--image', payload_file,
'--key', payload_key]
+ if no_hashtree:
+ cmd.append('--accept_zeroed_hashtree')
try:
common.RunAndCheckOutput(cmd)
except common.ExternalError as e:
@@ -91,7 +95,7 @@
Returns:
A dict that contains payload property-value pairs. The dict should at least
- contain Algorithm, Salt and apex.key.
+ contain Algorithm, Salt, Tree Size and apex.key.
"""
if not os.path.exists(payload_path):
raise ApexInfoError('Failed to find image: {}'.format(payload_path))
@@ -104,11 +108,11 @@
'Failed to get APEX payload info for {}:\n{}'.format(
payload_path, e))
- # Extract the Algorithm / Salt / Prop info from payload (i.e. an image signed
- # with avbtool). For example,
+ # Extract the Algorithm / Salt / Prop info / Tree size from payload (i.e. an
+ # image signed with avbtool). For example,
# Algorithm: SHA256_RSA4096
PAYLOAD_INFO_PATTERN = (
- r'^\s*(?P<key>Algorithm|Salt|Prop)\:\s*(?P<value>.*?)$')
+ r'^\s*(?P<key>Algorithm|Salt|Prop|Tree Size)\:\s*(?P<value>.*?)$')
payload_info_matcher = re.compile(PAYLOAD_INFO_PATTERN)
payload_info = {}
@@ -151,7 +155,7 @@
def SignApex(avbtool, apex_data, payload_key, container_key, container_pw,
- codename_to_api_level_map, signing_args=None):
+ codename_to_api_level_map, no_hashtree, signing_args=None):
"""Signs the current APEX with the given payload/container keys.
Args:
@@ -160,6 +164,7 @@
container_key: The path to container signing key (w/o extension).
container_pw: The matching password of the container_key, or None.
codename_to_api_level_map: A dict that maps from codename to API level.
+ no_hashtree: Don't include hashtree in the signed APEX.
signing_args: Additional args to be passed to the payload signer.
Returns:
@@ -187,6 +192,7 @@
payload_info['apex.key'],
payload_info['Algorithm'],
payload_info['Salt'],
+ no_hashtree,
signing_args)
# 1b. Update the embedded payload public key.
diff --git a/tools/releasetools/sign_apex.py b/tools/releasetools/sign_apex.py
index 2516e15..f2daa46 100755
--- a/tools/releasetools/sign_apex.py
+++ b/tools/releasetools/sign_apex.py
@@ -56,6 +56,7 @@
container_key=container_key,
container_pw=None,
codename_to_api_level_map=None,
+ no_hashtree=False,
signing_args=signing_args)
@@ -103,7 +104,8 @@
args[0],
options['payload_key'],
options['container_key'],
- options.get('payload_extra_args'))
+ no_hashtree=False,
+ signing_args=options.get('payload_extra_args'))
shutil.copyfile(signed_apex, args[1])
logger.info("done.")
diff --git a/tools/releasetools/sign_target_files_apks.py b/tools/releasetools/sign_target_files_apks.py
index 1f41431..710147b 100755
--- a/tools/releasetools/sign_target_files_apks.py
+++ b/tools/releasetools/sign_target_files_apks.py
@@ -479,7 +479,8 @@
container_key,
key_passwords[container_key],
codename_to_api_level_map,
- OPTIONS.avb_extra_args.get('apex'))
+ no_hashtree=True,
+ signing_args=OPTIONS.avb_extra_args.get('apex'))
common.ZipWrite(output_tf_zip, signed_apex, filename)
else:
diff --git a/tools/releasetools/test_apex_utils.py b/tools/releasetools/test_apex_utils.py
index e9c26f0..5d4cc77 100644
--- a/tools/releasetools/test_apex_utils.py
+++ b/tools/releasetools/test_apex_utils.py
@@ -44,19 +44,42 @@
payload_file = self._GetTestPayload()
apex_utils.SignApexPayload(
'avbtool', payload_file, self.payload_key, 'testkey', 'SHA256_RSA2048',
- self.SALT)
+ self.SALT, no_hashtree=True)
payload_info = apex_utils.ParseApexPayloadInfo('avbtool', payload_file)
self.assertEqual('SHA256_RSA2048', payload_info['Algorithm'])
self.assertEqual(self.SALT, payload_info['Salt'])
self.assertEqual('testkey', payload_info['apex.key'])
+ self.assertEqual('0 bytes', payload_info['Tree Size'])
@test_utils.SkipIfExternalToolsUnavailable()
def test_SignApexPayload(self):
payload_file = self._GetTestPayload()
apex_utils.SignApexPayload(
'avbtool', payload_file, self.payload_key, 'testkey', 'SHA256_RSA2048',
- self.SALT)
+ self.SALT, no_hashtree=True)
+ apex_utils.VerifyApexPayload(
+ 'avbtool', payload_file, self.payload_key, True)
+
+ @test_utils.SkipIfExternalToolsUnavailable()
+ def test_SignApexPayload_withHashtree(self):
+ payload_file = self._GetTestPayload()
+ apex_utils.SignApexPayload(
+ 'avbtool', payload_file, self.payload_key, 'testkey', 'SHA256_RSA2048',
+ self.SALT, no_hashtree=False)
apex_utils.VerifyApexPayload('avbtool', payload_file, self.payload_key)
+ payload_info = apex_utils.ParseApexPayloadInfo('avbtool', payload_file)
+ self.assertEqual('4096 bytes', payload_info['Tree Size'])
+
+ @test_utils.SkipIfExternalToolsUnavailable()
+ def test_SignApexPayload_noHashtree(self):
+ payload_file = self._GetTestPayload()
+ apex_utils.SignApexPayload(
+ 'avbtool', payload_file, self.payload_key, 'testkey', 'SHA256_RSA2048',
+ self.SALT, no_hashtree=True)
+ apex_utils.VerifyApexPayload('avbtool', payload_file, self.payload_key,
+ no_hashtree=True)
+ payload_info = apex_utils.ParseApexPayloadInfo('avbtool', payload_file)
+ self.assertEqual('0 bytes', payload_info['Tree Size'])
@test_utils.SkipIfExternalToolsUnavailable()
def test_SignApexPayload_withSignerHelper(self):
@@ -70,8 +93,10 @@
payload_file,
self.payload_key,
'testkey', 'SHA256_RSA2048', self.SALT,
+ True,
payload_signer_args)
- apex_utils.VerifyApexPayload('avbtool', payload_file, self.payload_key)
+ apex_utils.VerifyApexPayload(
+ 'avbtool', payload_file, self.payload_key, True)
@test_utils.SkipIfExternalToolsUnavailable()
def test_SignApexPayload_invalidKey(self):
@@ -83,18 +108,21 @@
os.path.join(self.testdata_dir, 'testkey.x509.pem'),
'testkey',
'SHA256_RSA2048',
- self.SALT)
+ self.SALT,
+ no_hashtree=True)
@test_utils.SkipIfExternalToolsUnavailable()
def test_VerifyApexPayload_wrongKey(self):
payload_file = self._GetTestPayload()
apex_utils.SignApexPayload(
'avbtool', payload_file, self.payload_key, 'testkey', 'SHA256_RSA2048',
- self.SALT)
- apex_utils.VerifyApexPayload('avbtool', payload_file, self.payload_key)
+ self.SALT, True)
+ apex_utils.VerifyApexPayload(
+ 'avbtool', payload_file, self.payload_key, True)
self.assertRaises(
apex_utils.ApexSigningError,
apex_utils.VerifyApexPayload,
'avbtool',
payload_file,
- os.path.join(self.testdata_dir, 'testkey_with_passwd.key'))
+ os.path.join(self.testdata_dir, 'testkey_with_passwd.key'),
+ no_hashtree=True)
diff --git a/tools/signapk/Android.bp b/tools/signapk/Android.bp
index ad9d957..c799dbf 100644
--- a/tools/signapk/Android.bp
+++ b/tools/signapk/Android.bp
@@ -24,7 +24,7 @@
"apksig",
"bouncycastle-unbundled",
"bouncycastle-bcpkix-unbundled",
- "conscrypt-nojarjar",
+ "conscrypt-unbundled",
],
required: ["libconscrypt_openjdk_jni"],
diff --git a/tools/warn.py b/tools/warn.py
index 48feb49..f91c811 100755
--- a/tools/warn.py
+++ b/tools/warn.py
@@ -159,11 +159,64 @@
return tidy_warn_pattern(description, description + r'-.+')
+def analyzer_high(description, patterns):
+ # Important clang analyzer warnings to be fixed ASAP.
+ return {
+ 'category': 'C/C++',
+ 'severity': Severity.HIGH,
+ 'description': description,
+ 'patterns': patterns
+ }
+
+
+def analyzer_high_check(check):
+ return analyzer_high(check, [r'.*: .+\[' + check + r'\]$'])
+
+
+def analyzer_group_high(check):
+ return analyzer_high(check, [r'.*: .+\[' + check + r'.+\]$'])
+
+
+def analyzer_warn(description, patterns):
+ return {
+ 'category': 'C/C++',
+ 'severity': Severity.ANALYZER,
+ 'description': description,
+ 'patterns': patterns
+ }
+
+
+def analyzer_warn_check(check):
+ return analyzer_warn(check, [r'.*: .+\[' + check + r'\]$'])
+
+
+def analyzer_group_check(check):
+ return analyzer_warn(check, [r'.*: .+\[' + check + r'.+\]$'])
+
+
+def java_warn(severity, description, patterns):
+ return {
+ 'category': 'Java',
+ 'severity': severity,
+ 'description': 'Java: ' + description,
+ 'patterns': patterns
+ }
+
+
+def java_high(description, patterns):
+ return java_warn(Severity.HIGH, description, patterns)
+
+
+def java_medium(description, patterns):
+ return java_warn(Severity.MEDIUM, description, patterns)
+
+
+def java_low(description, patterns):
+ return java_warn(Severity.LOW, description, patterns)
+
+
warn_patterns = [
# pylint:disable=line-too-long,g-inconsistent-quotes
- {'category': 'C/C++', 'severity': Severity.ANALYZER,
- 'description': 'clang-analyzer Security warning',
- 'patterns': [r".*: warning: .+\[clang-analyzer-security.*\]"]},
{'category': 'make', 'severity': Severity.MEDIUM,
'description': 'make: overriding commands/ignoring old commands',
'patterns': [r".*: warning: overriding commands for target .+",
@@ -183,7 +236,7 @@
{'category': 'make', 'severity': Severity.MEDIUM,
'description': 'Duplicate header copy',
'patterns': [r".*: warning: Duplicate header copy: .+"]},
- {'category': 'C/C++', 'severity': Severity.HIGH, 'option': '-Wimplicit-function-declaration',
+ {'category': 'C/C++', 'severity': Severity.MEDIUM, 'option': '-Wimplicit-function-declaration',
'description': 'Implicit function declaration',
'patterns': [r".*: warning: implicit declaration of function .+",
r".*: warning: implicitly declaring library function"]},
@@ -195,17 +248,17 @@
'patterns': [r".*: warning: comparison is always .+ due to limited range of data type",
r".*: warning: comparison of unsigned .*expression .+ is always true",
r".*: warning: comparison of unsigned .*expression .+ is always false"]},
- {'category': 'C/C++', 'severity': Severity.HIGH,
- 'description': 'Potential leak of memory, bad free, use after free',
- 'patterns': [r".*: warning: Potential leak of memory",
- r".*: warning: Potential memory leak",
- r".*: warning: Memory allocated by alloca\(\) should not be deallocated",
- r".*: warning: Memory allocated by .+ should be deallocated by .+ not .+",
- r".*: warning: 'delete' applied to a pointer that was allocated",
- r".*: warning: Use of memory after it is freed",
- r".*: warning: Argument to .+ is the address of .+ variable",
- r".*: warning: Argument to free\(\) is offset by .+ of memory allocated by",
- r".*: warning: Attempt to .+ released memory"]},
+ # {'category': 'C/C++', 'severity': Severity.HIGH,
+ # 'description': 'Potential leak of memory, bad free, use after free',
+ # 'patterns': [r".*: warning: Potential leak of memory",
+ # r".*: warning: Potential memory leak",
+ # r".*: warning: Memory allocated by alloca\(\) should not be deallocated",
+ # r".*: warning: Memory allocated by .+ should be deallocated by .+ not .+",
+ # r".*: warning: 'delete' applied to a pointer that was allocated",
+ # r".*: warning: Use of memory after it is freed",
+ # r".*: warning: Argument to .+ is the address of .+ variable",
+ # r".*: warning: Argument to free\(\) is offset by .+ of memory allocated by",
+ # r".*: warning: Attempt to .+ released memory"]},
{'category': 'C/C++', 'severity': Severity.HIGH,
'description': 'Use transient memory for control value',
'patterns': [r".*: warning: .+Using such transient memory for the control value is .*dangerous."]},
@@ -213,10 +266,10 @@
'description': 'Return address of stack memory',
'patterns': [r".*: warning: Address of stack memory .+ returned to caller",
r".*: warning: Address of stack memory .+ will be a dangling reference"]},
- {'category': 'C/C++', 'severity': Severity.HIGH,
- 'description': 'Problem with vfork',
- 'patterns': [r".*: warning: This .+ is prohibited after a successful vfork",
- r".*: warning: Call to function '.+' is insecure "]},
+ # {'category': 'C/C++', 'severity': Severity.HIGH,
+ # 'description': 'Problem with vfork',
+ # 'patterns': [r".*: warning: This .+ is prohibited after a successful vfork",
+ # r".*: warning: Call to function '.+' is insecure "]},
{'category': 'C/C++', 'severity': Severity.HIGH, 'option': 'infinite-recursion',
'description': 'Infinite recursion',
'patterns': [r".*: warning: all paths through this function will call itself"]},
@@ -271,6 +324,9 @@
{'category': 'C/C++', 'severity': Severity.MEDIUM, 'option': '-Wdate-time',
'description': 'Expansion of data or time macro',
'patterns': [r".*: warning: expansion of date or time macro is not reproducible"]},
+ {'category': 'C/C++', 'severity': Severity.MEDIUM, 'option': '-Wexpansion-to-defined',
+ 'description': 'Macro expansion has undefined behavior',
+ 'patterns': [r".*: warning: macro expansion .* has undefined behavior"]},
{'category': 'C/C++', 'severity': Severity.MEDIUM, 'option': '-Wformat',
'description': 'Format string does not match arguments',
'patterns': [r".*: warning: format '.+' expects type '.+', but argument [0-9]+ has type '.+'",
@@ -398,16 +454,16 @@
{'category': 'C/C++', 'severity': Severity.MEDIUM, 'option': '-Wimplicit int',
'description': 'No type or storage class (will default to int)',
'patterns': [r".*: warning: data definition has no type or storage class"]},
- {'category': 'C/C++', 'severity': Severity.MEDIUM,
- 'description': 'Null pointer',
- 'patterns': [r".*: warning: Dereference of null pointer",
- r".*: warning: Called .+ pointer is null",
- r".*: warning: Forming reference to null pointer",
- r".*: warning: Returning null reference",
- r".*: warning: Null pointer passed as an argument to a 'nonnull' parameter",
- r".*: warning: .+ results in a null pointer dereference",
- r".*: warning: Access to .+ results in a dereference of a null pointer",
- r".*: warning: Null pointer argument in"]},
+ # {'category': 'C/C++', 'severity': Severity.MEDIUM,
+ # 'description': 'Null pointer',
+ # 'patterns': [r".*: warning: Dereference of null pointer",
+ # r".*: warning: Called .+ pointer is null",
+ # r".*: warning: Forming reference to null pointer",
+ # r".*: warning: Returning null reference",
+ # r".*: warning: Null pointer passed as an argument to a 'nonnull' parameter",
+ # r".*: warning: .+ results in a null pointer dereference",
+ # r".*: warning: Access to .+ results in a dereference of a null pointer",
+ # r".*: warning: Null pointer argument in"]},
{'category': 'cont.', 'severity': Severity.SKIP,
'description': 'skip, parameter name (without types) in function declaration',
'patterns': [r".*: warning: parameter names \(without types\) in function declaration"]},
@@ -422,8 +478,8 @@
'description': 'Cast to pointer from integer of different size',
'patterns': [r".*: warning: cast to pointer from integer of different size"]},
{'category': 'C/C++', 'severity': Severity.MEDIUM,
- 'description': 'Symbol redefined',
- 'patterns': [r".*: warning: "".+"" redefined"]},
+ 'description': 'Macro redefined',
+ 'patterns': [r".*: warning: '.+' macro redefined"]},
{'category': 'cont.', 'severity': Severity.SKIP,
'description': 'skip, ... location of the previous definition',
'patterns': [r".*: warning: this is the location of the previous definition"]},
@@ -463,1700 +519,704 @@
{'category': 'cont.', 'severity': Severity.SKIP,
'description': 'skip, previous declaration ... was here',
'patterns': [r".*: warning: previous declaration of '.+' was here"]},
- {'category': 'C/C++', 'severity': Severity.MEDIUM, 'option': '-Wswitch-enum',
+ {'category': 'C/C++', 'severity': Severity.HIGH, 'option': '-Wswitch-enum',
'description': 'Enum value not handled in switch',
'patterns': [r".*: warning: .*enumeration value.* not handled in switch.+Wswitch"]},
{'category': 'C/C++', 'severity': Severity.MEDIUM, 'option': '-Wuser-defined-warnings',
'description': 'User defined warnings',
'patterns': [r".*: warning: .* \[-Wuser-defined-warnings\]$"]},
- {'category': 'java', 'severity': Severity.MEDIUM, 'option': '-encoding',
- 'description': 'Java: Non-ascii characters used, but ascii encoding specified',
- 'patterns': [r".*: warning: unmappable character for encoding ascii"]},
- {'category': 'java', 'severity': Severity.MEDIUM,
- 'description': 'Java: Non-varargs call of varargs method with inexact argument type for last parameter',
- 'patterns': [r".*: warning: non-varargs call of varargs method with inexact argument type for last parameter"]},
- {'category': 'java', 'severity': Severity.MEDIUM,
- 'description': 'Java: Unchecked method invocation',
- 'patterns': [r".*: warning: \[unchecked\] unchecked method invocation: .+ in class .+"]},
- {'category': 'java', 'severity': Severity.MEDIUM,
- 'description': 'Java: Unchecked conversion',
- 'patterns': [r".*: warning: \[unchecked\] unchecked conversion"]},
- {'category': 'java', 'severity': Severity.MEDIUM,
- 'description': '_ used as an identifier',
- 'patterns': [r".*: warning: '_' used as an identifier"]},
- {'category': 'java', 'severity': Severity.HIGH,
- 'description': 'Use of internal proprietary API',
- 'patterns': [r".*: warning: .* is internal proprietary API and may be removed"]},
+
+ # Java warnings
+ java_medium('Non-ascii characters used, but ascii encoding specified',
+ [r".*: warning: unmappable character for encoding ascii"]),
+ java_medium('Non-varargs call of varargs method with inexact argument type for last parameter',
+ [r".*: warning: non-varargs call of varargs method with inexact argument type for last parameter"]),
+ java_medium('Unchecked method invocation',
+ [r".*: warning: \[unchecked\] unchecked method invocation: .+ in class .+"]),
+ java_medium('Unchecked conversion',
+ [r".*: warning: \[unchecked\] unchecked conversion"]),
+ java_medium('_ used as an identifier',
+ [r".*: warning: '_' used as an identifier"]),
+ java_medium('hidden superclass',
+ [r".*: warning: .* stripped of .* superclass .* \[HiddenSuperclass\]"]),
+ java_high('Use of internal proprietary API',
+ [r".*: warning: .* is internal proprietary API and may be removed"]),
# Warnings from Javac
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description': 'Java: Use of deprecated member',
- 'patterns': [r'.*: warning: \[deprecation\] .+']},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description': 'Java: Unchecked conversion',
- 'patterns': [r'.*: warning: \[unchecked\] .+']},
+ java_medium('Use of deprecated member',
+ [r'.*: warning: \[deprecation\] .+']),
+ java_medium('Unchecked conversion',
+ [r'.*: warning: \[unchecked\] .+']),
# Begin warnings generated by Error Prone
- {'category': 'java',
- 'severity': Severity.LOW,
- 'description':
- 'Java: Use parameter comments to document ambiguous literals',
- 'patterns': [r".*: warning: \[BooleanParameter\] .+"]},
- {'category': 'java',
- 'severity': Severity.LOW,
- 'description':
- 'Java: This class\'s name looks like a Type Parameter.',
- 'patterns': [r".*: warning: \[ClassNamedLikeTypeParameter\] .+"]},
- {'category': 'java',
- 'severity': Severity.LOW,
- 'description':
- 'Java: Field name is CONSTANT_CASE, but field is not static and final',
- 'patterns': [r".*: warning: \[ConstantField\] .+"]},
- {'category': 'java',
- 'severity': Severity.LOW,
- 'description':
- 'Java: @Multibinds is a more efficient and declarative mechanism for ensuring that a set multibinding is present in the graph.',
- 'patterns': [r".*: warning: \[EmptySetMultibindingContributions\] .+"]},
- {'category': 'java',
- 'severity': Severity.LOW,
- 'description':
- 'Java: Prefer assertThrows to ExpectedException',
- 'patterns': [r".*: warning: \[ExpectedExceptionRefactoring\] .+"]},
- {'category': 'java',
- 'severity': Severity.LOW,
- 'description':
- 'Java: This field is only assigned during initialization; consider making it final',
- 'patterns': [r".*: warning: \[FieldCanBeFinal\] .+"]},
- {'category': 'java',
- 'severity': Severity.LOW,
- 'description':
- 'Java: Fields that can be null should be annotated @Nullable',
- 'patterns': [r".*: warning: \[FieldMissingNullable\] .+"]},
- {'category': 'java',
- 'severity': Severity.LOW,
- 'description':
- 'Java: Refactors uses of the JSR 305 @Immutable to Error Prone\'s annotation',
- 'patterns': [r".*: warning: \[ImmutableRefactoring\] .+"]},
- {'category': 'java',
- 'severity': Severity.LOW,
- 'description':
- u'Java: Use Java\'s utility functional interfaces instead of Function\u003cA, B> for primitive types.',
- 'patterns': [r".*: warning: \[LambdaFunctionalInterface\] .+"]},
- {'category': 'java',
- 'severity': Severity.LOW,
- 'description':
- 'Java: A private method that does not reference the enclosing instance can be static',
- 'patterns': [r".*: warning: \[MethodCanBeStatic\] .+"]},
- {'category': 'java',
- 'severity': Severity.LOW,
- 'description':
- 'Java: C-style array declarations should not be used',
- 'patterns': [r".*: warning: \[MixedArrayDimensions\] .+"]},
- {'category': 'java',
- 'severity': Severity.LOW,
- 'description':
- 'Java: Variable declarations should declare only one variable',
- 'patterns': [r".*: warning: \[MultiVariableDeclaration\] .+"]},
- {'category': 'java',
- 'severity': Severity.LOW,
- 'description':
- 'Java: Source files should not contain multiple top-level class declarations',
- 'patterns': [r".*: warning: \[MultipleTopLevelClasses\] .+"]},
- {'category': 'java',
- 'severity': Severity.LOW,
- 'description':
- 'Java: Avoid having multiple unary operators acting on the same variable in a method call',
- 'patterns': [r".*: warning: \[MultipleUnaryOperatorsInMethodCall\] .+"]},
- {'category': 'java',
- 'severity': Severity.LOW,
- 'description':
- 'Java: Package names should match the directory they are declared in',
- 'patterns': [r".*: warning: \[PackageLocation\] .+"]},
- {'category': 'java',
- 'severity': Severity.LOW,
- 'description':
- 'Java: Non-standard parameter comment; prefer `/* paramName= */ arg`',
- 'patterns': [r".*: warning: \[ParameterComment\] .+"]},
- {'category': 'java',
- 'severity': Severity.LOW,
- 'description':
- 'Java: Method parameters that aren\'t checked for null shouldn\'t be annotated @Nullable',
- 'patterns': [r".*: warning: \[ParameterNotNullable\] .+"]},
- {'category': 'java',
- 'severity': Severity.LOW,
- 'description':
- 'Java: Add a private constructor to modules that will not be instantiated by Dagger.',
- 'patterns': [r".*: warning: \[PrivateConstructorForNoninstantiableModule\] .+"]},
- {'category': 'java',
- 'severity': Severity.LOW,
- 'description':
- 'Java: Utility classes (only static members) are not designed to be instantiated and should be made noninstantiable with a default constructor.',
- 'patterns': [r".*: warning: \[PrivateConstructorForUtilityClass\] .+"]},
- {'category': 'java',
- 'severity': Severity.LOW,
- 'description':
- 'Java: Unused imports',
- 'patterns': [r".*: warning: \[RemoveUnusedImports\] .+"]},
- {'category': 'java',
- 'severity': Severity.LOW,
- 'description':
- 'Java: Methods that can return null should be annotated @Nullable',
- 'patterns': [r".*: warning: \[ReturnMissingNullable\] .+"]},
- {'category': 'java',
- 'severity': Severity.LOW,
- 'description':
- 'Java: Scopes on modules have no function and will soon be an error.',
- 'patterns': [r".*: warning: \[ScopeOnModule\] .+"]},
- {'category': 'java',
- 'severity': Severity.LOW,
- 'description':
- 'Java: The default case of a switch should appear at the end of the last statement group',
- 'patterns': [r".*: warning: \[SwitchDefault\] .+"]},
- {'category': 'java',
- 'severity': Severity.LOW,
- 'description':
- 'Java: Prefer assertThrows to @Test(expected=...)',
- 'patterns': [r".*: warning: \[TestExceptionRefactoring\] .+"]},
- {'category': 'java',
- 'severity': Severity.LOW,
- 'description':
- 'Java: Unchecked exceptions do not need to be declared in the method signature.',
- 'patterns': [r".*: warning: \[ThrowsUncheckedException\] .+"]},
- {'category': 'java',
- 'severity': Severity.LOW,
- 'description':
- 'Java: Prefer assertThrows to try/fail',
- 'patterns': [r".*: warning: \[TryFailRefactoring\] .+"]},
- {'category': 'java',
- 'severity': Severity.LOW,
- 'description':
- 'Java: Type parameters must be a single letter with an optional numeric suffix, or an UpperCamelCase name followed by the letter \'T\'.',
- 'patterns': [r".*: warning: \[TypeParameterNaming\] .+"]},
- {'category': 'java',
- 'severity': Severity.LOW,
- 'description':
- 'Java: Constructors and methods with the same name should appear sequentially with no other code in between. Please re-order or re-name methods.',
- 'patterns': [r".*: warning: \[UngroupedOverloads\] .+"]},
- {'category': 'java',
- 'severity': Severity.LOW,
- 'description':
- 'Java: Unnecessary call to NullPointerTester#setDefault',
- 'patterns': [r".*: warning: \[UnnecessarySetDefault\] .+"]},
- {'category': 'java',
- 'severity': Severity.LOW,
- 'description':
- 'Java: Using static imports for types is unnecessary',
- 'patterns': [r".*: warning: \[UnnecessaryStaticImport\] .+"]},
- {'category': 'java',
- 'severity': Severity.LOW,
- 'description':
- 'Java: @Binds is a more efficient and declarative mechanism for delegating a binding.',
- 'patterns': [r".*: warning: \[UseBinds\] .+"]},
- {'category': 'java',
- 'severity': Severity.LOW,
- 'description':
- 'Java: Wildcard imports, static or otherwise, should not be used',
- 'patterns': [r".*: warning: \[WildcardImport\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Method reference is ambiguous',
- 'patterns': [r".*: warning: \[AmbiguousMethodReference\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: This method passes a pair of parameters through to String.format, but the enclosing method wasn\'t annotated @FormatMethod. Doing so gives compile-time rather than run-time protection against malformed format strings.',
- 'patterns': [r".*: warning: \[AnnotateFormatMethod\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Annotations should be positioned after Javadocs, but before modifiers..',
- 'patterns': [r".*: warning: \[AnnotationPosition\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Arguments are in the wrong order or could be commented for clarity.',
- 'patterns': [r".*: warning: \[ArgumentSelectionDefectChecker\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Arrays do not override equals() or hashCode, so comparisons will be done on reference equality only. If neither deduplication nor lookup are needed, consider using a List instead. Otherwise, use IdentityHashMap/Set, a Map from a library that handles object arrays, or an Iterable/List of pairs.',
- 'patterns': [r".*: warning: \[ArrayAsKeyOfSetOrMap\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Arguments are swapped in assertEquals-like call',
- 'patterns': [r".*: warning: \[AssertEqualsArgumentOrderChecker\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Assertions may be disabled at runtime and do not guarantee that execution will halt here; consider throwing an exception instead',
- 'patterns': [r".*: warning: \[AssertFalse\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: The lambda passed to assertThrows should contain exactly one statement',
- 'patterns': [r".*: warning: \[AssertThrowsMultipleStatements\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: This assertion throws an AssertionError if it fails, which will be caught by an enclosing try block.',
- 'patterns': [r".*: warning: \[AssertionFailureIgnored\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: @AssistedInject and @Inject should not be used on different constructors in the same class.',
- 'patterns': [r".*: warning: \[AssistedInjectAndInjectOnConstructors\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Make toString(), hashCode() and equals() final in AutoValue classes, so it is clear to readers that AutoValue is not overriding them',
- 'patterns': [r".*: warning: \[AutoValueFinalMethods\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Classes that implement Annotation must override equals and hashCode. Consider using AutoAnnotation instead of implementing Annotation by hand.',
- 'patterns': [r".*: warning: \[BadAnnotationImplementation\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Possible sign flip from narrowing conversion',
- 'patterns': [r".*: warning: \[BadComparable\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Importing nested classes/static methods/static fields with commonly-used names can make code harder to read, because it may not be clear from the context exactly which type is being referred to. Qualifying the name with that of the containing class can make the code clearer.',
- 'patterns': [r".*: warning: \[BadImport\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: instanceof used in a way that is equivalent to a null check.',
- 'patterns': [r".*: warning: \[BadInstanceof\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: BigDecimal#equals has surprising behavior: it also compares scale.',
- 'patterns': [r".*: warning: \[BigDecimalEquals\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: new BigDecimal(double) loses precision in this case.',
- 'patterns': [r".*: warning: \[BigDecimalLiteralDouble\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: A call to Binder.clearCallingIdentity() should be followed by Binder.restoreCallingIdentity() in a finally block. Otherwise the wrong Binder identity may be used by subsequent code.',
- 'patterns': [r".*: warning: \[BinderIdentityRestoredDangerously\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: This code declares a binding for a common value type without a Qualifier annotation.',
- 'patterns': [r".*: warning: \[BindingToUnqualifiedCommonType\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: valueOf or autoboxing provides better time and space performance',
- 'patterns': [r".*: warning: \[BoxedPrimitiveConstructor\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: ByteBuffer.array() shouldn\'t be called unless ByteBuffer.arrayOffset() is used or if the ByteBuffer was initialized using ByteBuffer.wrap() or ByteBuffer.allocate().',
- 'patterns': [r".*: warning: \[ByteBufferBackingArray\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Mockito cannot mock final classes',
- 'patterns': [r".*: warning: \[CannotMockFinalClass\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Duration can be expressed more clearly with different units',
- 'patterns': [r".*: warning: \[CanonicalDuration\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Logging or rethrowing exceptions should usually be preferred to catching and calling printStackTrace',
- 'patterns': [r".*: warning: \[CatchAndPrintStackTrace\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Ignoring exceptions and calling fail() is unnecessary, and makes test output less useful',
- 'patterns': [r".*: warning: \[CatchFail\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Inner class is non-static but does not reference enclosing class',
- 'patterns': [r".*: warning: \[ClassCanBeStatic\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Class.newInstance() bypasses exception checking; prefer getDeclaredConstructor().newInstance()',
- 'patterns': [r".*: warning: \[ClassNewInstance\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Providing Closeable resources makes their lifecycle unclear',
- 'patterns': [r".*: warning: \[CloseableProvides\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: The type of the array parameter of Collection.toArray needs to be compatible with the array type',
- 'patterns': [r".*: warning: \[CollectionToArraySafeParameter\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Collector.of() should not use state',
- 'patterns': [r".*: warning: \[CollectorShouldNotUseState\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Class should not implement both `Comparable` and `Comparator`',
- 'patterns': [r".*: warning: \[ComparableAndComparator\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Constructors should not invoke overridable methods.',
- 'patterns': [r".*: warning: \[ConstructorInvokesOverridable\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Constructors should not pass the \'this\' reference out in method invocations, since the object may not be fully constructed.',
- 'patterns': [r".*: warning: \[ConstructorLeaksThis\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: DateFormat is not thread-safe, and should not be used as a constant field.',
- 'patterns': [r".*: warning: \[DateFormatConstant\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Implicit use of the platform default charset, which can result in differing behaviour between JVM executions or incorrect behavior if the encoding of the data source doesn\'t match expectations.',
- 'patterns': [r".*: warning: \[DefaultCharset\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Avoid deprecated Thread methods; read the method\'s javadoc for details.',
- 'patterns': [r".*: warning: \[DeprecatedThreadMethods\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Prefer collection factory methods or builders to the double-brace initialization pattern.',
- 'patterns': [r".*: warning: \[DoubleBraceInitialization\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Double-checked locking on non-volatile fields is unsafe',
- 'patterns': [r".*: warning: \[DoubleCheckedLocking\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Empty top-level type declaration',
- 'patterns': [r".*: warning: \[EmptyTopLevelDeclaration\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: equals() implementation may throw NullPointerException when given null',
- 'patterns': [r".*: warning: \[EqualsBrokenForNull\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Overriding Object#equals in a non-final class by using getClass rather than instanceof breaks substitutability of subclasses.',
- 'patterns': [r".*: warning: \[EqualsGetClass\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Classes that override equals should also override hashCode.',
- 'patterns': [r".*: warning: \[EqualsHashCode\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: An equality test between objects with incompatible types always returns false',
- 'patterns': [r".*: warning: \[EqualsIncompatibleType\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: The contract of #equals states that it should return false for incompatible types, while this implementation may throw ClassCastException.',
- 'patterns': [r".*: warning: \[EqualsUnsafeCast\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Implementing #equals by just comparing hashCodes is fragile. Hashes collide frequently, and this will lead to false positives in #equals.',
- 'patterns': [r".*: warning: \[EqualsUsingHashCode\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Calls to ExpectedException#expect should always be followed by exactly one statement.',
- 'patterns': [r".*: warning: \[ExpectedExceptionChecker\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: When only using JUnit Assert\'s static methods, you should import statically instead of extending.',
- 'patterns': [r".*: warning: \[ExtendingJUnitAssert\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Switch case may fall through',
- 'patterns': [r".*: warning: \[FallThrough\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: If you return or throw from a finally, then values returned or thrown from the try-catch block will be ignored. Consider using try-with-resources instead.',
- 'patterns': [r".*: warning: \[Finally\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Use parentheses to make the precedence explicit',
- 'patterns': [r".*: warning: \[FloatCast\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: This fuzzy equality check is using a tolerance less than the gap to the next number. You may want a less restrictive tolerance, or to assert equality.',
- 'patterns': [r".*: warning: \[FloatingPointAssertionWithinEpsilon\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Floating point literal loses precision',
- 'patterns': [r".*: warning: \[FloatingPointLiteralPrecision\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Classes extending PreferenceActivity must implement isValidFragment such that it does not unconditionally return true to prevent vulnerability to fragment injection attacks.',
- 'patterns': [r".*: warning: \[FragmentInjection\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Subclasses of Fragment must be instantiable via Class#newInstance(): the class must be public, static and have a public nullary constructor',
- 'patterns': [r".*: warning: \[FragmentNotInstantiable\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Overloads will be ambiguous when passing lambda arguments',
- 'patterns': [r".*: warning: \[FunctionalInterfaceClash\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Return value of methods returning Future must be checked. Ignoring returned Futures suppresses exceptions thrown from the code that completes the Future.',
- 'patterns': [r".*: warning: \[FutureReturnValueIgnored\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Calling getClass() on an enum may return a subclass of the enum type',
- 'patterns': [r".*: warning: \[GetClassOnEnum\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Hardcoded reference to /sdcard',
- 'patterns': [r".*: warning: \[HardCodedSdCardPath\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Hiding fields of superclasses may cause confusion and errors',
- 'patterns': [r".*: warning: \[HidingField\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Annotations should always be immutable',
- 'patterns': [r".*: warning: \[ImmutableAnnotationChecker\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Enums should always be immutable',
- 'patterns': [r".*: warning: \[ImmutableEnumChecker\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: This annotation has incompatible modifiers as specified by its @IncompatibleModifiers annotation',
- 'patterns': [r".*: warning: \[IncompatibleModifiers\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: It is confusing to have a field and a parameter under the same scope that differ only in capitalization.',
- 'patterns': [r".*: warning: \[InconsistentCapitalization\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Including fields in hashCode which are not compared in equals violates the contract of hashCode.',
- 'patterns': [r".*: warning: \[InconsistentHashCode\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: The ordering of parameters in overloaded methods should be as consistent as possible (when viewed from left to right)',
- 'patterns': [r".*: warning: \[InconsistentOverloads\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: This for loop increments the same variable in the header and in the body',
- 'patterns': [r".*: warning: \[IncrementInForLoopAndHeader\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Constructors on abstract classes are never directly @Injected, only the constructors of their subclasses can be @Inject\'ed.',
- 'patterns': [r".*: warning: \[InjectOnConstructorOfAbstractClass\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Please also override int read(byte[], int, int), otherwise multi-byte reads from this input stream are likely to be slow.',
- 'patterns': [r".*: warning: \[InputStreamSlowMultibyteRead\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Casting inside an if block should be plausibly consistent with the instanceof type',
- 'patterns': [r".*: warning: \[InstanceOfAndCastMatchWrongType\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Expression of type int may overflow before being assigned to a long',
- 'patterns': [r".*: warning: \[IntLongMath\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: This @param tag doesn\'t refer to a parameter of the method.',
- 'patterns': [r".*: warning: \[InvalidParam\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: This tag is invalid.',
- 'patterns': [r".*: warning: \[InvalidTag\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: The documented method doesn\'t actually throw this checked exception.',
- 'patterns': [r".*: warning: \[InvalidThrows\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Class should not implement both `Iterable` and `Iterator`',
- 'patterns': [r".*: warning: \[IterableAndIterator\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Floating-point comparison without error tolerance',
- 'patterns': [r".*: warning: \[JUnit3FloatingPointComparisonWithoutDelta\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Some JUnit4 construct cannot be used in a JUnit3 context. Convert your class to JUnit4 style to use them.',
- 'patterns': [r".*: warning: \[JUnit4ClassUsedInJUnit3\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Test class inherits from JUnit 3\'s TestCase but has JUnit 4 @Test annotations.',
- 'patterns': [r".*: warning: \[JUnitAmbiguousTestClass\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Never reuse class names from java.lang',
- 'patterns': [r".*: warning: \[JavaLangClash\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Suggests alternatives to obsolete JDK classes.',
- 'patterns': [r".*: warning: \[JdkObsolete\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Calls to Lock#lock should be immediately followed by a try block which releases the lock.',
- 'patterns': [r".*: warning: \[LockNotBeforeTry\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Assignment where a boolean expression was expected; use == if this assignment wasn\'t expected or add parentheses for clarity.',
- 'patterns': [r".*: warning: \[LogicalAssignment\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Math.abs does not always give a positive result. Please consider other methods for positive random numbers.',
- 'patterns': [r".*: warning: \[MathAbsoluteRandom\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Switches on enum types should either handle all values, or have a default case.',
- 'patterns': [r".*: warning: \[MissingCasesInEnumSwitch\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: The Google Java Style Guide requires that each switch statement includes a default statement group, even if it contains no code. (This requirement is lifted for any switch statement that covers all values of an enum.)',
- 'patterns': [r".*: warning: \[MissingDefault\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Not calling fail() when expecting an exception masks bugs',
- 'patterns': [r".*: warning: \[MissingFail\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: method overrides method in supertype; expected @Override',
- 'patterns': [r".*: warning: \[MissingOverride\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: A collection or proto builder was created, but its values were never accessed.',
- 'patterns': [r".*: warning: \[ModifiedButNotUsed\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Modifying a collection while iterating over it in a loop may cause a ConcurrentModificationException to be thrown.',
- 'patterns': [r".*: warning: \[ModifyCollectionInEnhancedForLoop\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Multiple calls to either parallel or sequential are unnecessary and cause confusion.',
- 'patterns': [r".*: warning: \[MultipleParallelOrSequentialCalls\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Constant field declarations should use the immutable type (such as ImmutableList) instead of the general collection interface type (such as List)',
- 'patterns': [r".*: warning: \[MutableConstantField\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Method return type should use the immutable type (such as ImmutableList) instead of the general collection interface type (such as List)',
- 'patterns': [r".*: warning: \[MutableMethodReturnType\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Compound assignments may hide dangerous casts',
- 'patterns': [r".*: warning: \[NarrowingCompoundAssignment\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Nested instanceOf conditions of disjoint types create blocks of code that never execute',
- 'patterns': [r".*: warning: \[NestedInstanceOfConditions\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Instead of returning a functional type, return the actual type that the returned function would return and use lambdas at use site.',
- 'patterns': [r".*: warning: \[NoFunctionalReturnType\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: This update of a volatile variable is non-atomic',
- 'patterns': [r".*: warning: \[NonAtomicVolatileUpdate\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Static import of member uses non-canonical name',
- 'patterns': [r".*: warning: \[NonCanonicalStaticMemberImport\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: equals method doesn\'t override Object.equals',
- 'patterns': [r".*: warning: \[NonOverridingEquals\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Constructors should not be annotated with @Nullable since they cannot return null',
- 'patterns': [r".*: warning: \[NullableConstructor\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Dereference of possibly-null value',
- 'patterns': [r".*: warning: \[NullableDereference\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: @Nullable should not be used for primitive types since they cannot be null',
- 'patterns': [r".*: warning: \[NullablePrimitive\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: void-returning methods should not be annotated with @Nullable, since they cannot return null',
- 'patterns': [r".*: warning: \[NullableVoid\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Calling toString on Objects that don\'t override toString() doesn\'t provide useful information',
- 'patterns': [r".*: warning: \[ObjectToString\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Objects.hashCode(Object o) should not be passed a primitive value',
- 'patterns': [r".*: warning: \[ObjectsHashCodePrimitive\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Use grouping parenthesis to make the operator precedence explicit',
- 'patterns': [r".*: warning: \[OperatorPrecedence\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: One should not call optional.get() inside an if statement that checks !optional.isPresent',
- 'patterns': [r".*: warning: \[OptionalNotPresent\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: String literal contains format specifiers, but is not passed to a format method',
- 'patterns': [r".*: warning: \[OrphanedFormatString\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: To return a custom message with a Throwable class, one should override getMessage() instead of toString() for Throwable.',
- 'patterns': [r".*: warning: \[OverrideThrowableToString\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Varargs doesn\'t agree for overridden method',
- 'patterns': [r".*: warning: \[Overrides\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: This method is not annotated with @Inject, but it overrides a method that is annotated with @com.google.inject.Inject. Guice will inject this method, and it is recommended to annotate it explicitly.',
- 'patterns': [r".*: warning: \[OverridesGuiceInjectableMethod\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Detects `/* name= */`-style comments on actual parameters where the name doesn\'t match the formal parameter',
- 'patterns': [r".*: warning: \[ParameterName\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Preconditions only accepts the %s placeholder in error message strings',
- 'patterns': [r".*: warning: \[PreconditionsInvalidPlaceholder\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Passing a primitive array to a varargs method is usually wrong',
- 'patterns': [r".*: warning: \[PrimitiveArrayPassedToVarargsMethod\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: A field on a protocol buffer was set twice in the same chained expression.',
- 'patterns': [r".*: warning: \[ProtoRedundantSet\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Protos should not be used as a key to a map, in a set, or in a contains method on a descendant of a collection. Protos have non deterministic ordering and proto equality is deep, which is a performance issue.',
- 'patterns': [r".*: warning: \[ProtosAsKeyOfSetOrMap\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: BugChecker has incorrect ProvidesFix tag, please update',
- 'patterns': [r".*: warning: \[ProvidesFix\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Qualifiers/Scope annotations on @Inject methods don\'t have any effect. Move the qualifier annotation to the binding location.',
- 'patterns': [r".*: warning: \[QualifierOrScopeOnInjectMethod\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Injection frameworks currently don\'t understand Qualifiers in TYPE_PARAMETER or TYPE_USE contexts.',
- 'patterns': [r".*: warning: \[QualifierWithTypeUse\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: reachabilityFence should always be called inside a finally block',
- 'patterns': [r".*: warning: \[ReachabilityFenceUsage\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Thrown exception is a subtype of another',
- 'patterns': [r".*: warning: \[RedundantThrows\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Comparison using reference equality instead of value equality',
- 'patterns': [r".*: warning: \[ReferenceEquality\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: This annotation is missing required modifiers as specified by its @RequiredModifiers annotation',
- 'patterns': [r".*: warning: \[RequiredModifiers\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Void methods should not have a @return tag.',
- 'patterns': [r".*: warning: \[ReturnFromVoid\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- u'Java: Prefer the short-circuiting boolean operators \u0026\u0026 and || to \u0026 and |.',
- 'patterns': [r".*: warning: \[ShortCircuitBoolean\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Writes to static fields should not be guarded by instance locks',
- 'patterns': [r".*: warning: \[StaticGuardedByInstance\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: A static variable or method should be qualified with a class name, not expression',
- 'patterns': [r".*: warning: \[StaticQualifiedUsingExpression\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Streams that encapsulate a closeable resource should be closed using try-with-resources',
- 'patterns': [r".*: warning: \[StreamResourceLeak\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: String comparison using reference equality instead of value equality',
- 'patterns': [r".*: warning: \[StringEquality\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: String.split(String) has surprising behavior',
- 'patterns': [r".*: warning: \[StringSplitter\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: SWIG generated code that can\'t call a C++ destructor will leak memory',
- 'patterns': [r".*: warning: \[SwigMemoryLeak\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Synchronizing on non-final fields is not safe: if the field is ever updated, different threads may end up locking on different objects.',
- 'patterns': [r".*: warning: \[SynchronizeOnNonFinalField\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Code that contains System.exit() is untestable.',
- 'patterns': [r".*: warning: \[SystemExitOutsideMain\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Using @Test(expected=...) is discouraged, since the test will pass if *any* statement in the test method throws the expected exception',
- 'patterns': [r".*: warning: \[TestExceptionChecker\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Thread.join needs to be surrounded by a loop until it succeeds, as in Uninterruptibles.joinUninterruptibly.',
- 'patterns': [r".*: warning: \[ThreadJoinLoop\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: ThreadLocals should be stored in static fields',
- 'patterns': [r".*: warning: \[ThreadLocalUsage\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Relying on the thread scheduler is discouraged; see Effective Java Item 72 (2nd edition) / 84 (3rd edition).',
- 'patterns': [r".*: warning: \[ThreadPriorityCheck\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Three-letter time zone identifiers are deprecated, may be ambiguous, and might not do what you intend; the full IANA time zone ID should be used instead.',
- 'patterns': [r".*: warning: \[ThreeLetterTimeZoneID\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: An implementation of Object.toString() should never return null.',
- 'patterns': [r".*: warning: \[ToStringReturnsNull\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: The actual and expected values appear to be swapped, which results in poor assertion failure messages. The actual value should come first.',
- 'patterns': [r".*: warning: \[TruthAssertExpected\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Truth Library assert is called on a constant.',
- 'patterns': [r".*: warning: \[TruthConstantAsserts\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Argument is not compatible with the subject\'s type.',
- 'patterns': [r".*: warning: \[TruthIncompatibleType\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Type parameter declaration shadows another named type',
- 'patterns': [r".*: warning: \[TypeNameShadowing\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Type parameter declaration overrides another type parameter already declared',
- 'patterns': [r".*: warning: \[TypeParameterShadowing\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Declaring a type parameter that is only used in the return type is a misuse of generics: operations on the type parameter are unchecked, it hides unsafe casts at invocations of the method, and it interacts badly with method overload resolution.',
- 'patterns': [r".*: warning: \[TypeParameterUnusedInFormals\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Avoid hash-based containers of java.net.URL--the containers rely on equals() and hashCode(), which cause java.net.URL to make blocking internet connections.',
- 'patterns': [r".*: warning: \[URLEqualsHashCode\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Collection, Iterable, Multimap, and Queue do not have well-defined equals behavior',
- 'patterns': [r".*: warning: \[UndefinedEquals\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Switch handles all enum values: an explicit default case is unnecessary and defeats error checking for non-exhaustive switches.',
- 'patterns': [r".*: warning: \[UnnecessaryDefaultInEnumSwitch\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Unnecessary use of grouping parentheses',
- 'patterns': [r".*: warning: \[UnnecessaryParentheses\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Finalizer may run before native code finishes execution',
- 'patterns': [r".*: warning: \[UnsafeFinalization\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Prefer `asSubclass` instead of casting the result of `newInstance`, to detect classes of incorrect type before invoking their constructors.This way, if the class is of the incorrect type,it will throw an exception before invoking its constructor.',
- 'patterns': [r".*: warning: \[UnsafeReflectiveConstructionCast\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Unsynchronized method overrides a synchronized method.',
- 'patterns': [r".*: warning: \[UnsynchronizedOverridesSynchronized\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Unused.',
- 'patterns': [r".*: warning: \[Unused\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: This catch block catches an exception and re-throws another, but swallows the caught exception rather than setting it as a cause. This can make debugging harder.',
- 'patterns': [r".*: warning: \[UnusedException\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Java assert is used in test. For testing purposes Assert.* matchers should be used.',
- 'patterns': [r".*: warning: \[UseCorrectAssertInTests\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Non-constant variable missing @Var annotation',
- 'patterns': [r".*: warning: \[Var\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: variableName and type with the same name would refer to the static field instead of the class',
- 'patterns': [r".*: warning: \[VariableNameSameAsType\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: Because of spurious wakeups, Object.wait() and Condition.await() must always be called in a loop',
- 'patterns': [r".*: warning: \[WaitNotInLoop\] .+"]},
- {'category': 'java',
- 'severity': Severity.MEDIUM,
- 'description':
- 'Java: A wakelock acquired with a timeout may be released by the system before calling `release`, even after checking `isHeld()`. If so, it will throw a RuntimeException. Please wrap in a try/catch block.',
- 'patterns': [r".*: warning: \[WakelockReleasedDangerously\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: AndroidInjection.inject() should always be invoked before calling super.lifecycleMethod()',
- 'patterns': [r".*: warning: \[AndroidInjectionBeforeSuper\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Use of class, field, or method that is not compatible with legacy Android devices',
- 'patterns': [r".*: warning: \[AndroidJdkLibsChecker\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Reference equality used to compare arrays',
- 'patterns': [r".*: warning: \[ArrayEquals\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Arrays.fill(Object[], Object) called with incompatible types.',
- 'patterns': [r".*: warning: \[ArrayFillIncompatibleType\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: hashcode method on array does not hash array contents',
- 'patterns': [r".*: warning: \[ArrayHashCode\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Calling toString on an array does not provide useful information',
- 'patterns': [r".*: warning: \[ArrayToString\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Arrays.asList does not autobox primitive arrays, as one might expect.',
- 'patterns': [r".*: warning: \[ArraysAsListPrimitiveArray\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: @AssistedInject and @Inject cannot be used on the same constructor.',
- 'patterns': [r".*: warning: \[AssistedInjectAndInjectOnSameConstructor\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: AsyncCallable should not return a null Future, only a Future whose result is null.',
- 'patterns': [r".*: warning: \[AsyncCallableReturnsNull\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: AsyncFunction should not return a null Future, only a Future whose result is null.',
- 'patterns': [r".*: warning: \[AsyncFunctionReturnsNull\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: @AutoFactory and @Inject should not be used in the same type.',
- 'patterns': [r".*: warning: \[AutoFactoryAtInject\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Arguments to AutoValue constructor are in the wrong order',
- 'patterns': [r".*: warning: \[AutoValueConstructorOrderChecker\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Shift by an amount that is out of range',
- 'patterns': [r".*: warning: \[BadShiftAmount\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Object serialized in Bundle may have been flattened to base type.',
- 'patterns': [r".*: warning: \[BundleDeserializationCast\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: The called constructor accepts a parameter with the same name and type as one of its caller\'s parameters, but its caller doesn\'t pass that parameter to it. It\'s likely that it was intended to.',
- 'patterns': [r".*: warning: \[ChainingConstructorIgnoresParameter\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Ignored return value of method that is annotated with @CheckReturnValue',
- 'patterns': [r".*: warning: \[CheckReturnValue\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: The source file name should match the name of the top-level class it contains',
- 'patterns': [r".*: warning: \[ClassName\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Incompatible type as argument to Object-accepting Java collections method',
- 'patterns': [r".*: warning: \[CollectionIncompatibleType\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- u'Java: Implementing \'Comparable\u003cT>\' where T is not compatible with the implementing class.',
- 'patterns': [r".*: warning: \[ComparableType\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: this == null is always false, this != null is always true',
- 'patterns': [r".*: warning: \[ComparingThisWithNull\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: This comparison method violates the contract',
- 'patterns': [r".*: warning: \[ComparisonContractViolated\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Comparison to value that is out of range for the compared type',
- 'patterns': [r".*: warning: \[ComparisonOutOfRange\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: @CompatibleWith\'s value is not a type argument.',
- 'patterns': [r".*: warning: \[CompatibleWithAnnotationMisuse\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Non-compile-time constant expression passed to parameter with @CompileTimeConstant type annotation.',
- 'patterns': [r".*: warning: \[CompileTimeConstant\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Non-trivial compile time constant boolean expressions shouldn\'t be used.',
- 'patterns': [r".*: warning: \[ComplexBooleanConstant\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: A conditional expression with numeric operands of differing types will perform binary numeric promotion of the operands; when these operands are of reference types, the expression\'s result may not be of the expected type.',
- 'patterns': [r".*: warning: \[ConditionalExpressionNumericPromotion\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Compile-time constant expression overflows',
- 'patterns': [r".*: warning: \[ConstantOverflow\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Dagger @Provides methods may not return null unless annotated with @Nullable',
- 'patterns': [r".*: warning: \[DaggerProvidesNull\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Exception created but not thrown',
- 'patterns': [r".*: warning: \[DeadException\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Thread created but not started',
- 'patterns': [r".*: warning: \[DeadThread\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Deprecated item is not annotated with @Deprecated',
- 'patterns': [r".*: warning: \[DepAnn\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Division by integer literal zero',
- 'patterns': [r".*: warning: \[DivZero\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: This method should not be called.',
- 'patterns': [r".*: warning: \[DoNotCall\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Empty statement after if',
- 'patterns': [r".*: warning: \[EmptyIf\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: == NaN always returns false; use the isNaN methods instead',
- 'patterns': [r".*: warning: \[EqualsNaN\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: == must be used in equals method to check equality to itself or an infinite loop will occur.',
- 'patterns': [r".*: warning: \[EqualsReference\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Comparing different pairs of fields/getters in an equals implementation is probably a mistake.',
- 'patterns': [r".*: warning: \[EqualsWrongThing\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Method annotated @ForOverride must be protected or package-private and only invoked from declaring class, or from an override of the method',
- 'patterns': [r".*: warning: \[ForOverride\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Invalid printf-style format string',
- 'patterns': [r".*: warning: \[FormatString\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Invalid format string passed to formatting method.',
- 'patterns': [r".*: warning: \[FormatStringAnnotation\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Casting a lambda to this @FunctionalInterface can cause a behavior change from casting to a functional superinterface, which is surprising to users. Prefer decorator methods to this surprising behavior.',
- 'patterns': [r".*: warning: \[FunctionalInterfaceMethodChanged\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Futures.getChecked requires a checked exception type with a standard constructor.',
- 'patterns': [r".*: warning: \[FuturesGetCheckedIllegalExceptionType\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: DoubleMath.fuzzyEquals should never be used in an Object.equals() method',
- 'patterns': [r".*: warning: \[FuzzyEqualsShouldNotBeUsedInEqualsMethod\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Calling getClass() on an annotation may return a proxy class',
- 'patterns': [r".*: warning: \[GetClassOnAnnotation\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Calling getClass() on an object of type Class returns the Class object for java.lang.Class; you probably meant to operate on the object directly',
- 'patterns': [r".*: warning: \[GetClassOnClass\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Checks for unguarded accesses to fields and methods with @GuardedBy annotations',
- 'patterns': [r".*: warning: \[GuardedBy\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Scope annotation on implementation class of AssistedInject factory is not allowed',
- 'patterns': [r".*: warning: \[GuiceAssistedInjectScoping\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: A constructor cannot have two @Assisted parameters of the same type unless they are disambiguated with named @Assisted annotations.',
- 'patterns': [r".*: warning: \[GuiceAssistedParameters\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Although Guice allows injecting final fields, doing so is disallowed because the injected value may not be visible to other threads.',
- 'patterns': [r".*: warning: \[GuiceInjectOnFinalField\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: contains() is a legacy method that is equivalent to containsValue()',
- 'patterns': [r".*: warning: \[HashtableContains\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: A binary expression where both operands are the same is usually incorrect.',
- 'patterns': [r".*: warning: \[IdentityBinaryExpression\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Type declaration annotated with @Immutable is not immutable',
- 'patterns': [r".*: warning: \[Immutable\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Modifying an immutable collection is guaranteed to throw an exception and leave the collection unmodified',
- 'patterns': [r".*: warning: \[ImmutableModification\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Passing argument to a generic method with an incompatible type.',
- 'patterns': [r".*: warning: \[IncompatibleArgumentType\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: The first argument to indexOf is a Unicode code point, and the second is the index to start the search from',
- 'patterns': [r".*: warning: \[IndexOfChar\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Conditional expression in varargs call contains array and non-array arguments',
- 'patterns': [r".*: warning: \[InexactVarargsConditional\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: This method always recurses, and will cause a StackOverflowError',
- 'patterns': [r".*: warning: \[InfiniteRecursion\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: A scoping annotation\'s Target should include TYPE and METHOD.',
- 'patterns': [r".*: warning: \[InjectInvalidTargetingOnScopingAnnotation\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Using more than one qualifier annotation on the same element is not allowed.',
- 'patterns': [r".*: warning: \[InjectMoreThanOneQualifier\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: A class can be annotated with at most one scope annotation.',
- 'patterns': [r".*: warning: \[InjectMoreThanOneScopeAnnotationOnClass\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Members shouldn\'t be annotated with @Inject if constructor is already annotated @Inject',
- 'patterns': [r".*: warning: \[InjectOnMemberAndConstructor\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Scope annotation on an interface or abstact class is not allowed',
- 'patterns': [r".*: warning: \[InjectScopeAnnotationOnInterfaceOrAbstractClass\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Scoping and qualifier annotations must have runtime retention.',
- 'patterns': [r".*: warning: \[InjectScopeOrQualifierAnnotationRetention\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Injected constructors cannot be optional nor have binding annotations',
- 'patterns': [r".*: warning: \[InjectedConstructorAnnotations\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: A standard cryptographic operation is used in a mode that is prone to vulnerabilities',
- 'patterns': [r".*: warning: \[InsecureCryptoUsage\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Invalid syntax used for a regular expression',
- 'patterns': [r".*: warning: \[InvalidPatternSyntax\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Invalid time zone identifier. TimeZone.getTimeZone(String) will silently return GMT instead of the time zone you intended.',
- 'patterns': [r".*: warning: \[InvalidTimeZoneID\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: The argument to Class#isInstance(Object) should not be a Class',
- 'patterns': [r".*: warning: \[IsInstanceOfClass\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Log tag too long, cannot exceed 23 characters.',
- 'patterns': [r".*: warning: \[IsLoggableTagLength\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- u'Java: Path implements Iterable\u003cPath>; prefer Collection\u003cPath> for clarity',
- 'patterns': [r".*: warning: \[IterablePathParameter\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: jMock tests must have a @RunWith(JMock.class) annotation, or the Mockery field must have a @Rule JUnit annotation',
- 'patterns': [r".*: warning: \[JMockTestWithoutRunWithOrRuleAnnotation\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Test method will not be run; please correct method signature (Should be public, non-static, and method name should begin with "test").',
- 'patterns': [r".*: warning: \[JUnit3TestNotRun\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: This method should be static',
- 'patterns': [r".*: warning: \[JUnit4ClassAnnotationNonStatic\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: setUp() method will not be run; please add JUnit\'s @Before annotation',
- 'patterns': [r".*: warning: \[JUnit4SetUpNotRun\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: tearDown() method will not be run; please add JUnit\'s @After annotation',
- 'patterns': [r".*: warning: \[JUnit4TearDownNotRun\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: This looks like a test method but is not run; please add @Test and @Ignore, or, if this is a helper method, reduce its visibility.',
- 'patterns': [r".*: warning: \[JUnit4TestNotRun\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: An object is tested for reference equality to itself using JUnit library.',
- 'patterns': [r".*: warning: \[JUnitAssertSameCheck\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Use of class, field, or method that is not compatible with JDK 7',
- 'patterns': [r".*: warning: \[Java7ApiChecker\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Abstract and default methods are not injectable with javax.inject.Inject',
- 'patterns': [r".*: warning: \[JavaxInjectOnAbstractMethod\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: @javax.inject.Inject cannot be put on a final field.',
- 'patterns': [r".*: warning: \[JavaxInjectOnFinalField\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: This pattern will silently corrupt certain byte sequences from the serialized protocol message. Use ByteString or byte[] directly',
- 'patterns': [r".*: warning: \[LiteByteStringUtf8\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: This method does not acquire the locks specified by its @LockMethod annotation',
- 'patterns': [r".*: warning: \[LockMethodChecker\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Prefer \'L\' to \'l\' for the suffix to long literals',
- 'patterns': [r".*: warning: \[LongLiteralLowerCaseSuffix\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Loop condition is never modified in loop body.',
- 'patterns': [r".*: warning: \[LoopConditionChecker\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Math.round(Integer) results in truncation',
- 'patterns': [r".*: warning: \[MathRoundIntLong\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Certain resources in `android.R.string` have names that do not match their content',
- 'patterns': [r".*: warning: \[MislabeledAndroidString\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Overriding method is missing a call to overridden super method',
- 'patterns': [r".*: warning: \[MissingSuperCall\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: A terminating method call is required for a test helper to have any effect.',
- 'patterns': [r".*: warning: \[MissingTestCall\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Use of "YYYY" (week year) in a date pattern without "ww" (week in year). You probably meant to use "yyyy" (year) instead.',
- 'patterns': [r".*: warning: \[MisusedWeekYear\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: A bug in Mockito will cause this test to fail at runtime with a ClassCastException',
- 'patterns': [r".*: warning: \[MockitoCast\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Missing method call for verify(mock) here',
- 'patterns': [r".*: warning: \[MockitoUsage\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Using a collection function with itself as the argument.',
- 'patterns': [r".*: warning: \[ModifyingCollectionWithItself\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: This class has more than one @Inject-annotated constructor. Please remove the @Inject annotation from all but one of them.',
- 'patterns': [r".*: warning: \[MoreThanOneInjectableConstructor\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: The result of this method must be closed.',
- 'patterns': [r".*: warning: \[MustBeClosedChecker\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: The first argument to nCopies is the number of copies, and the second is the item to copy',
- 'patterns': [r".*: warning: \[NCopiesOfChar\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: @NoAllocation was specified on this method, but something was found that would trigger an allocation',
- 'patterns': [r".*: warning: \[NoAllocation\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Static import of type uses non-canonical name',
- 'patterns': [r".*: warning: \[NonCanonicalStaticImport\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: @CompileTimeConstant parameters should be final or effectively final',
- 'patterns': [r".*: warning: \[NonFinalCompileTimeConstant\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Calling getAnnotation on an annotation that is not retained at runtime.',
- 'patterns': [r".*: warning: \[NonRuntimeAnnotation\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: This conditional expression may evaluate to null, which will result in an NPE when the result is unboxed.',
- 'patterns': [r".*: warning: \[NullTernary\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Numeric comparison using reference equality instead of value equality',
- 'patterns': [r".*: warning: \[NumericEquality\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Comparison using reference equality instead of value equality',
- 'patterns': [r".*: warning: \[OptionalEquality\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Annotations cannot be both Scope annotations and Qualifier annotations: this causes confusion when trying to use them.',
- 'patterns': [r".*: warning: \[OverlappingQualifierAndScopeAnnotation\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: This method is not annotated with @Inject, but it overrides a method that is annotated with @javax.inject.Inject. The method will not be Injected.',
- 'patterns': [r".*: warning: \[OverridesJavaxInjectableMethod\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Declaring types inside package-info.java files is very bad form',
- 'patterns': [r".*: warning: \[PackageInfo\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Method parameter has wrong package',
- 'patterns': [r".*: warning: \[ParameterPackage\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Detects classes which implement Parcelable but don\'t have CREATOR',
- 'patterns': [r".*: warning: \[ParcelableCreator\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Literal passed as first argument to Preconditions.checkNotNull() can never be null',
- 'patterns': [r".*: warning: \[PreconditionsCheckNotNull\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: First argument to `Preconditions.checkNotNull()` is a primitive rather than an object reference',
- 'patterns': [r".*: warning: \[PreconditionsCheckNotNullPrimitive\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Using ::equals or ::isInstance as an incompatible Predicate; the predicate will always return false',
- 'patterns': [r".*: warning: \[PredicateIncompatibleType\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Access to a private protocol buffer field is forbidden. This protocol buffer carries a security contract, and can only be created using an approved library. Direct access to the fields is forbidden.',
- 'patterns': [r".*: warning: \[PrivateSecurityContractProtoAccess\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Protobuf fields cannot be null.',
- 'patterns': [r".*: warning: \[ProtoFieldNullComparison\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Comparing protobuf fields of type String using reference equality',
- 'patterns': [r".*: warning: \[ProtoStringFieldReferenceEquality\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: To get the tag number of a protocol buffer enum, use getNumber() instead.',
- 'patterns': [r".*: warning: \[ProtocolBufferOrdinal\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: @Provides methods need to be declared in a Module to have any effect.',
- 'patterns': [r".*: warning: \[ProvidesMethodOutsideOfModule\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Casting a random number in the range [0.0, 1.0) to an integer or long always results in 0.',
- 'patterns': [r".*: warning: \[RandomCast\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Use Random.nextInt(int). Random.nextInt() % n can have negative results',
- 'patterns': [r".*: warning: \[RandomModInteger\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Return value of android.graphics.Rect.intersect() must be checked',
- 'patterns': [r".*: warning: \[RectIntersectReturnValueIgnored\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Use of method or class annotated with @RestrictTo',
- 'patterns': [r".*: warning: \[RestrictTo\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Check for non-whitelisted callers to RestrictedApiChecker.',
- 'patterns': [r".*: warning: \[RestrictedApiChecker\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Return value of this method must be used',
- 'patterns': [r".*: warning: \[ReturnValueIgnored\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Variable assigned to itself',
- 'patterns': [r".*: warning: \[SelfAssignment\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: An object is compared to itself',
- 'patterns': [r".*: warning: \[SelfComparison\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Testing an object for equality with itself will always be true.',
- 'patterns': [r".*: warning: \[SelfEquals\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: This method must be called with an even number of arguments.',
- 'patterns': [r".*: warning: \[ShouldHaveEvenArgs\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Comparison of a size >= 0 is always true, did you intend to check for non-emptiness?',
- 'patterns': [r".*: warning: \[SizeGreaterThanOrEqualsZero\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Static and default interface methods are not natively supported on older Android devices. ',
- 'patterns': [r".*: warning: \[StaticOrDefaultInterfaceMethod\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Calling toString on a Stream does not provide useful information',
- 'patterns': [r".*: warning: \[StreamToString\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: StringBuilder does not have a char constructor; this invokes the int constructor.',
- 'patterns': [r".*: warning: \[StringBuilderInitWithChar\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: String.substring(0) returns the original String',
- 'patterns': [r".*: warning: \[SubstringOfZero\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Suppressing "deprecated" is probably a typo for "deprecation"',
- 'patterns': [r".*: warning: \[SuppressWarningsDeprecated\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: throwIfUnchecked(knownCheckedException) is a no-op.',
- 'patterns': [r".*: warning: \[ThrowIfUncheckedKnownChecked\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Throwing \'null\' always results in a NullPointerException being thrown.',
- 'patterns': [r".*: warning: \[ThrowNull\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: isEqualTo should not be used to test an object for equality with itself; the assertion will never fail.',
- 'patterns': [r".*: warning: \[TruthSelfEquals\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Catching Throwable/Error masks failures from fail() or assert*() in the try block',
- 'patterns': [r".*: warning: \[TryFailThrowable\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Type parameter used as type qualifier',
- 'patterns': [r".*: warning: \[TypeParameterQualifier\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: This method does not acquire the locks specified by its @UnlockMethod annotation',
- 'patterns': [r".*: warning: \[UnlockMethod\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Non-generic methods should not be invoked with type arguments',
- 'patterns': [r".*: warning: \[UnnecessaryTypeArgument\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Instance created but never used',
- 'patterns': [r".*: warning: \[UnusedAnonymousClass\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: Collection is modified in place, but the result is not used',
- 'patterns': [r".*: warning: \[UnusedCollectionModifiedInPlace\] .+"]},
- {'category': 'java',
- 'severity': Severity.HIGH,
- 'description':
- 'Java: `var` should not be used as a type name.',
- 'patterns': [r".*: warning: \[VarTypeName\] .+"]},
+ java_low('Use parameter comments to document ambiguous literals',
+ [r".*: warning: \[BooleanParameter\] .+"]),
+ java_low('This class\'s name looks like a Type Parameter.',
+ [r".*: warning: \[ClassNamedLikeTypeParameter\] .+"]),
+ java_low('Field name is CONSTANT_CASE, but field is not static and final',
+ [r".*: warning: \[ConstantField\] .+"]),
+ java_low('@Multibinds is a more efficient and declarative mechanism for ensuring that a set multibinding is present in the graph.',
+ [r".*: warning: \[EmptySetMultibindingContributions\] .+"]),
+ java_low('Prefer assertThrows to ExpectedException',
+ [r".*: warning: \[ExpectedExceptionRefactoring\] .+"]),
+ java_low('This field is only assigned during initialization; consider making it final',
+ [r".*: warning: \[FieldCanBeFinal\] .+"]),
+ java_low('Fields that can be null should be annotated @Nullable',
+ [r".*: warning: \[FieldMissingNullable\] .+"]),
+ java_low('Refactors uses of the JSR 305 @Immutable to Error Prone\'s annotation',
+ [r".*: warning: \[ImmutableRefactoring\] .+"]),
+ java_low(u'Use Java\'s utility functional interfaces instead of Function\u003cA, B> for primitive types.',
+ [r".*: warning: \[LambdaFunctionalInterface\] .+"]),
+ java_low('A private method that does not reference the enclosing instance can be static',
+ [r".*: warning: \[MethodCanBeStatic\] .+"]),
+ java_low('C-style array declarations should not be used',
+ [r".*: warning: \[MixedArrayDimensions\] .+"]),
+ java_low('Variable declarations should declare only one variable',
+ [r".*: warning: \[MultiVariableDeclaration\] .+"]),
+ java_low('Source files should not contain multiple top-level class declarations',
+ [r".*: warning: \[MultipleTopLevelClasses\] .+"]),
+ java_low('Avoid having multiple unary operators acting on the same variable in a method call',
+ [r".*: warning: \[MultipleUnaryOperatorsInMethodCall\] .+"]),
+ java_low('Package names should match the directory they are declared in',
+ [r".*: warning: \[PackageLocation\] .+"]),
+ java_low('Non-standard parameter comment; prefer `/* paramName= */ arg`',
+ [r".*: warning: \[ParameterComment\] .+"]),
+ java_low('Method parameters that aren\'t checked for null shouldn\'t be annotated @Nullable',
+ [r".*: warning: \[ParameterNotNullable\] .+"]),
+ java_low('Add a private constructor to modules that will not be instantiated by Dagger.',
+ [r".*: warning: \[PrivateConstructorForNoninstantiableModule\] .+"]),
+ java_low('Utility classes (only static members) are not designed to be instantiated and should be made noninstantiable with a default constructor.',
+ [r".*: warning: \[PrivateConstructorForUtilityClass\] .+"]),
+ java_low('Unused imports',
+ [r".*: warning: \[RemoveUnusedImports\] .+"]),
+ java_low('Methods that can return null should be annotated @Nullable',
+ [r".*: warning: \[ReturnMissingNullable\] .+"]),
+ java_low('Scopes on modules have no function and will soon be an error.',
+ [r".*: warning: \[ScopeOnModule\] .+"]),
+ java_low('The default case of a switch should appear at the end of the last statement group',
+ [r".*: warning: \[SwitchDefault\] .+"]),
+ java_low('Prefer assertThrows to @Test(expected=...)',
+ [r".*: warning: \[TestExceptionRefactoring\] .+"]),
+ java_low('Unchecked exceptions do not need to be declared in the method signature.',
+ [r".*: warning: \[ThrowsUncheckedException\] .+"]),
+ java_low('Prefer assertThrows to try/fail',
+ [r".*: warning: \[TryFailRefactoring\] .+"]),
+ java_low('Type parameters must be a single letter with an optional numeric suffix, or an UpperCamelCase name followed by the letter \'T\'.',
+ [r".*: warning: \[TypeParameterNaming\] .+"]),
+ java_low('Constructors and methods with the same name should appear sequentially with no other code in between. Please re-order or re-name methods.',
+ [r".*: warning: \[UngroupedOverloads\] .+"]),
+ java_low('Unnecessary call to NullPointerTester#setDefault',
+ [r".*: warning: \[UnnecessarySetDefault\] .+"]),
+ java_low('Using static imports for types is unnecessary',
+ [r".*: warning: \[UnnecessaryStaticImport\] .+"]),
+ java_low('@Binds is a more efficient and declarative mechanism for delegating a binding.',
+ [r".*: warning: \[UseBinds\] .+"]),
+ java_low('Wildcard imports, static or otherwise, should not be used',
+ [r".*: warning: \[WildcardImport\] .+"]),
+ java_medium('Method reference is ambiguous',
+ [r".*: warning: \[AmbiguousMethodReference\] .+"]),
+ java_medium('This method passes a pair of parameters through to String.format, but the enclosing method wasn\'t annotated @FormatMethod. Doing so gives compile-time rather than run-time protection against malformed format strings.',
+ [r".*: warning: \[AnnotateFormatMethod\] .+"]),
+ java_medium('Annotations should be positioned after Javadocs, but before modifiers..',
+ [r".*: warning: \[AnnotationPosition\] .+"]),
+ java_medium('Arguments are in the wrong order or could be commented for clarity.',
+ [r".*: warning: \[ArgumentSelectionDefectChecker\] .+"]),
+ java_medium('Arrays do not override equals() or hashCode, so comparisons will be done on reference equality only. If neither deduplication nor lookup are needed, consider using a List instead. Otherwise, use IdentityHashMap/Set, a Map from a library that handles object arrays, or an Iterable/List of pairs.',
+ [r".*: warning: \[ArrayAsKeyOfSetOrMap\] .+"]),
+ java_medium('Arguments are swapped in assertEquals-like call',
+ [r".*: warning: \[AssertEqualsArgumentOrderChecker\] .+"]),
+ java_medium('Assertions may be disabled at runtime and do not guarantee that execution will halt here; consider throwing an exception instead',
+ [r".*: warning: \[AssertFalse\] .+"]),
+ java_medium('The lambda passed to assertThrows should contain exactly one statement',
+ [r".*: warning: \[AssertThrowsMultipleStatements\] .+"]),
+ java_medium('This assertion throws an AssertionError if it fails, which will be caught by an enclosing try block.',
+ [r".*: warning: \[AssertionFailureIgnored\] .+"]),
+ java_medium('@AssistedInject and @Inject should not be used on different constructors in the same class.',
+ [r".*: warning: \[AssistedInjectAndInjectOnConstructors\] .+"]),
+ java_medium('Make toString(), hashCode() and equals() final in AutoValue classes, so it is clear to readers that AutoValue is not overriding them',
+ [r".*: warning: \[AutoValueFinalMethods\] .+"]),
+ java_medium('Classes that implement Annotation must override equals and hashCode. Consider using AutoAnnotation instead of implementing Annotation by hand.',
+ [r".*: warning: \[BadAnnotationImplementation\] .+"]),
+ java_medium('Possible sign flip from narrowing conversion',
+ [r".*: warning: \[BadComparable\] .+"]),
+ java_medium('Importing nested classes/static methods/static fields with commonly-used names can make code harder to read, because it may not be clear from the context exactly which type is being referred to. Qualifying the name with that of the containing class can make the code clearer.',
+ [r".*: warning: \[BadImport\] .+"]),
+ java_medium('instanceof used in a way that is equivalent to a null check.',
+ [r".*: warning: \[BadInstanceof\] .+"]),
+ java_medium('BigDecimal#equals has surprising behavior: it also compares scale.',
+ [r".*: warning: \[BigDecimalEquals\] .+"]),
+ java_medium('new BigDecimal(double) loses precision in this case.',
+ [r".*: warning: \[BigDecimalLiteralDouble\] .+"]),
+ java_medium('A call to Binder.clearCallingIdentity() should be followed by Binder.restoreCallingIdentity() in a finally block. Otherwise the wrong Binder identity may be used by subsequent code.',
+ [r".*: warning: \[BinderIdentityRestoredDangerously\] .+"]),
+ java_medium('This code declares a binding for a common value type without a Qualifier annotation.',
+ [r".*: warning: \[BindingToUnqualifiedCommonType\] .+"]),
+ java_medium('valueOf or autoboxing provides better time and space performance',
+ [r".*: warning: \[BoxedPrimitiveConstructor\] .+"]),
+ java_medium('ByteBuffer.array() shouldn\'t be called unless ByteBuffer.arrayOffset() is used or if the ByteBuffer was initialized using ByteBuffer.wrap() or ByteBuffer.allocate().',
+ [r".*: warning: \[ByteBufferBackingArray\] .+"]),
+ java_medium('Mockito cannot mock final classes',
+ [r".*: warning: \[CannotMockFinalClass\] .+"]),
+ java_medium('Duration can be expressed more clearly with different units',
+ [r".*: warning: \[CanonicalDuration\] .+"]),
+ java_medium('Logging or rethrowing exceptions should usually be preferred to catching and calling printStackTrace',
+ [r".*: warning: \[CatchAndPrintStackTrace\] .+"]),
+ java_medium('Ignoring exceptions and calling fail() is unnecessary, and makes test output less useful',
+ [r".*: warning: \[CatchFail\] .+"]),
+ java_medium('Inner class is non-static but does not reference enclosing class',
+ [r".*: warning: \[ClassCanBeStatic\] .+"]),
+ java_medium('Class.newInstance() bypasses exception checking; prefer getDeclaredConstructor().newInstance()',
+ [r".*: warning: \[ClassNewInstance\] .+"]),
+ java_medium('Providing Closeable resources makes their lifecycle unclear',
+ [r".*: warning: \[CloseableProvides\] .+"]),
+ java_medium('The type of the array parameter of Collection.toArray needs to be compatible with the array type',
+ [r".*: warning: \[CollectionToArraySafeParameter\] .+"]),
+ java_medium('Collector.of() should not use state',
+ [r".*: warning: \[CollectorShouldNotUseState\] .+"]),
+ java_medium('Class should not implement both `Comparable` and `Comparator`',
+ [r".*: warning: \[ComparableAndComparator\] .+"]),
+ java_medium('Constructors should not invoke overridable methods.',
+ [r".*: warning: \[ConstructorInvokesOverridable\] .+"]),
+ java_medium('Constructors should not pass the \'this\' reference out in method invocations, since the object may not be fully constructed.',
+ [r".*: warning: \[ConstructorLeaksThis\] .+"]),
+ java_medium('DateFormat is not thread-safe, and should not be used as a constant field.',
+ [r".*: warning: \[DateFormatConstant\] .+"]),
+ java_medium('Implicit use of the platform default charset, which can result in differing behaviour between JVM executions or incorrect behavior if the encoding of the data source doesn\'t match expectations.',
+ [r".*: warning: \[DefaultCharset\] .+"]),
+ java_medium('Avoid deprecated Thread methods; read the method\'s javadoc for details.',
+ [r".*: warning: \[DeprecatedThreadMethods\] .+"]),
+ java_medium('Prefer collection factory methods or builders to the double-brace initialization pattern.',
+ [r".*: warning: \[DoubleBraceInitialization\] .+"]),
+ java_medium('Double-checked locking on non-volatile fields is unsafe',
+ [r".*: warning: \[DoubleCheckedLocking\] .+"]),
+ java_medium('Empty top-level type declaration',
+ [r".*: warning: \[EmptyTopLevelDeclaration\] .+"]),
+ java_medium('equals() implementation may throw NullPointerException when given null',
+ [r".*: warning: \[EqualsBrokenForNull\] .+"]),
+ java_medium('Overriding Object#equals in a non-final class by using getClass rather than instanceof breaks substitutability of subclasses.',
+ [r".*: warning: \[EqualsGetClass\] .+"]),
+ java_medium('Classes that override equals should also override hashCode.',
+ [r".*: warning: \[EqualsHashCode\] .+"]),
+ java_medium('An equality test between objects with incompatible types always returns false',
+ [r".*: warning: \[EqualsIncompatibleType\] .+"]),
+ java_medium('The contract of #equals states that it should return false for incompatible types, while this implementation may throw ClassCastException.',
+ [r".*: warning: \[EqualsUnsafeCast\] .+"]),
+ java_medium('Implementing #equals by just comparing hashCodes is fragile. Hashes collide frequently, and this will lead to false positives in #equals.',
+ [r".*: warning: \[EqualsUsingHashCode\] .+"]),
+ java_medium('Calls to ExpectedException#expect should always be followed by exactly one statement.',
+ [r".*: warning: \[ExpectedExceptionChecker\] .+"]),
+ java_medium('When only using JUnit Assert\'s static methods, you should import statically instead of extending.',
+ [r".*: warning: \[ExtendingJUnitAssert\] .+"]),
+ java_medium('Switch case may fall through',
+ [r".*: warning: \[FallThrough\] .+"]),
+ java_medium('If you return or throw from a finally, then values returned or thrown from the try-catch block will be ignored. Consider using try-with-resources instead.',
+ [r".*: warning: \[Finally\] .+"]),
+ java_medium('Use parentheses to make the precedence explicit',
+ [r".*: warning: \[FloatCast\] .+"]),
+ java_medium('This fuzzy equality check is using a tolerance less than the gap to the next number. You may want a less restrictive tolerance, or to assert equality.',
+ [r".*: warning: \[FloatingPointAssertionWithinEpsilon\] .+"]),
+ java_medium('Floating point literal loses precision',
+ [r".*: warning: \[FloatingPointLiteralPrecision\] .+"]),
+ java_medium('Classes extending PreferenceActivity must implement isValidFragment such that it does not unconditionally return true to prevent vulnerability to fragment injection attacks.',
+ [r".*: warning: \[FragmentInjection\] .+"]),
+ java_medium('Subclasses of Fragment must be instantiable via Class#newInstance(): the class must be public, static and have a public nullary constructor',
+ [r".*: warning: \[FragmentNotInstantiable\] .+"]),
+ java_medium('Overloads will be ambiguous when passing lambda arguments',
+ [r".*: warning: \[FunctionalInterfaceClash\] .+"]),
+ java_medium('Return value of methods returning Future must be checked. Ignoring returned Futures suppresses exceptions thrown from the code that completes the Future.',
+ [r".*: warning: \[FutureReturnValueIgnored\] .+"]),
+ java_medium('Calling getClass() on an enum may return a subclass of the enum type',
+ [r".*: warning: \[GetClassOnEnum\] .+"]),
+ java_medium('Hardcoded reference to /sdcard',
+ [r".*: warning: \[HardCodedSdCardPath\] .+"]),
+ java_medium('Hiding fields of superclasses may cause confusion and errors',
+ [r".*: warning: \[HidingField\] .+"]),
+ java_medium('Annotations should always be immutable',
+ [r".*: warning: \[ImmutableAnnotationChecker\] .+"]),
+ java_medium('Enums should always be immutable',
+ [r".*: warning: \[ImmutableEnumChecker\] .+"]),
+ java_medium('This annotation has incompatible modifiers as specified by its @IncompatibleModifiers annotation',
+ [r".*: warning: \[IncompatibleModifiers\] .+"]),
+ java_medium('It is confusing to have a field and a parameter under the same scope that differ only in capitalization.',
+ [r".*: warning: \[InconsistentCapitalization\] .+"]),
+ java_medium('Including fields in hashCode which are not compared in equals violates the contract of hashCode.',
+ [r".*: warning: \[InconsistentHashCode\] .+"]),
+ java_medium('The ordering of parameters in overloaded methods should be as consistent as possible (when viewed from left to right)',
+ [r".*: warning: \[InconsistentOverloads\] .+"]),
+ java_medium('This for loop increments the same variable in the header and in the body',
+ [r".*: warning: \[IncrementInForLoopAndHeader\] .+"]),
+ java_medium('Constructors on abstract classes are never directly @Injected, only the constructors of their subclasses can be @Inject\'ed.',
+ [r".*: warning: \[InjectOnConstructorOfAbstractClass\] .+"]),
+ java_medium('Please also override int read(byte[], int, int), otherwise multi-byte reads from this input stream are likely to be slow.',
+ [r".*: warning: \[InputStreamSlowMultibyteRead\] .+"]),
+ java_medium('Casting inside an if block should be plausibly consistent with the instanceof type',
+ [r".*: warning: \[InstanceOfAndCastMatchWrongType\] .+"]),
+ java_medium('Expression of type int may overflow before being assigned to a long',
+ [r".*: warning: \[IntLongMath\] .+"]),
+ java_medium('This @param tag doesn\'t refer to a parameter of the method.',
+ [r".*: warning: \[InvalidParam\] .+"]),
+ java_medium('This tag is invalid.',
+ [r".*: warning: \[InvalidTag\] .+"]),
+ java_medium('The documented method doesn\'t actually throw this checked exception.',
+ [r".*: warning: \[InvalidThrows\] .+"]),
+ java_medium('Class should not implement both `Iterable` and `Iterator`',
+ [r".*: warning: \[IterableAndIterator\] .+"]),
+ java_medium('Floating-point comparison without error tolerance',
+ [r".*: warning: \[JUnit3FloatingPointComparisonWithoutDelta\] .+"]),
+ java_medium('Some JUnit4 construct cannot be used in a JUnit3 context. Convert your class to JUnit4 style to use them.',
+ [r".*: warning: \[JUnit4ClassUsedInJUnit3\] .+"]),
+ java_medium('Test class inherits from JUnit 3\'s TestCase but has JUnit 4 @Test annotations.',
+ [r".*: warning: \[JUnitAmbiguousTestClass\] .+"]),
+ java_medium('Never reuse class names from java.lang',
+ [r".*: warning: \[JavaLangClash\] .+"]),
+ java_medium('Suggests alternatives to obsolete JDK classes.',
+ [r".*: warning: \[JdkObsolete\] .+"]),
+ java_medium('Calls to Lock#lock should be immediately followed by a try block which releases the lock.',
+ [r".*: warning: \[LockNotBeforeTry\] .+"]),
+ java_medium('Assignment where a boolean expression was expected; use == if this assignment wasn\'t expected or add parentheses for clarity.',
+ [r".*: warning: \[LogicalAssignment\] .+"]),
+ java_medium('Math.abs does not always give a positive result. Please consider other methods for positive random numbers.',
+ [r".*: warning: \[MathAbsoluteRandom\] .+"]),
+ java_medium('Switches on enum types should either handle all values, or have a default case.',
+ [r".*: warning: \[MissingCasesInEnumSwitch\] .+"]),
+ java_medium('The Google Java Style Guide requires that each switch statement includes a default statement group, even if it contains no code. (This requirement is lifted for any switch statement that covers all values of an enum.)',
+ [r".*: warning: \[MissingDefault\] .+"]),
+ java_medium('Not calling fail() when expecting an exception masks bugs',
+ [r".*: warning: \[MissingFail\] .+"]),
+ java_medium('method overrides method in supertype; expected @Override',
+ [r".*: warning: \[MissingOverride\] .+"]),
+ java_medium('A collection or proto builder was created, but its values were never accessed.',
+ [r".*: warning: \[ModifiedButNotUsed\] .+"]),
+ java_medium('Modifying a collection while iterating over it in a loop may cause a ConcurrentModificationException to be thrown.',
+ [r".*: warning: \[ModifyCollectionInEnhancedForLoop\] .+"]),
+ java_medium('Multiple calls to either parallel or sequential are unnecessary and cause confusion.',
+ [r".*: warning: \[MultipleParallelOrSequentialCalls\] .+"]),
+ java_medium('Constant field declarations should use the immutable type (such as ImmutableList) instead of the general collection interface type (such as List)',
+ [r".*: warning: \[MutableConstantField\] .+"]),
+ java_medium('Method return type should use the immutable type (such as ImmutableList) instead of the general collection interface type (such as List)',
+ [r".*: warning: \[MutableMethodReturnType\] .+"]),
+ java_medium('Compound assignments may hide dangerous casts',
+ [r".*: warning: \[NarrowingCompoundAssignment\] .+"]),
+ java_medium('Nested instanceOf conditions of disjoint types create blocks of code that never execute',
+ [r".*: warning: \[NestedInstanceOfConditions\] .+"]),
+ java_medium('Instead of returning a functional type, return the actual type that the returned function would return and use lambdas at use site.',
+ [r".*: warning: \[NoFunctionalReturnType\] .+"]),
+ java_medium('This update of a volatile variable is non-atomic',
+ [r".*: warning: \[NonAtomicVolatileUpdate\] .+"]),
+ java_medium('Static import of member uses non-canonical name',
+ [r".*: warning: \[NonCanonicalStaticMemberImport\] .+"]),
+ java_medium('equals method doesn\'t override Object.equals',
+ [r".*: warning: \[NonOverridingEquals\] .+"]),
+ java_medium('Constructors should not be annotated with @Nullable since they cannot return null',
+ [r".*: warning: \[NullableConstructor\] .+"]),
+ java_medium('Dereference of possibly-null value',
+ [r".*: warning: \[NullableDereference\] .+"]),
+ java_medium('@Nullable should not be used for primitive types since they cannot be null',
+ [r".*: warning: \[NullablePrimitive\] .+"]),
+ java_medium('void-returning methods should not be annotated with @Nullable, since they cannot return null',
+ [r".*: warning: \[NullableVoid\] .+"]),
+ java_medium('Calling toString on Objects that don\'t override toString() doesn\'t provide useful information',
+ [r".*: warning: \[ObjectToString\] .+"]),
+ java_medium('Objects.hashCode(Object o) should not be passed a primitive value',
+ [r".*: warning: \[ObjectsHashCodePrimitive\] .+"]),
+ java_medium('Use grouping parenthesis to make the operator precedence explicit',
+ [r".*: warning: \[OperatorPrecedence\] .+"]),
+ java_medium('One should not call optional.get() inside an if statement that checks !optional.isPresent',
+ [r".*: warning: \[OptionalNotPresent\] .+"]),
+ java_medium('String literal contains format specifiers, but is not passed to a format method',
+ [r".*: warning: \[OrphanedFormatString\] .+"]),
+ java_medium('To return a custom message with a Throwable class, one should override getMessage() instead of toString() for Throwable.',
+ [r".*: warning: \[OverrideThrowableToString\] .+"]),
+ java_medium('Varargs doesn\'t agree for overridden method',
+ [r".*: warning: \[Overrides\] .+"]),
+ java_medium('This method is not annotated with @Inject, but it overrides a method that is annotated with @com.google.inject.Inject. Guice will inject this method, and it is recommended to annotate it explicitly.',
+ [r".*: warning: \[OverridesGuiceInjectableMethod\] .+"]),
+ java_medium('Detects `/* name= */`-style comments on actual parameters where the name doesn\'t match the formal parameter',
+ [r".*: warning: \[ParameterName\] .+"]),
+ java_medium('Preconditions only accepts the %s placeholder in error message strings',
+ [r".*: warning: \[PreconditionsInvalidPlaceholder\] .+"]),
+ java_medium('Passing a primitive array to a varargs method is usually wrong',
+ [r".*: warning: \[PrimitiveArrayPassedToVarargsMethod\] .+"]),
+ java_medium('A field on a protocol buffer was set twice in the same chained expression.',
+ [r".*: warning: \[ProtoRedundantSet\] .+"]),
+ java_medium('Protos should not be used as a key to a map, in a set, or in a contains method on a descendant of a collection. Protos have non deterministic ordering and proto equality is deep, which is a performance issue.',
+ [r".*: warning: \[ProtosAsKeyOfSetOrMap\] .+"]),
+ java_medium('BugChecker has incorrect ProvidesFix tag, please update',
+ [r".*: warning: \[ProvidesFix\] .+"]),
+ java_medium('Qualifiers/Scope annotations on @Inject methods don\'t have any effect. Move the qualifier annotation to the binding location.',
+ [r".*: warning: \[QualifierOrScopeOnInjectMethod\] .+"]),
+ java_medium('Injection frameworks currently don\'t understand Qualifiers in TYPE_PARAMETER or TYPE_USE contexts.',
+ [r".*: warning: \[QualifierWithTypeUse\] .+"]),
+ java_medium('reachabilityFence should always be called inside a finally block',
+ [r".*: warning: \[ReachabilityFenceUsage\] .+"]),
+ java_medium('Thrown exception is a subtype of another',
+ [r".*: warning: \[RedundantThrows\] .+"]),
+ java_medium('Comparison using reference equality instead of value equality',
+ [r".*: warning: \[ReferenceEquality\] .+"]),
+ java_medium('This annotation is missing required modifiers as specified by its @RequiredModifiers annotation',
+ [r".*: warning: \[RequiredModifiers\] .+"]),
+ java_medium('Void methods should not have a @return tag.',
+ [r".*: warning: \[ReturnFromVoid\] .+"]),
+ java_medium(u'Prefer the short-circuiting boolean operators \u0026\u0026 and || to \u0026 and |.',
+ [r".*: warning: \[ShortCircuitBoolean\] .+"]),
+ java_medium('Writes to static fields should not be guarded by instance locks',
+ [r".*: warning: \[StaticGuardedByInstance\] .+"]),
+ java_medium('A static variable or method should be qualified with a class name, not expression',
+ [r".*: warning: \[StaticQualifiedUsingExpression\] .+"]),
+ java_medium('Streams that encapsulate a closeable resource should be closed using try-with-resources',
+ [r".*: warning: \[StreamResourceLeak\] .+"]),
+ java_medium('String comparison using reference equality instead of value equality',
+ [r".*: warning: \[StringEquality\] .+"]),
+ java_medium('String.split(String) has surprising behavior',
+ [r".*: warning: \[StringSplitter\] .+"]),
+ java_medium('SWIG generated code that can\'t call a C++ destructor will leak memory',
+ [r".*: warning: \[SwigMemoryLeak\] .+"]),
+ java_medium('Synchronizing on non-final fields is not safe: if the field is ever updated, different threads may end up locking on different objects.',
+ [r".*: warning: \[SynchronizeOnNonFinalField\] .+"]),
+ java_medium('Code that contains System.exit() is untestable.',
+ [r".*: warning: \[SystemExitOutsideMain\] .+"]),
+ java_medium('Using @Test(expected=...) is discouraged, since the test will pass if *any* statement in the test method throws the expected exception',
+ [r".*: warning: \[TestExceptionChecker\] .+"]),
+ java_medium('Thread.join needs to be surrounded by a loop until it succeeds, as in Uninterruptibles.joinUninterruptibly.',
+ [r".*: warning: \[ThreadJoinLoop\] .+"]),
+ java_medium('ThreadLocals should be stored in static fields',
+ [r".*: warning: \[ThreadLocalUsage\] .+"]),
+ java_medium('Relying on the thread scheduler is discouraged; see Effective Java Item 72 (2nd edition) / 84 (3rd edition).',
+ [r".*: warning: \[ThreadPriorityCheck\] .+"]),
+ java_medium('Three-letter time zone identifiers are deprecated, may be ambiguous, and might not do what you intend; the full IANA time zone ID should be used instead.',
+ [r".*: warning: \[ThreeLetterTimeZoneID\] .+"]),
+ java_medium('An implementation of Object.toString() should never return null.',
+ [r".*: warning: \[ToStringReturnsNull\] .+"]),
+ java_medium('The actual and expected values appear to be swapped, which results in poor assertion failure messages. The actual value should come first.',
+ [r".*: warning: \[TruthAssertExpected\] .+"]),
+ java_medium('Truth Library assert is called on a constant.',
+ [r".*: warning: \[TruthConstantAsserts\] .+"]),
+ java_medium('Argument is not compatible with the subject\'s type.',
+ [r".*: warning: \[TruthIncompatibleType\] .+"]),
+ java_medium('Type parameter declaration shadows another named type',
+ [r".*: warning: \[TypeNameShadowing\] .+"]),
+ java_medium('Type parameter declaration overrides another type parameter already declared',
+ [r".*: warning: \[TypeParameterShadowing\] .+"]),
+ java_medium('Declaring a type parameter that is only used in the return type is a misuse of generics: operations on the type parameter are unchecked, it hides unsafe casts at invocations of the method, and it interacts badly with method overload resolution.',
+ [r".*: warning: \[TypeParameterUnusedInFormals\] .+"]),
+ java_medium('Avoid hash-based containers of java.net.URL--the containers rely on equals() and hashCode(), which cause java.net.URL to make blocking internet connections.',
+ [r".*: warning: \[URLEqualsHashCode\] .+"]),
+ java_medium('Collection, Iterable, Multimap, and Queue do not have well-defined equals behavior',
+ [r".*: warning: \[UndefinedEquals\] .+"]),
+ java_medium('Switch handles all enum values: an explicit default case is unnecessary and defeats error checking for non-exhaustive switches.',
+ [r".*: warning: \[UnnecessaryDefaultInEnumSwitch\] .+"]),
+ java_medium('Unnecessary use of grouping parentheses',
+ [r".*: warning: \[UnnecessaryParentheses\] .+"]),
+ java_medium('Finalizer may run before native code finishes execution',
+ [r".*: warning: \[UnsafeFinalization\] .+"]),
+ java_medium('Prefer `asSubclass` instead of casting the result of `newInstance`, to detect classes of incorrect type before invoking their constructors.This way, if the class is of the incorrect type,it will throw an exception before invoking its constructor.',
+ [r".*: warning: \[UnsafeReflectiveConstructionCast\] .+"]),
+ java_medium('Unsynchronized method overrides a synchronized method.',
+ [r".*: warning: \[UnsynchronizedOverridesSynchronized\] .+"]),
+ java_medium('Unused.',
+ [r".*: warning: \[Unused\] .+"]),
+ java_medium('This catch block catches an exception and re-throws another, but swallows the caught exception rather than setting it as a cause. This can make debugging harder.',
+ [r".*: warning: \[UnusedException\] .+"]),
+ java_medium('Java assert is used in test. For testing purposes Assert.* matchers should be used.',
+ [r".*: warning: \[UseCorrectAssertInTests\] .+"]),
+ java_medium('Non-constant variable missing @Var annotation',
+ [r".*: warning: \[Var\] .+"]),
+ java_medium('variableName and type with the same name would refer to the static field instead of the class',
+ [r".*: warning: \[VariableNameSameAsType\] .+"]),
+ java_medium('Because of spurious wakeups, Object.wait() and Condition.await() must always be called in a loop',
+ [r".*: warning: \[WaitNotInLoop\] .+"]),
+ java_medium('A wakelock acquired with a timeout may be released by the system before calling `release`, even after checking `isHeld()`. If so, it will throw a RuntimeException. Please wrap in a try/catch block.',
+ [r".*: warning: \[WakelockReleasedDangerously\] .+"]),
+ java_high('AndroidInjection.inject() should always be invoked before calling super.lifecycleMethod()',
+ [r".*: warning: \[AndroidInjectionBeforeSuper\] .+"]),
+ java_high('Use of class, field, or method that is not compatible with legacy Android devices',
+ [r".*: warning: \[AndroidJdkLibsChecker\] .+"]),
+ java_high('Reference equality used to compare arrays',
+ [r".*: warning: \[ArrayEquals\] .+"]),
+ java_high('Arrays.fill(Object[], Object) called with incompatible types.',
+ [r".*: warning: \[ArrayFillIncompatibleType\] .+"]),
+ java_high('hashcode method on array does not hash array contents',
+ [r".*: warning: \[ArrayHashCode\] .+"]),
+ java_high('Calling toString on an array does not provide useful information',
+ [r".*: warning: \[ArrayToString\] .+"]),
+ java_high('Arrays.asList does not autobox primitive arrays, as one might expect.',
+ [r".*: warning: \[ArraysAsListPrimitiveArray\] .+"]),
+ java_high('@AssistedInject and @Inject cannot be used on the same constructor.',
+ [r".*: warning: \[AssistedInjectAndInjectOnSameConstructor\] .+"]),
+ java_high('AsyncCallable should not return a null Future, only a Future whose result is null.',
+ [r".*: warning: \[AsyncCallableReturnsNull\] .+"]),
+ java_high('AsyncFunction should not return a null Future, only a Future whose result is null.',
+ [r".*: warning: \[AsyncFunctionReturnsNull\] .+"]),
+ java_high('@AutoFactory and @Inject should not be used in the same type.',
+ [r".*: warning: \[AutoFactoryAtInject\] .+"]),
+ java_high('Arguments to AutoValue constructor are in the wrong order',
+ [r".*: warning: \[AutoValueConstructorOrderChecker\] .+"]),
+ java_high('Shift by an amount that is out of range',
+ [r".*: warning: \[BadShiftAmount\] .+"]),
+ java_high('Object serialized in Bundle may have been flattened to base type.',
+ [r".*: warning: \[BundleDeserializationCast\] .+"]),
+ java_high('The called constructor accepts a parameter with the same name and type as one of its caller\'s parameters, but its caller doesn\'t pass that parameter to it. It\'s likely that it was intended to.',
+ [r".*: warning: \[ChainingConstructorIgnoresParameter\] .+"]),
+ java_high('Ignored return value of method that is annotated with @CheckReturnValue',
+ [r".*: warning: \[CheckReturnValue\] .+"]),
+ java_high('The source file name should match the name of the top-level class it contains',
+ [r".*: warning: \[ClassName\] .+"]),
+ java_high('Incompatible type as argument to Object-accepting Java collections method',
+ [r".*: warning: \[CollectionIncompatibleType\] .+"]),
+ java_high(u'Implementing \'Comparable\u003cT>\' where T is not compatible with the implementing class.',
+ [r".*: warning: \[ComparableType\] .+"]),
+ java_high('this == null is always false, this != null is always true',
+ [r".*: warning: \[ComparingThisWithNull\] .+"]),
+ java_high('This comparison method violates the contract',
+ [r".*: warning: \[ComparisonContractViolated\] .+"]),
+ java_high('Comparison to value that is out of range for the compared type',
+ [r".*: warning: \[ComparisonOutOfRange\] .+"]),
+ java_high('@CompatibleWith\'s value is not a type argument.',
+ [r".*: warning: \[CompatibleWithAnnotationMisuse\] .+"]),
+ java_high('Non-compile-time constant expression passed to parameter with @CompileTimeConstant type annotation.',
+ [r".*: warning: \[CompileTimeConstant\] .+"]),
+ java_high('Non-trivial compile time constant boolean expressions shouldn\'t be used.',
+ [r".*: warning: \[ComplexBooleanConstant\] .+"]),
+ java_high('A conditional expression with numeric operands of differing types will perform binary numeric promotion of the operands; when these operands are of reference types, the expression\'s result may not be of the expected type.',
+ [r".*: warning: \[ConditionalExpressionNumericPromotion\] .+"]),
+ java_high('Compile-time constant expression overflows',
+ [r".*: warning: \[ConstantOverflow\] .+"]),
+ java_high('Dagger @Provides methods may not return null unless annotated with @Nullable',
+ [r".*: warning: \[DaggerProvidesNull\] .+"]),
+ java_high('Exception created but not thrown',
+ [r".*: warning: \[DeadException\] .+"]),
+ java_high('Thread created but not started',
+ [r".*: warning: \[DeadThread\] .+"]),
+ java_high('Deprecated item is not annotated with @Deprecated',
+ [r".*: warning: \[DepAnn\] .+"]),
+ java_high('Division by integer literal zero',
+ [r".*: warning: \[DivZero\] .+"]),
+ java_high('This method should not be called.',
+ [r".*: warning: \[DoNotCall\] .+"]),
+ java_high('Empty statement after if',
+ [r".*: warning: \[EmptyIf\] .+"]),
+ java_high('== NaN always returns false; use the isNaN methods instead',
+ [r".*: warning: \[EqualsNaN\] .+"]),
+ java_high('== must be used in equals method to check equality to itself or an infinite loop will occur.',
+ [r".*: warning: \[EqualsReference\] .+"]),
+ java_high('Comparing different pairs of fields/getters in an equals implementation is probably a mistake.',
+ [r".*: warning: \[EqualsWrongThing\] .+"]),
+ java_high('Method annotated @ForOverride must be protected or package-private and only invoked from declaring class, or from an override of the method',
+ [r".*: warning: \[ForOverride\] .+"]),
+ java_high('Invalid printf-style format string',
+ [r".*: warning: \[FormatString\] .+"]),
+ java_high('Invalid format string passed to formatting method.',
+ [r".*: warning: \[FormatStringAnnotation\] .+"]),
+ java_high('Casting a lambda to this @FunctionalInterface can cause a behavior change from casting to a functional superinterface, which is surprising to users. Prefer decorator methods to this surprising behavior.',
+ [r".*: warning: \[FunctionalInterfaceMethodChanged\] .+"]),
+ java_high('Futures.getChecked requires a checked exception type with a standard constructor.',
+ [r".*: warning: \[FuturesGetCheckedIllegalExceptionType\] .+"]),
+ java_high('DoubleMath.fuzzyEquals should never be used in an Object.equals() method',
+ [r".*: warning: \[FuzzyEqualsShouldNotBeUsedInEqualsMethod\] .+"]),
+ java_high('Calling getClass() on an annotation may return a proxy class',
+ [r".*: warning: \[GetClassOnAnnotation\] .+"]),
+ java_high('Calling getClass() on an object of type Class returns the Class object for java.lang.Class; you probably meant to operate on the object directly',
+ [r".*: warning: \[GetClassOnClass\] .+"]),
+ java_high('Checks for unguarded accesses to fields and methods with @GuardedBy annotations',
+ [r".*: warning: \[GuardedBy\] .+"]),
+ java_high('Scope annotation on implementation class of AssistedInject factory is not allowed',
+ [r".*: warning: \[GuiceAssistedInjectScoping\] .+"]),
+ java_high('A constructor cannot have two @Assisted parameters of the same type unless they are disambiguated with named @Assisted annotations.',
+ [r".*: warning: \[GuiceAssistedParameters\] .+"]),
+ java_high('Although Guice allows injecting final fields, doing so is disallowed because the injected value may not be visible to other threads.',
+ [r".*: warning: \[GuiceInjectOnFinalField\] .+"]),
+ java_high('contains() is a legacy method that is equivalent to containsValue()',
+ [r".*: warning: \[HashtableContains\] .+"]),
+ java_high('A binary expression where both operands are the same is usually incorrect.',
+ [r".*: warning: \[IdentityBinaryExpression\] .+"]),
+ java_high('Type declaration annotated with @Immutable is not immutable',
+ [r".*: warning: \[Immutable\] .+"]),
+ java_high('Modifying an immutable collection is guaranteed to throw an exception and leave the collection unmodified',
+ [r".*: warning: \[ImmutableModification\] .+"]),
+ java_high('Passing argument to a generic method with an incompatible type.',
+ [r".*: warning: \[IncompatibleArgumentType\] .+"]),
+ java_high('The first argument to indexOf is a Unicode code point, and the second is the index to start the search from',
+ [r".*: warning: \[IndexOfChar\] .+"]),
+ java_high('Conditional expression in varargs call contains array and non-array arguments',
+ [r".*: warning: \[InexactVarargsConditional\] .+"]),
+ java_high('This method always recurses, and will cause a StackOverflowError',
+ [r".*: warning: \[InfiniteRecursion\] .+"]),
+ java_high('A scoping annotation\'s Target should include TYPE and METHOD.',
+ [r".*: warning: \[InjectInvalidTargetingOnScopingAnnotation\] .+"]),
+ java_high('Using more than one qualifier annotation on the same element is not allowed.',
+ [r".*: warning: \[InjectMoreThanOneQualifier\] .+"]),
+ java_high('A class can be annotated with at most one scope annotation.',
+ [r".*: warning: \[InjectMoreThanOneScopeAnnotationOnClass\] .+"]),
+ java_high('Members shouldn\'t be annotated with @Inject if constructor is already annotated @Inject',
+ [r".*: warning: \[InjectOnMemberAndConstructor\] .+"]),
+ java_high('Scope annotation on an interface or abstact class is not allowed',
+ [r".*: warning: \[InjectScopeAnnotationOnInterfaceOrAbstractClass\] .+"]),
+ java_high('Scoping and qualifier annotations must have runtime retention.',
+ [r".*: warning: \[InjectScopeOrQualifierAnnotationRetention\] .+"]),
+ java_high('Injected constructors cannot be optional nor have binding annotations',
+ [r".*: warning: \[InjectedConstructorAnnotations\] .+"]),
+ java_high('A standard cryptographic operation is used in a mode that is prone to vulnerabilities',
+ [r".*: warning: \[InsecureCryptoUsage\] .+"]),
+ java_high('Invalid syntax used for a regular expression',
+ [r".*: warning: \[InvalidPatternSyntax\] .+"]),
+ java_high('Invalid time zone identifier. TimeZone.getTimeZone(String) will silently return GMT instead of the time zone you intended.',
+ [r".*: warning: \[InvalidTimeZoneID\] .+"]),
+ java_high('The argument to Class#isInstance(Object) should not be a Class',
+ [r".*: warning: \[IsInstanceOfClass\] .+"]),
+ java_high('Log tag too long, cannot exceed 23 characters.',
+ [r".*: warning: \[IsLoggableTagLength\] .+"]),
+ java_high(u'Path implements Iterable\u003cPath>; prefer Collection\u003cPath> for clarity',
+ [r".*: warning: \[IterablePathParameter\] .+"]),
+ java_high('jMock tests must have a @RunWith(JMock.class) annotation, or the Mockery field must have a @Rule JUnit annotation',
+ [r".*: warning: \[JMockTestWithoutRunWithOrRuleAnnotation\] .+"]),
+ java_high('Test method will not be run; please correct method signature (Should be public, non-static, and method name should begin with "test").',
+ [r".*: warning: \[JUnit3TestNotRun\] .+"]),
+ java_high('This method should be static',
+ [r".*: warning: \[JUnit4ClassAnnotationNonStatic\] .+"]),
+ java_high('setUp() method will not be run; please add JUnit\'s @Before annotation',
+ [r".*: warning: \[JUnit4SetUpNotRun\] .+"]),
+ java_high('tearDown() method will not be run; please add JUnit\'s @After annotation',
+ [r".*: warning: \[JUnit4TearDownNotRun\] .+"]),
+ java_high('This looks like a test method but is not run; please add @Test and @Ignore, or, if this is a helper method, reduce its visibility.',
+ [r".*: warning: \[JUnit4TestNotRun\] .+"]),
+ java_high('An object is tested for reference equality to itself using JUnit library.',
+ [r".*: warning: \[JUnitAssertSameCheck\] .+"]),
+ java_high('Use of class, field, or method that is not compatible with JDK 7',
+ [r".*: warning: \[Java7ApiChecker\] .+"]),
+ java_high('Abstract and default methods are not injectable with javax.inject.Inject',
+ [r".*: warning: \[JavaxInjectOnAbstractMethod\] .+"]),
+ java_high('@javax.inject.Inject cannot be put on a final field.',
+ [r".*: warning: \[JavaxInjectOnFinalField\] .+"]),
+ java_high('This pattern will silently corrupt certain byte sequences from the serialized protocol message. Use ByteString or byte[] directly',
+ [r".*: warning: \[LiteByteStringUtf8\] .+"]),
+ java_high('This method does not acquire the locks specified by its @LockMethod annotation',
+ [r".*: warning: \[LockMethodChecker\] .+"]),
+ java_high('Prefer \'L\' to \'l\' for the suffix to long literals',
+ [r".*: warning: \[LongLiteralLowerCaseSuffix\] .+"]),
+ java_high('Loop condition is never modified in loop body.',
+ [r".*: warning: \[LoopConditionChecker\] .+"]),
+ java_high('Math.round(Integer) results in truncation',
+ [r".*: warning: \[MathRoundIntLong\] .+"]),
+ java_high('Certain resources in `android.R.string` have names that do not match their content',
+ [r".*: warning: \[MislabeledAndroidString\] .+"]),
+ java_high('Overriding method is missing a call to overridden super method',
+ [r".*: warning: \[MissingSuperCall\] .+"]),
+ java_high('A terminating method call is required for a test helper to have any effect.',
+ [r".*: warning: \[MissingTestCall\] .+"]),
+ java_high('Use of "YYYY" (week year) in a date pattern without "ww" (week in year). You probably meant to use "yyyy" (year) instead.',
+ [r".*: warning: \[MisusedWeekYear\] .+"]),
+ java_high('A bug in Mockito will cause this test to fail at runtime with a ClassCastException',
+ [r".*: warning: \[MockitoCast\] .+"]),
+ java_high('Missing method call for verify(mock) here',
+ [r".*: warning: \[MockitoUsage\] .+"]),
+ java_high('Using a collection function with itself as the argument.',
+ [r".*: warning: \[ModifyingCollectionWithItself\] .+"]),
+ java_high('This class has more than one @Inject-annotated constructor. Please remove the @Inject annotation from all but one of them.',
+ [r".*: warning: \[MoreThanOneInjectableConstructor\] .+"]),
+ java_high('The result of this method must be closed.',
+ [r".*: warning: \[MustBeClosedChecker\] .+"]),
+ java_high('The first argument to nCopies is the number of copies, and the second is the item to copy',
+ [r".*: warning: \[NCopiesOfChar\] .+"]),
+ java_high('@NoAllocation was specified on this method, but something was found that would trigger an allocation',
+ [r".*: warning: \[NoAllocation\] .+"]),
+ java_high('Static import of type uses non-canonical name',
+ [r".*: warning: \[NonCanonicalStaticImport\] .+"]),
+ java_high('@CompileTimeConstant parameters should be final or effectively final',
+ [r".*: warning: \[NonFinalCompileTimeConstant\] .+"]),
+ java_high('Calling getAnnotation on an annotation that is not retained at runtime.',
+ [r".*: warning: \[NonRuntimeAnnotation\] .+"]),
+ java_high('This conditional expression may evaluate to null, which will result in an NPE when the result is unboxed.',
+ [r".*: warning: \[NullTernary\] .+"]),
+ java_high('Numeric comparison using reference equality instead of value equality',
+ [r".*: warning: \[NumericEquality\] .+"]),
+ java_high('Comparison using reference equality instead of value equality',
+ [r".*: warning: \[OptionalEquality\] .+"]),
+ java_high('Annotations cannot be both Scope annotations and Qualifier annotations: this causes confusion when trying to use them.',
+ [r".*: warning: \[OverlappingQualifierAndScopeAnnotation\] .+"]),
+ java_high('This method is not annotated with @Inject, but it overrides a method that is annotated with @javax.inject.Inject. The method will not be Injected.',
+ [r".*: warning: \[OverridesJavaxInjectableMethod\] .+"]),
+ java_high('Declaring types inside package-info.java files is very bad form',
+ [r".*: warning: \[PackageInfo\] .+"]),
+ java_high('Method parameter has wrong package',
+ [r".*: warning: \[ParameterPackage\] .+"]),
+ java_high('Detects classes which implement Parcelable but don\'t have CREATOR',
+ [r".*: warning: \[ParcelableCreator\] .+"]),
+ java_high('Literal passed as first argument to Preconditions.checkNotNull() can never be null',
+ [r".*: warning: \[PreconditionsCheckNotNull\] .+"]),
+ java_high('First argument to `Preconditions.checkNotNull()` is a primitive rather than an object reference',
+ [r".*: warning: \[PreconditionsCheckNotNullPrimitive\] .+"]),
+ java_high('Using ::equals or ::isInstance as an incompatible Predicate; the predicate will always return false',
+ [r".*: warning: \[PredicateIncompatibleType\] .+"]),
+ java_high('Access to a private protocol buffer field is forbidden. This protocol buffer carries a security contract, and can only be created using an approved library. Direct access to the fields is forbidden.',
+ [r".*: warning: \[PrivateSecurityContractProtoAccess\] .+"]),
+ java_high('Protobuf fields cannot be null.',
+ [r".*: warning: \[ProtoFieldNullComparison\] .+"]),
+ java_high('Comparing protobuf fields of type String using reference equality',
+ [r".*: warning: \[ProtoStringFieldReferenceEquality\] .+"]),
+ java_high('To get the tag number of a protocol buffer enum, use getNumber() instead.',
+ [r".*: warning: \[ProtocolBufferOrdinal\] .+"]),
+ java_high('@Provides methods need to be declared in a Module to have any effect.',
+ [r".*: warning: \[ProvidesMethodOutsideOfModule\] .+"]),
+ java_high('Casting a random number in the range [0.0, 1.0) to an integer or long always results in 0.',
+ [r".*: warning: \[RandomCast\] .+"]),
+ java_high('Use Random.nextInt(int). Random.nextInt() % n can have negative results',
+ [r".*: warning: \[RandomModInteger\] .+"]),
+ java_high('Return value of android.graphics.Rect.intersect() must be checked',
+ [r".*: warning: \[RectIntersectReturnValueIgnored\] .+"]),
+ java_high('Use of method or class annotated with @RestrictTo',
+ [r".*: warning: \[RestrictTo\] .+"]),
+ java_high(' Check for non-whitelisted callers to RestrictedApiChecker.',
+ [r".*: warning: \[RestrictedApiChecker\] .+"]),
+ java_high('Return value of this method must be used',
+ [r".*: warning: \[ReturnValueIgnored\] .+"]),
+ java_high('Variable assigned to itself',
+ [r".*: warning: \[SelfAssignment\] .+"]),
+ java_high('An object is compared to itself',
+ [r".*: warning: \[SelfComparison\] .+"]),
+ java_high('Testing an object for equality with itself will always be true.',
+ [r".*: warning: \[SelfEquals\] .+"]),
+ java_high('This method must be called with an even number of arguments.',
+ [r".*: warning: \[ShouldHaveEvenArgs\] .+"]),
+ java_high('Comparison of a size >= 0 is always true, did you intend to check for non-emptiness?',
+ [r".*: warning: \[SizeGreaterThanOrEqualsZero\] .+"]),
+ java_high('Static and default interface methods are not natively supported on older Android devices. ',
+ [r".*: warning: \[StaticOrDefaultInterfaceMethod\] .+"]),
+ java_high('Calling toString on a Stream does not provide useful information',
+ [r".*: warning: \[StreamToString\] .+"]),
+ java_high('StringBuilder does not have a char constructor; this invokes the int constructor.',
+ [r".*: warning: \[StringBuilderInitWithChar\] .+"]),
+ java_high('String.substring(0) returns the original String',
+ [r".*: warning: \[SubstringOfZero\] .+"]),
+ java_high('Suppressing "deprecated" is probably a typo for "deprecation"',
+ [r".*: warning: \[SuppressWarningsDeprecated\] .+"]),
+ java_high('throwIfUnchecked(knownCheckedException) is a no-op.',
+ [r".*: warning: \[ThrowIfUncheckedKnownChecked\] .+"]),
+ java_high('Throwing \'null\' always results in a NullPointerException being thrown.',
+ [r".*: warning: \[ThrowNull\] .+"]),
+ java_high('isEqualTo should not be used to test an object for equality with itself; the assertion will never fail.',
+ [r".*: warning: \[TruthSelfEquals\] .+"]),
+ java_high('Catching Throwable/Error masks failures from fail() or assert*() in the try block',
+ [r".*: warning: \[TryFailThrowable\] .+"]),
+ java_high('Type parameter used as type qualifier',
+ [r".*: warning: \[TypeParameterQualifier\] .+"]),
+ java_high('This method does not acquire the locks specified by its @UnlockMethod annotation',
+ [r".*: warning: \[UnlockMethod\] .+"]),
+ java_high('Non-generic methods should not be invoked with type arguments',
+ [r".*: warning: \[UnnecessaryTypeArgument\] .+"]),
+ java_high('Instance created but never used',
+ [r".*: warning: \[UnusedAnonymousClass\] .+"]),
+ java_high('Collection is modified in place, but the result is not used',
+ [r".*: warning: \[UnusedCollectionModifiedInPlace\] .+"]),
+ java_high('`var` should not be used as a type name.',
+ [r".*: warning: \[VarTypeName\] .+"]),
# End warnings generated by Error Prone
- {'category': 'java',
- 'severity': Severity.UNKNOWN,
- 'description': 'Java: Unclassified/unrecognized warnings',
- 'patterns': [r".*: warning: \[.+\] .+"]},
+ java_warn(Severity.UNKNOWN,
+ 'Unclassified/unrecognized warnings',
+ [r".*: warning: \[.+\] .+"]), # TODO(chh) use more specific pattern
+ # aapt warnings
{'category': 'aapt', 'severity': Severity.MEDIUM,
'description': 'aapt: No default translation',
'patterns': [r".*: warning: string '.+' has no default translation in .*"]},
@@ -2169,6 +1229,8 @@
{'category': 'aapt', 'severity': Severity.MEDIUM,
'description': 'aapt: empty span in string',
'patterns': [r".*: warning: empty '.+' span found in text '.+"]},
+
+ # C/C++ warnings
{'category': 'C/C++', 'severity': Severity.MEDIUM,
'description': 'Taking address of temporary',
'patterns': [r".*: warning: taking address of temporary"]},
@@ -2184,11 +1246,11 @@
{'category': 'C/C++', 'severity': Severity.MEDIUM, 'option': '-Wundefined-inline',
'description': 'Inline function is not defined',
'patterns': [r".*: warning: inline function '.*' is not defined"]},
- {'category': 'C/C++', 'severity': Severity.MEDIUM, 'option': '-Warray-bounds',
- 'description': 'Array subscript out of bounds',
- 'patterns': [r".*: warning: array subscript is above array bounds",
- r".*: warning: Array subscript is undefined",
- r".*: warning: array subscript is below array bounds"]},
+ # {'category': 'C/C++', 'severity': Severity.MEDIUM, 'option': '-Warray-bounds',
+ # 'description': 'Array subscript out of bounds',
+ # 'patterns': [r".*: warning: array subscript is above array bounds",
+ # r".*: warning: Array subscript is undefined",
+ # r".*: warning: array subscript is below array bounds"]},
{'category': 'C/C++', 'severity': Severity.MEDIUM,
'description': 'Excess elements in initializer',
'patterns': [r".*: warning: excess elements in .+ initializer"]},
@@ -2289,14 +1351,7 @@
'patterns': [r".*: warning: multi-line comment"]},
{'category': 'C/C++', 'severity': Severity.LOW, 'option': '-Wcomment',
'description': 'Comment inside comment',
- 'patterns': [r".*: warning: "".+"" within comment"]},
- # Warning "value stored is never read" could be from clang-tidy or clang static analyzer.
- {'category': 'C/C++', 'severity': Severity.ANALYZER,
- 'description': 'clang-analyzer Value stored is never read',
- 'patterns': [r".*: warning: Value stored to .+ is never read.*clang-analyzer-deadcode.DeadStores"]},
- {'category': 'C/C++', 'severity': Severity.LOW,
- 'description': 'Value stored is never read',
- 'patterns': [r".*: warning: Value stored to .+ is never read"]},
+ 'patterns': [r".*: warning: '.+' within block comment .*-Wcomment"]},
{'category': 'C/C++', 'severity': Severity.LOW, 'option': '-Wdeprecated-declarations',
'description': 'Deprecated declarations',
'patterns': [r".*: warning: .+ is deprecated.+deprecated-declarations"]},
@@ -2311,7 +1366,8 @@
'patterns': [r".*: warning: extra tokens at end of #endif directive"]},
{'category': 'C/C++', 'severity': Severity.MEDIUM, 'option': '-Wenum-compare',
'description': 'Comparison between different enums',
- 'patterns': [r".*: warning: comparison between '.+' and '.+'.+Wenum-compare"]},
+ 'patterns': [r".*: warning: comparison between '.+' and '.+'.+Wenum-compare",
+ r".*: warning: comparison of .* enumeration types .*-Wenum-compare-switch"]},
{'category': 'C/C++', 'severity': Severity.MEDIUM, 'option': '-Wconversion',
'description': 'Conversion may change value',
'patterns': [r".*: warning: converting negative value '.+' to '.+'",
@@ -2329,8 +1385,9 @@
'description': 'Zero used as null pointer',
'patterns': [r".*: warning: expression .* zero treated as a null pointer constant"]},
{'category': 'C/C++', 'severity': Severity.MEDIUM,
- 'description': 'Implicit conversion changes value',
- 'patterns': [r".*: warning: implicit conversion .* changes value from .* to .*-conversion"]},
+ 'description': 'Implicit conversion changes value or loses precision',
+ 'patterns': [r".*: warning: implicit conversion .* changes value from .* to .*-conversion",
+ r".*: warning: implicit conversion loses integer precision:"]},
{'category': 'C/C++', 'severity': Severity.MEDIUM,
'description': 'Passing NULL as non-pointer argument',
'patterns': [r".*: warning: passing NULL to non-pointer argument [0-9]+ of '.+'"]},
@@ -2361,9 +1418,9 @@
{'category': 'C/C++', 'severity': Severity.HIGH, 'option': '-Wextra',
'description': 'Base should be explicitly initialized in copy constructor',
'patterns': [r".*: warning: base class '.+' should be explicitly initialized in the copy constructor"]},
- {'category': 'C/C++', 'severity': Severity.MEDIUM,
- 'description': 'VLA has zero or negative size',
- 'patterns': [r".*: warning: Declared variable-length array \(VLA\) has .+ size"]},
+ # {'category': 'C/C++', 'severity': Severity.MEDIUM,
+ # 'description': 'VLA has zero or negative size',
+ # 'patterns': [r".*: warning: Declared variable-length array \(VLA\) has .+ size"]},
{'category': 'C/C++', 'severity': Severity.MEDIUM,
'description': 'Return value from void function',
'patterns': [r".*: warning: 'return' with a value, in function returning void"]},
@@ -2420,9 +1477,9 @@
{'category': 'logtags', 'severity': Severity.LOW, 'option': 'overloaded-virtual',
'description': 'Hides overloaded virtual function',
'patterns': [r".*: '.+' hides overloaded virtual function"]},
- {'category': 'logtags', 'severity': Severity.LOW, 'option': 'incompatible-pointer-types',
+ {'category': 'logtags', 'severity': Severity.LOW,
'description': 'Incompatible pointer types',
- 'patterns': [r".*: warning: incompatible pointer types .+Wincompatible-pointer-types"]},
+ 'patterns': [r".*: warning: incompatible .*pointer types .*-Wincompatible-.*pointer-types"]},
{'category': 'logtags', 'severity': Severity.LOW, 'option': 'asm-operand-widths',
'description': 'ASM value size does not match register size',
'patterns': [r".*: warning: value size does not match register size specified by the constraint and modifier"]},
@@ -2474,35 +1531,38 @@
{'category': 'C/C++', 'severity': Severity.MEDIUM, 'option': 'switch',
'description': 'case value not in enumerated type',
'patterns': [r".*: warning: case value not in enumerated type '.+'"]},
- {'category': 'C/C++', 'severity': Severity.MEDIUM,
- 'description': 'Undefined result',
- 'patterns': [r".*: warning: The result of .+ is undefined",
- r".*: warning: passing an object that .+ has undefined behavior \[-Wvarargs\]",
- r".*: warning: 'this' pointer cannot be null in well-defined C\+\+ code;",
- r".*: warning: shifting a negative signed value is undefined"]},
- {'category': 'C/C++', 'severity': Severity.MEDIUM,
- 'description': 'Division by zero',
- 'patterns': [r".*: warning: Division by zero"]},
+ # {'category': 'C/C++', 'severity': Severity.MEDIUM,
+ # 'description': 'Undefined result',
+ # 'patterns': [r".*: warning: The result of .+ is undefined",
+ # r".*: warning: passing an object that .+ has undefined behavior \[-Wvarargs\]",
+ # r".*: warning: 'this' pointer cannot be null in well-defined C\+\+ code;",
+ # r".*: warning: shifting a negative signed value is undefined"]},
+ # {'category': 'C/C++', 'severity': Severity.MEDIUM,
+ # 'description': 'Division by zero',
+ # 'patterns': [r".*: warning: Division by zero"]},
{'category': 'C/C++', 'severity': Severity.MEDIUM,
'description': 'Use of deprecated method',
'patterns': [r".*: warning: '.+' is deprecated .+"]},
{'category': 'C/C++', 'severity': Severity.MEDIUM,
'description': 'Use of garbage or uninitialized value',
- 'patterns': [r".*: warning: .+ is a garbage value",
- r".*: warning: Function call argument is an uninitialized value",
- r".*: warning: Undefined or garbage value returned to caller",
- r".*: warning: Called .+ pointer is.+uninitialized",
- r".*: warning: Called .+ pointer is.+uninitalized", # match a typo in compiler message
- r".*: warning: Use of zero-allocated memory",
- r".*: warning: Dereference of undefined pointer value",
- r".*: warning: Passed-by-value .+ contains uninitialized data",
- r".*: warning: Branch condition evaluates to a garbage value",
- r".*: warning: The .+ of .+ is an uninitialized value.",
- r".*: warning: .+ is used uninitialized whenever .+sometimes-uninitialized",
- r".*: warning: Assigned value is garbage or undefined"]},
- {'category': 'C/C++', 'severity': Severity.MEDIUM,
- 'description': 'Result of malloc type incompatible with sizeof operand type',
- 'patterns': [r".*: warning: Result of '.+' is converted to .+ incompatible with sizeof operand type"]},
+ 'patterns': [r".*: warning: .+ uninitialized .+\[-Wsometimes-uninitialized\]"]},
+ # {'category': 'C/C++', 'severity': Severity.MEDIUM,
+ # 'description': 'Use of garbage or uninitialized value',
+ # 'patterns': [r".*: warning: .+ is a garbage value",
+ # r".*: warning: Function call argument is an uninitialized value",
+ # r".*: warning: Undefined or garbage value returned to caller",
+ # r".*: warning: Called .+ pointer is.+uninitialized",
+ # r".*: warning: Called .+ pointer is.+uninitalized", # match a typo in compiler message
+ # r".*: warning: Use of zero-allocated memory",
+ # r".*: warning: Dereference of undefined pointer value",
+ # r".*: warning: Passed-by-value .+ contains uninitialized data",
+ # r".*: warning: Branch condition evaluates to a garbage value",
+ # r".*: warning: The .+ of .+ is an uninitialized value.",
+ # r".*: warning: .+ is used uninitialized whenever .+sometimes-uninitialized",
+ # r".*: warning: Assigned value is garbage or undefined"]},
+ # {'category': 'C/C++', 'severity': Severity.MEDIUM,
+ # 'description': 'Result of malloc type incompatible with sizeof operand type',
+ # 'patterns': [r".*: warning: Result of '.+' is converted to .+ incompatible with sizeof operand type"]},
{'category': 'C/C++', 'severity': Severity.MEDIUM, 'option': '-Wsizeof-array-argument',
'description': 'Sizeof on array argument',
'patterns': [r".*: warning: sizeof on array function parameter will return"]},
@@ -2515,12 +1575,12 @@
{'category': 'C/C++', 'severity': Severity.MEDIUM,
'description': 'Possible heap pollution',
'patterns': [r".*: warning: .*Possible heap pollution from .+ type .+"]},
- {'category': 'C/C++', 'severity': Severity.MEDIUM,
- 'description': 'Allocation size of 0 byte',
- 'patterns': [r".*: warning: Call to .+ has an allocation size of 0 byte"]},
- {'category': 'C/C++', 'severity': Severity.MEDIUM,
- 'description': 'Result of malloc type incompatible with sizeof operand type',
- 'patterns': [r".*: warning: Result of '.+' is converted to .+ incompatible with sizeof operand type"]},
+ # {'category': 'C/C++', 'severity': Severity.MEDIUM,
+ # 'description': 'Allocation size of 0 byte',
+ # 'patterns': [r".*: warning: Call to .+ has an allocation size of 0 byte"]},
+ # {'category': 'C/C++', 'severity': Severity.MEDIUM,
+ # 'description': 'Result of malloc type incompatible with sizeof operand type',
+ # 'patterns': [r".*: warning: Result of '.+' is converted to .+ incompatible with sizeof operand type"]},
{'category': 'C/C++', 'severity': Severity.MEDIUM, 'option': '-Wfor-loop-analysis',
'description': 'Variable used in loop condition not modified in loop body',
'patterns': [r".*: warning: variable '.+' used in loop condition.*Wfor-loop-analysis"]},
@@ -2554,12 +1614,15 @@
{'category': 'FindEmulator', 'severity': Severity.HARMLESS,
'description': 'FindEmulator: No such file or directory',
'patterns': [r".*: warning: FindEmulator: .* No such file or directory"]},
- {'category': 'google_tests', 'severity': Severity.HARMLESS,
- 'description': 'google_tests: unknown installed file',
+ {'category': 'make', 'severity': Severity.HARMLESS,
+ 'description': 'make: unknown installed file',
'patterns': [r".*: warning: .*_tests: Unknown installed file for module"]},
{'category': 'make', 'severity': Severity.HARMLESS,
'description': 'unusual tags debug eng',
'patterns': [r".*: warning: .*: unusual tags debug eng"]},
+ {'category': 'make', 'severity': Severity.MEDIUM,
+ 'description': 'make: please convert to soong',
+ 'patterns': [r".*: warning: .* has been deprecated. Please convert to Soong."]},
# these next ones are to deal with formatting problems resulting from the log being mixed up by 'make -j'
{'category': 'C/C++', 'severity': Severity.SKIP,
@@ -2574,6 +1637,7 @@
# warnings from clang-tidy
group_tidy_warn_pattern('android'),
+ simple_tidy_warn_pattern('abseil-string-find-startswith'),
simple_tidy_warn_pattern('bugprone-argument-comment'),
simple_tidy_warn_pattern('bugprone-copy-constructor-init'),
simple_tidy_warn_pattern('bugprone-fold-init-type'),
@@ -2633,56 +1697,99 @@
simple_tidy_warn_pattern('performance-type-promotion-in-math-fn'),
simple_tidy_warn_pattern('performance-unnecessary-copy-initialization'),
simple_tidy_warn_pattern('performance-unnecessary-value-param'),
+ simple_tidy_warn_pattern('portability-simd-intrinsics'),
group_tidy_warn_pattern('performance'),
group_tidy_warn_pattern('readability'),
# warnings from clang-tidy's clang-analyzer checks
- {'category': 'C/C++', 'severity': Severity.ANALYZER,
- 'description': 'clang-analyzer Unreachable code',
- 'patterns': [r".*: warning: This statement is never executed.*UnreachableCode"]},
- {'category': 'C/C++', 'severity': Severity.ANALYZER,
- 'description': 'clang-analyzer Size of malloc may overflow',
- 'patterns': [r".*: warning: .* size of .* may overflow .*MallocOverflow"]},
- {'category': 'C/C++', 'severity': Severity.ANALYZER,
- 'description': 'clang-analyzer Stream pointer might be NULL',
- 'patterns': [r".*: warning: Stream pointer might be NULL .*unix.Stream"]},
- {'category': 'C/C++', 'severity': Severity.ANALYZER,
- 'description': 'clang-analyzer Opened file never closed',
- 'patterns': [r".*: warning: Opened File never closed.*unix.Stream"]},
- {'category': 'C/C++', 'severity': Severity.ANALYZER,
- 'description': 'clang-analyzer sozeof() on a pointer type',
- 'patterns': [r".*: warning: .*calls sizeof.* on a pointer type.*SizeofPtr"]},
- {'category': 'C/C++', 'severity': Severity.ANALYZER,
- 'description': 'clang-analyzer Pointer arithmetic on non-array variables',
- 'patterns': [r".*: warning: Pointer arithmetic on non-array variables .*PointerArithm"]},
- {'category': 'C/C++', 'severity': Severity.ANALYZER,
- 'description': 'clang-analyzer Subtraction of pointers of different memory chunks',
- 'patterns': [r".*: warning: Subtraction of two pointers .*PointerSub"]},
- {'category': 'C/C++', 'severity': Severity.ANALYZER,
- 'description': 'clang-analyzer Access out-of-bound array element',
- 'patterns': [r".*: warning: Access out-of-bound array element .*ArrayBound"]},
- {'category': 'C/C++', 'severity': Severity.ANALYZER,
- 'description': 'clang-analyzer Out of bound memory access',
- 'patterns': [r".*: warning: Out of bound memory access .*ArrayBoundV2"]},
- {'category': 'C/C++', 'severity': Severity.ANALYZER,
- 'description': 'clang-analyzer Possible lock order reversal',
- 'patterns': [r".*: warning: .* Possible lock order reversal.*PthreadLock"]},
- {'category': 'C/C++', 'severity': Severity.ANALYZER,
- 'description': 'clang-analyzer Argument is a pointer to uninitialized value',
- 'patterns': [r".*: warning: .* argument is a pointer to uninitialized value .*CallAndMessage"]},
- {'category': 'C/C++', 'severity': Severity.ANALYZER,
- 'description': 'clang-analyzer cast to struct',
- 'patterns': [r".*: warning: Casting a non-structure type to a structure type .*CastToStruct"]},
- {'category': 'C/C++', 'severity': Severity.ANALYZER,
- 'description': 'clang-analyzer call path problems',
- 'patterns': [r".*: warning: Call Path : .+"]},
- {'category': 'C/C++', 'severity': Severity.ANALYZER,
- 'description': 'clang-analyzer excessive padding',
- 'patterns': [r".*: warning: Excessive padding in '.*'"]},
- {'category': 'C/C++', 'severity': Severity.ANALYZER,
- 'description': 'clang-analyzer other',
- 'patterns': [r".*: .+\[clang-analyzer-.+\]$",
- r".*: Call Path : .+$"]},
+ analyzer_high('clang-analyzer-core, null pointer',
+ [r".*: warning: .+ pointer is null .*\[clang-analyzer-core"]),
+ analyzer_high('clang-analyzer-core, uninitialized value',
+ [r".*: warning: .+ uninitialized (value|data) .*\[clang-analyzer-core"]),
+ analyzer_warn('clang-analyzer-optin.performance.Padding',
+ [r".*: warning: Excessive padding in '.*'"]),
+ # analyzer_warn('clang-analyzer Unreachable code',
+ # [r".*: warning: This statement is never executed.*UnreachableCode"]),
+ analyzer_warn('clang-analyzer Size of malloc may overflow',
+ [r".*: warning: .* size of .* may overflow .*MallocOverflow"]),
+ analyzer_warn('clang-analyzer sozeof() on a pointer type',
+ [r".*: warning: .*calls sizeof.* on a pointer type.*SizeofPtr"]),
+ analyzer_warn('clang-analyzer Pointer arithmetic on non-array variables',
+ [r".*: warning: Pointer arithmetic on non-array variables .*PointerArithm"]),
+ analyzer_warn('clang-analyzer Subtraction of pointers of different memory chunks',
+ [r".*: warning: Subtraction of two pointers .*PointerSub"]),
+ analyzer_warn('clang-analyzer Access out-of-bound array element',
+ [r".*: warning: Access out-of-bound array element .*ArrayBound"]),
+ analyzer_warn('clang-analyzer Out of bound memory access',
+ [r".*: warning: Out of bound memory access .*ArrayBoundV2"]),
+ analyzer_warn('clang-analyzer Possible lock order reversal',
+ [r".*: warning: .* Possible lock order reversal.*PthreadLock"]),
+ analyzer_warn('clang-analyzer call path problems',
+ [r".*: warning: Call Path : .+"]),
+ analyzer_warn_check('clang-analyzer-core.CallAndMessage'),
+ analyzer_high_check('clang-analyzer-core.NonNullParamChecker'),
+ analyzer_high_check('clang-analyzer-core.NullDereference'),
+ analyzer_warn_check('clang-analyzer-core.UndefinedBinaryOperatorResult'),
+ analyzer_warn_check('clang-analyzer-core.DivideZero'),
+ analyzer_warn_check('clang-analyzer-core.VLASize'),
+ analyzer_warn_check('clang-analyzer-core.uninitialized.ArraySubscript'),
+ analyzer_warn_check('clang-analyzer-core.uninitialized.Assign'),
+ analyzer_warn_check('clang-analyzer-core.uninitialized.UndefReturn'),
+ analyzer_warn_check('clang-analyzer-cplusplus.Move'),
+ analyzer_warn_check('clang-analyzer-deadcode.DeadStores'),
+ analyzer_warn_check('clang-analyzer-optin.cplusplus.UninitializedObject'),
+ analyzer_warn_check('clang-analyzer-optin.cplusplus.VirtualCall'),
+ analyzer_warn_check('clang-analyzer-portability.UnixAPI'),
+ analyzer_warn_check('clang-analyzer-unix.cstring.NullArg'),
+ analyzer_high_check('clang-analyzer-unix.MallocSizeof'),
+ analyzer_warn_check('clang-analyzer-valist.Uninitialized'),
+ analyzer_warn_check('clang-analyzer-valist.Unterminated'),
+ analyzer_group_check('clang-analyzer-core.uninitialized'),
+ analyzer_group_check('clang-analyzer-deadcode'),
+ analyzer_warn_check('clang-analyzer-security.insecureAPI.strcpy'),
+ analyzer_group_high('clang-analyzer-security.insecureAPI'),
+ analyzer_group_high('clang-analyzer-security'),
+ analyzer_group_check('clang-analyzer-unix.Malloc'),
+ analyzer_group_check('clang-analyzer-unix'),
+ analyzer_group_check('clang-analyzer'), # catch al
+
+ # Assembler warnings
+ {'category': 'Asm', 'severity': Severity.MEDIUM,
+ 'description': 'Asm: IT instruction is deprecated',
+ 'patterns': [r".*: warning: applying IT instruction .* is deprecated"]},
+
+ # NDK warnings
+ {'category': 'NDK', 'severity': Severity.HIGH,
+ 'description': 'NDK: Generate guard with empty availability, obsoleted',
+ 'patterns': [r".*: warning: .* generate guard with empty availability: obsoleted ="]},
+
+ # Protoc warnings
+ {'category': 'Protoc', 'severity': Severity.MEDIUM,
+ 'description': 'Proto: Enum name colision after strip',
+ 'patterns': [r".*: warning: Enum .* has the same name .* ignore case and strip"]},
+
+ # Kotlin warnings
+ {'category': 'Kotlin', 'severity': Severity.MEDIUM,
+ 'description': 'Kotlin: never used parameter',
+ 'patterns': [r".*: warning: parameter '.*' is never used"]},
+ {'category': 'Kotlin', 'severity': Severity.MEDIUM,
+ 'description': 'Kotlin: Deprecated in Java',
+ 'patterns': [r".*: warning: '.*' is deprecated. Deprecated in Java"]},
+ {'category': 'Kotlin', 'severity': Severity.MEDIUM,
+ 'description': 'Kotlin: library has Kotlin runtime',
+ 'patterns': [r".*: warning: library has Kotlin runtime bundled into it",
+ r".*: warning: some JAR files .* have the Kotlin Runtime library"]},
+
+ # rustc warnings
+ {'category': 'Rust', 'severity': Severity.HIGH,
+ 'description': 'Rust: Does not derive Copy',
+ 'patterns': [r".*: warning: .+ does not derive Copy"]},
+ {'category': 'Rust', 'severity': Severity.MEDIUM,
+ 'description': 'Rust: Deprecated range pattern',
+ 'patterns': [r".*: warning: .+ range patterns are deprecated"]},
+ {'category': 'Rust', 'severity': Severity.MEDIUM,
+ 'description': 'Rust: Deprecated missing explicit \'dyn\'',
+ 'patterns': [r".*: warning: .+ without an explicit `dyn` are deprecated"]},
# catch-all for warnings this script doesn't know about yet
{'category': 'C/C++', 'severity': Severity.UNKNOWN,
@@ -3208,16 +2315,42 @@
global target_variant
line_counter = 0
- # handle only warning messages with a file path
- warning_pattern = re.compile('^[^ ]*/[^ ]*: warning: .*')
+ # rustc warning messages have two lines that should be combined:
+ # warning: description
+ # --> file_path:line_number:column_number
+ # Some warning messages have no file name:
+ # warning: macro replacement list ... [bugprone-macro-parentheses]
+ # Some makefile warning messages have no line number:
+ # some/path/file.mk: warning: description
+ # C/C++ compiler warning messages have line and column numbers:
+ # some/path/file.c:line_number:column_number: warning: description
+ warning_pattern = re.compile('(^[^ ]*/[^ ]*: warning: .*)|(^warning: .*)')
+ warning_without_file = re.compile('^warning: .*')
+ rustc_file_position = re.compile('^[ ]+--> [^ ]*/[^ ]*:[0-9]+:[0-9]+')
# Collect all warnings into the warning_lines set.
warning_lines = set()
+ prev_warning = ''
for line in infile:
+ if prev_warning:
+ if rustc_file_position.match(line):
+ # must be a rustc warning, combine 2 lines into one warning
+ line = line.strip().replace('--> ', '') + ': ' + prev_warning
+ warning_lines.add(normalize_warning_line(line))
+ prev_warning = ''
+ continue
+ # add prev_warning, and then process the current line
+ prev_warning = 'unknown_source_file: ' + prev_warning
+ warning_lines.add(normalize_warning_line(prev_warning))
+ prev_warning = ''
if warning_pattern.match(line):
- line = normalize_warning_line(line)
- warning_lines.add(line)
- elif line_counter < 100:
+ if warning_without_file.match(line):
+ # save this line and combine it with the next line
+ prev_warning = line
+ else:
+ warning_lines.add(normalize_warning_line(line))
+ continue
+ if line_counter < 100:
# save a little bit of time by only doing this for the first few lines
line_counter += 1
m = re.search('(?<=^PLATFORM_VERSION=).*', line)
