From 5883d15e7cef6e0cfa95a2fde273877890296378 Mon Sep 17 00:00:00 2001 From: Paul Buetow Date: Sun, 24 May 2026 00:14:11 +0300 Subject: docs: fix inaccurate comments in operations_interfaces.go - SetMode: corrected comment from 'number formatting' to accurately reflect that CalculationMode controls calculation precision type (FloatMode, RationalMode), not formatting. - Operator design note: added 'custom metric commands' to the list of method categories mixed into the interface, matching the actual CustomList, CustomDefine, and CustomUndefine methods. --- internal/rpn/operations_interfaces.go | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/internal/rpn/operations_interfaces.go b/internal/rpn/operations_interfaces.go index 0f461f0..6076ec0 100644 --- a/internal/rpn/operations_interfaces.go +++ b/internal/rpn/operations_interfaces.go @@ -72,10 +72,10 @@ type PowerIntOperator interface { // This allows RPN to depend on an abstraction instead of the concrete Operations type. // // Design note: Operator intentionally mixes behavioral methods (arithmetic, stack, -// boolean ops) with configuration methods (SetMode, SetPrefixMode, GetPrefixMode) -// and metric command handlers. Per ISP this could be split, but RPN is the sole -// client and splitting would add indirection without practical benefit. The -// concrete *Operations type satisfies this interface exclusively. +// boolean ops) with configuration methods (SetMode, SetPrefixMode, GetPrefixMode), +// metric command handlers, and custom metric commands. Per ISP this could be split, +// but RPN is the sole client and splitting would add indirection without practical +// benefit. The concrete *Operations type satisfies this interface exclusively. type Operator interface { ArithmeticOperator BooleanOperator @@ -84,7 +84,7 @@ type Operator interface { VariableOperator ConstantOperator PowerIntOperator - // SetMode sets the calculation mode for number formatting + // SetMode sets the calculation mode (e.g., FloatMode, RationalMode). SetMode(CalculationMode) // SetPrefixMode sets the prefix mode for data size calculations SetPrefixMode(PrefixMode) -- cgit v1.2.3