[litho] Don't suppress reports on private methods

Summary: We missed a legit issue due to this outdated heuristic

Reviewed By: skcho

Differential Revision: D21327809

fbshipit-source-id: 0c69702ad
master
Ezgi Çiçek 5 years ago committed by Facebook GitHub Bot
parent b419541af2
commit ffde64c0c9

@ -181,9 +181,7 @@ let satisfies_heuristic ~callee_pname ~callee_summary_opt tenv =
let should_report proc_desc tenv = let should_report proc_desc tenv =
let pname = Procdesc.get_proc_name proc_desc in let pname = Procdesc.get_proc_name proc_desc in
(not (is_litho_function pname)) (not (is_litho_function pname)) && not (is_component_build_method pname tenv)
&& (not (is_component_build_method pname tenv))
&& not (PredSymb.equal_access (Procdesc.get_access proc_desc) Private)
let report astate tenv summary = let report astate tenv summary =

Loading…
Cancel
Save