Hi hackers,

While reading analyze.c in current master, I noticed a small cleanup opportunity in the ANALYZE code path.

Currently, analyze_rel() computes elevel from params.options, but does not use it directly. It only passes the value down to do_analyze_rel(). At the same time, do_analyze_rel() already receives params and derives verbose from params.options for its own purposes.

The attached patch moves the elevel calculation into do_analyze_rel() and removes the extra elevel argument from its declaration, definition, and call sites.

No behavioral change is intended here. The logging level selection remains the same; the calculation is just moved closer to the actual use sites.

Comments and feedback would be appreciated.

--

Best regards,

Lev Nikolaev,

Tantor Labs LLC,

https://tantorlabs.com/