Merge lp:~jontai/openvista-gtm-integration/bug381117 into lp:openvista-gtm-integration

Proposed by Jon Tai
Status: Merged
Merged at revision: not available
Proposed branch: lp:~jontai/openvista-gtm-integration/bug381117
Merge into: lp:openvista-gtm-integration
Diff against target: None lines
To merge this branch: bzr merge lp:~jontai/openvista-gtm-integration/bug381117
Reviewer Review Type Date Requested Status
JSHER Needs Information
Review via email: mp+6838@code.launchpad.net
To post a comment you must log in.
Revision history for this message
JSHER (joel-sher) wrote :

Jon,
would you want to retain

I Y="* D DOLRO Q

Otherwise if Y is "*" there is a syntax error.
CCDH>S Y="*"

CCDH>S X="^JDSY("

CCDH>D ORDER^%ZOSV

 S (Y,Y1)=$P(Y,"*",1) I $D(@Y)=0 F %=0:0 S Y=$O(@Y) Q:Y=""!(Y[Y1)

(This error is true in the current CACHE version also)
                      ^
<SYNTAX>ORDER+1^%ZOSV

review: Needs Information
Revision history for this message
Jon Tai (jontai) wrote :

So just to be perfectly clear.. you're suggesting adding the line:

I Y="*" D DOLRO Q

as the first line of ORDER^ZOSVGUX and ORDER^ZOSVONT?

So the ORDER subroutine in both would read:

ORDER ;SAVE PART OF SYMBOL TABLE IN LOCATION SPECIFIED BY X
 I Y="*" D DOLRO Q
 S (Y,Y1)=$P(Y,"*",1) I $D(@Y)=0 F %=0:0 S Y=$O(@Y) Q:Y=""!(Y[Y1)
 Q:Y="" S %=$D(@Y) S:%#2 @(X_"Y)="_Y) I %>9 S %X=Y_"(",%Y=X_"Y," D %XY^%RCR
 F %=0:0 S Y=$O(@Y) Q:Y=""!(Y'[Y1) S %=$D(@Y) S:%#2 @(X_"Y)="_Y) I %>9 S %X=Y_"(",%Y=X_"Y," D %XY^%RCR
 K %,X,Y,Y1

Revision history for this message
JSHER (joel-sher) wrote :

Yes,add the line as you describe.

> So just to be perfectly clear.. you're suggesting adding the line:
>
> I Y="*" D DOLRO Q
>
> as the first line of ORDER^ZOSVGUX and ORDER^ZOSVONT?
>
> So the ORDER subroutine in both would read:
>
> ORDER ;SAVE PART OF SYMBOL TABLE IN LOCATION SPECIFIED BY X
> I Y="*" D DOLRO Q
> S (Y,Y1)=$P(Y,"*",1) I $D(@Y)=0 F %=0:0 S Y=$O(@Y) Q:Y=""!(Y[Y1)
> Q:Y="" S %=$D(@Y) S:%#2 @(X_"Y)="_Y) I %>9 S %X=Y_"(",%Y=X_"Y," D %XY^%RCR
> F %=0:0 S Y=$O(@Y) Q:Y=""!(Y'[Y1) S %=$D(@Y) S:%#2 @(X_"Y)="_Y) I %>9 S
> %X=Y_"(",%Y=X_"Y," D %XY^%RCR
> K %,X,Y,Y1

49. By Jon Tai

add check for Y="*" at Joel's suggestion

50. By Jon Tai

add pristine ZOSVONT

51. By Jon Tai

add the same Y="*" check for the Cache version of ZOSV

Revision history for this message
Jon Tai (jontai) wrote :

Alright, I've added the line for both Cache and GT.M variants of ZOSV. Did you have any other comments, or should I merge this branch into mainline?

52. By Jon Tai

sneak in an extra newline on the system user name prompt

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'mumps/ZOSVGUX.m'
2--- mumps/ZOSVGUX.m 2009-04-15 03:13:40 +0000
3+++ mumps/ZOSVGUX.m 2009-05-27 23:12:59 +0000
4@@ -1,4 +1,4 @@
5-%ZOSV ;SFISC/AC,PUG/TOAD,HOU/DHW,MSC/JDA - View commands & special functions. ;09/03/2003 15:34
6+%ZOSV ;SFISC/AC,PUG/TOAD,HOU/DHW,MSC/JDA/JKT - View commands & special functions. ;27MAY2009
7 ;;8.0;KERNEL;**275,MSC**;Jul 10, 1995
8 ;
9 ACTJ() ; # active jobs
10@@ -114,14 +114,11 @@
11 . I $D(@Y)>9 S %X=Y_"(",%Y=X_"Y," D %XY^%RCR
12 K %X,%Y,Y Q
13 ;
14-ORDER ;SAVE PARTS OF SYMBOL TABLE IN LOCATION SPECIFIED BY X
15- ;PARTS INDICATED BY X1("NAMESPACE*")="" ARRAY
16- I $D(X1("*"))#2 D DOLRO Q
17- S X1="" F S X1=$O(X1(X1)) Q:X1="" D
18- . S (Y,Y1)=$P(X1,"*") I $D(@Y)=0 F S Y=$O(@Y) Q:Y=""!(Y[Y1)
19- . Q:Y="" S %=$D(@Y) S:%#2 @(X_"Y)="_Y) I %>9 S %X=Y_"(",%Y=X_"Y," D %XY^%RCR
20- . F S Y=$O(@Y) Q:Y=""!(Y'[Y1) S %=$D(@Y) S:%#2 @(X_"Y)="_Y) I %>9 S %X=Y_"(",%Y=X_"Y," D %XY^%RCR
21- K %,%X,%Y,Y,Y1
22+ORDER ;SAVE PART OF SYMBOL TABLE IN LOCATION SPECIFIED BY X
23+ S (Y,Y1)=$P(Y,"*",1) I $D(@Y)=0 F %=0:0 S Y=$O(@Y) Q:Y=""!(Y[Y1)
24+ Q:Y="" S %=$D(@Y) S:%#2 @(X_"Y)="_Y) I %>9 S %X=Y_"(",%Y=X_"Y," D %XY^%RCR
25+ F %=0:0 S Y=$O(@Y) Q:Y=""!(Y'[Y1) S %=$D(@Y) S:%#2 @(X_"Y)="_Y) I %>9 S %X=Y_"(",%Y=X_"Y," D %XY^%RCR
26+ K %,X,Y,Y1
27 Q
28 ;
29 PARSIZ ;

Subscribers

People subscribed via source and target branches