Ticket #74 (closed enhancement: fixed)

Opened 9 months ago

Last modified 6 months ago

Change GkiKernel.getStdin/out/err to use local over default when defined and a tty

Reported by: sontag Assigned to: sontag
Priority: normal Milestone: PyRAF 1.5 stsci_python 2.6
Component: none Version:
Severity: normal Keywords:
Cc:

Description

We had an external user (ESO) request a change to GkiKernel?.getStdin/out/err() so that they return the self.stdin/out/err members over the default arg, in the case when the local members are defined and pass the isatty() test. Eso is redirecting PyRAF i/o to use it embedded within their application.

Because PyRAF scripts can be run via redirection and not (interactively), it is difficult to tell ahead of time what this might break. Logically, the change will only affect the case where a defined non-tty default is used in conjunction with a tty-like local attr. In this case, now the local attr would be returned instead of the default. This seems to be in keeping with the idea of default args, but it s unclear if the GkiKernel? functions are being called in "expected" ways.

Change History

12/03/07 16:38:15 changed by sontag

  • status changed from new to assigned.

12/10/07 09:02:33 changed by sontag

  • status changed from assigned to closed.
  • resolution set to fixed.

02/28/08 09:03:05 changed by sontag

  • milestone changed from stsci_python 2.6 release to PyRAF 1.5 stsci_python 2.6.

Milestone stsci_python 2.6 release deleted