LabVIEW Idea Exchange

cancel
Showing results for 
Search instead for 
Did you mean: 
RyanWright

XML Close Reference VIs should not have standard error in functionality

The various instances of the polymoprhic XML Close VI all have standard error in functionality.  In other words, they run normally only if no error occurred before they run.  That can lead to references not getting closed if errors occur in sections of code that execute earlier.

 

Instead, the XML Close VI instances should run normally even if an error occurred before they run.  When calling the XML Close VI, it is a nuisance to have to clear the error passed in to it and then merge the previous error with the error output from the VI.

 

In addition, most of the other Close functions and VIs in LabVIEW run normally even if an error occurred before they run (Close File, UDP Close, VISA Close, Close Reference, DataSocket Close, etc.).

 

There may be other VIs and functions that have the same behavior as the current XML Close VI and they should be changed as well.  Examples include TDMS Close, Close Zip File VI, etc.

5 Comments
crossrulz
Knight of NI

The same needs to be done for the Network Stream's Destroy Stream Endpoint.


GCentral
There are only two ways to tell somebody thanks: Kudos and Marked Solutions
Unofficial Forum Rules and Guidelines
"Not that we are sufficient in ourselves to claim anything as coming from us, but our sufficiency is from God" - 2 Corinthians 3:5
DamienGaudry
Member

Add "RT FIFO Delete" to this list.

 

RT FIFO Delete.PNG

 

 

 

 

Darren
Proven Zealot

I've filed several CARs based on the feedback in this thread:

 

CAR 583443 - XML Close

CAR 583436 - TDMS Close (documentation only)

CAR 583438 - Close Zip File (documentation only)

CAR 583441 - Destroy Stream Endpoint

CAR 583440 - RT FIFO Delete

 

TDMS Close and Close Zip File are documentation only because they are already implemented in the correct way, i.e. they execute even on incoming error. It's just that their documentation doesn't reflect that.

Darren
Proven Zealot
Status changed to: Declined
Darren
Proven Zealot

Update on CAR 583441 - we have determined in LV R&D that the Destroy Stream Endpoint function does in fact execute normally even with an incoming error, so I have changed this CAR to a documentation CAR to get the error in parameter description for Destroy Stream Endpoint updated.