No need to gate hook with enabledebug

Author: terryburton

CHANGES

@@ -1,6 +1,7 @@
 XXXX-XX-XX
 
 * The bearer bars were reworked to render as filled paths rather than stroked lines, matching the border and the data bars for consistent rendering at small scales.
+* The hooks framework was amended so that hook procedures are activated by the presence of a hooks dictionary in the global context, without requiring the enabledebug option.
 
 
 2026-05-31

CONTRIBUTING.md

@@ -119,7 +119,7 @@ large values at define time from small operands and reference via `//name`.
 containing optional keys that affect the behaviour of BWIPP:
 
 - `preload`        - Perform eager initialisation of normally lazy variables during resource definition
-- `enabledebug`    - Allow the user to set debug options (e.g. `debugcws`), for development purposes including activation of hooks
+- `enabledebug`    - Allow the user to set debug options (e.g. `debugcws`), for development purposes
 - `enabledontdraw` - Allow the user to set dontdraw, in case they are providing custom renderers
 - `hooks`          - Dictionary of hook procedures for debugging purposes, including profiling
 - `default_*`      - Defaults for certain renderer options set by the user, e.g. `default_barcolor`
@@ -1074,13 +1074,12 @@ By default hooks will do nothing other than pop the parameter.
 The above pair of hooks is intended to allow measurement of time spent in each
 phase via the definition of suitable procedures within global context.
 
-The global context must be configured with `enabledebug` set and a `hooks`
-dictionary containing custom procedures related to the hooks, for example:
+The global context must be configured with a `hooks` dictionary containing
+custom procedures related to the hooks, for example:
 
 ```postscript
 currentglobal true setglobal
 /uk.co.terryburton.bwipp.global_ctx <<
-    /enabledebug true
     /hooks <<
         /before {20 string cvs print ( before ) print print ( ) print realtime =}
         /after  {20 string cvs print ( after  ) print print ( ) print realtime =}

src/setuphooks.ps.src

@@ -49,13 +49,11 @@ begin
     /targetdict exch def
 
     %
-    % Get hooks dict from global_ctx if enabledebug is set
+    % Get hooks dict from global_ctx, if present
     %
     /hooksdict null def
     /uk.co.terryburton.bwipp.global_ctx dup where {
-        exch get dup /enabledebug known {
-            /hooks 2 copy known {get /hooksdict exch def} {pop pop} ifelse
-        } {pop} ifelse
+        exch get /hooks 2 copy known {get /hooksdict exch def} {pop pop} ifelse
     } {pop} ifelse
 
     hooks {

tests/ps_tests/setuphooks.ps.test

@@ -101,19 +101,18 @@
 
 
 %
-% Without enabledebug, hooks are default no-ops
+% Hooks activate from the presence of a hooks dict; enabledebug is not required
 %
 /uk.co.terryburton.bwipp.global_ctx <<
-    /hooks << /_test.nodebug.hook { pop pop /_test.nodebug.hook_wrong } >>
+    /hooks << /_test.nodebug.hook { pop pop /_test.nodebug.hook_ran } >>
 >> def
 
 {
     10 dict begin
     /_test.nodebug.enc [/_test.nodebug.hook] setuphooks
     (phase) _test.nodebug.enc._test.nodebug.hook
-    true
     end
-} true isEqual
+} /_test.nodebug.hook_ran isEqual
 
 
 %