Skip to content

Fix: mdsconnect of IDL API now always updates !MDS_SOCKET #3009

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
mwinkel-dev wants to merge 6 commits into
base: alpha
Choose a base branch
from
+16 −9
Draft

Fix: mdsconnect of IDL API now always updates !MDS_SOCKET #3009

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
2fa303f
Fix: mdsconnect of IDL API now always updates !MDS_SOCKET
mwinkel-dev Dec 16, 2025
68a8a34
Fix: create !MDS_SOCKET if it doesn't exist
mwinkel-dev Dec 17, 2025
155103b
FIx: correct typo regarding !MDS_SOCKET
mwinkel-dev Dec 17, 2025
5e2a4e5
Fix: change from Long to Int so IDL tests pass
mwinkel-dev Dec 17, 2025
bc211e6
Fix: the MdsConnect procedure now accpets an optional database keyword
mwinkel-dev Dec 19, 2025
cb78208
Fix: add the /database keyword to second MDSDbConnect procedure
mwinkel-dev Dec 19, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 4 additions & 6 deletions idl/mdsconnect.pro
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -111,6 +111,7 @@ pro Mds$SendArg,sock,n,idx,arg
return
end

; This procedure is likely obsolete
Copy link
Contributor Author

@mwinkel-dev mwinkel-dev Dec 16, 2025 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This procedure is not used anywhere in the MDSplus source code. Plus it appears to be replaced by the mdsconnect procedure.

However, there is a chance that customers have old IDL programs that use this routine.

We should select one of these options:

  • delete the procedure, or
  • keep the procedure, or
  • add a print statement to it indicating that it is deprecated and the user should instead use mdsconnect

pro mds$connect,host,status=status,quiet=quiet,port=port
on_error,2
mds$disconnect,/quiet
Expand Down Expand Up @@ -141,14 +142,12 @@ pro mds$disconnect,status=status,quiet=quiet
return
end


; This procedure is likely obsolete
Copy link
Contributor Author

@mwinkel-dev mwinkel-dev Dec 16, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Delete, keep or deprecate?

pro connect,host,_EXTRA=e
mds$connect,host,_EXTRA=e
return
end

;*/


pro mdsconnect,host,status=status,quiet=quiet,port=port,socket=socket
forward_function mds_keyword_set
Expand All @@ -163,9 +162,8 @@ pro mdsconnect,host,status=status,quiet=quiet,port=port,socket=socket
sockmin=sockmin()
if (sock ge sockmin) then begin
status = 1
if not mds_keyword_set(socket=socket) then $
!MDS_SOCKET = sock $
else $
!MDS_SOCKET = sock
if mds_keyword_set(socket=socket) then $
socket = sock
endif else begin
if not keyword_set(quiet) then message,'Error connecting to '+host,/IOERROR
Expand Down