https://issues.apache.org/bugzilla/show_bug.cgi?id=53642
Yegor Kozlov <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- OS| |All --- Comment #1 from Yegor Kozlov <[email protected]> --- >* Note: I know, the use of a public static attribute is a no-go for real code, >but this is for specific development use only, simple to use in this way, and >worst case situation is missing or surplus logging. Why did you make dbgEvaluationOutputForNextEval static ? It is an instance-scope variable and it is much better to set it per-evaluator. That is, user code woulde change from WorkbookEvaluator.dbgEvaluationOutputForNextEval = true; evaluator.evaluateFormulaCell(cell); to evaluator.dbgEvaluationOutputForNextEval = true; evaluator.evaluateFormulaCell(cell); or even better: evaluator.setDebugEvaluationOutputForNextEval(true); evaluator.evaluateFormulaCell(cell); I agree that a "public static" modifier is a "no go" for production but since POI is a public program API we should minimize possibilities of its incorrect usage. Regards, Yegor -- You are receiving this mail because: You are the assignee for the bug. --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
