Signature for ZnSetLabelFromOm has been extended with a reason parameter.

This patch makes this parameter a permanent part of the signature. It'll not
be possible to suppress it by the OM_TECH define. In this way the interface
is more predictible, the signature is constant.

3 files changed, 15 insertions, 21 deletions

diff --git a/generic/Track.c b/generic/Track.c
index 8fb3e0c..911a2c0 100644
--- a/generic/Track.c
+++ b/generic/Track.c
@@ -1976,16 +1976,13 @@ ZnSendTrackToOm(void    *ptr,
 **********************************************************************************
 */
 void
-ZnSetLabelAngleFromOm(void   *ptr,   /* No longer in use. */
-                      void   *item,
-                      int    rho,
-                      int    theta
-#ifdef OM_TECH
-                      , char *reason /* Technical data explaining algorithm processing */
-#endif
-                      )
+ZnSetLabelAngleFromOm(void *ptr,   /* No longer in use. */
+                      void *item,
+                      int  rho,
+                      int  theta,
+                      char *reason) /* Technical data explaining algorithm processing */
 {
-  TrackItem     track = (TrackItem) item;
+  TrackItem track = (TrackItem) item;
 
   theta %= 360;
   if (theta < 0) {
@@ -2002,10 +1999,12 @@ ZnSetLabelAngleFromOm(void   *ptr,   /* No longer in use. */
   }
 #ifdef OM_TECH
   /* Setup reason regardless of redrawing options. */
-  if (!track->omtechstuff) {
-    track->omtechstuff = ZnMalloc(  (OMTECHSTUFFLEN + 1) * sizeof(char));
+  if (reason) {
+    if (!track->omtechstuff) {
+      track->omtechstuff = ZnMalloc(  (OMTECHSTUFFLEN + 1) * sizeof(char));
+    }
+    strncpy(track->omtechstuff, reason, OMTECHSTUFFLEN); 
   }
-  strncpy(track->omtechstuff, reason, OMTECHSTUFFLEN); 
 #endif 
 }
 
diff --git a/generic/Track.h b/generic/Track.h
index 98bc4c5..b52cc9c 100644
--- a/generic/Track.h
+++ b/generic/Track.h
@@ -38,11 +38,8 @@ void *ZnSendTrackToOm(void *ptr, void *item, int *x, int *y,
                          int *label_width, int *label_height,*/
                       int *rho, int *theta, int *visibility, int *locked,
                       int *preferred_angle, int *convergence_style);
-void ZnSetLabelAngleFromOm(void *ptr, void *item, int rho, int theta
-#ifdef OM_TECH
-                           , char *reason /* Technical data explaining algorithm processing */
-#endif
-                           );
+void ZnSetLabelAngleFromOm(void *ptr, void *item, int rho, int theta,
+                           char *reason); /* Technical data explaining algorithm processing */
 void ZnQueryLabelPosition(void *ptr, void *item, int theta,
                           int *x, int *y, int *w, int *h);  
 void ZnSetHistoryVisibility(ZnItem item, int index, ZnBool visibility);
diff --git a/generic/tkZinc.c b/generic/tkZinc.c
index 2eeb75f..d4aac38 100644
--- a/generic/tkZinc.c
+++ b/generic/tkZinc.c
@@ -6678,8 +6678,7 @@ Configure(Tcl_Interp            *interp,/* Used for error reporting. */
       if (entry != NULL) {
         grp = (ZnItem) Tcl_GetHashValue(entry);
         if (grp->class == ZnGroup) {
-          OmRegister((void *) wi, ZnSendTrackToOm,
-                     ZnSetLabelAngleFromOm, ZnQueryLabelPosition);
+          OmRegister((void *) wi, ZnSendTrackToOm, ZnSetLabelAngleFromOm, ZnQueryLabelPosition);
           wi->om_group = grp;
         }
       }
@@ -7008,8 +7007,7 @@ Configure(Tcl_Interp            *interp,/* Used for error reporting. */
         if (entry != NULL) {
           grp = (ZnItem) Tcl_GetHashValue(entry);
           if (grp->class == ZnGroup) {
-            OmRegister((void *) wi, ZnSendTrackToOm,
-                       ZnSetLabelAngleFromOm, ZnQueryLabelPosition);
+            OmRegister((void *) wi, ZnSendTrackToOm, ZnSetLabelAngleFromOm, ZnQueryLabelPosition);
             wi->om_group = grp;
           }
         }