nikhilgupta10 / GridLAB-D

Other
1 stars 0 forks source link

#700 kml output fails, #2438

Closed nikhilgupta10 closed 7 years ago

nikhilgupta10 commented 7 years ago

The failure only happens on Windows release versions. See https://sourceforge.net/projects/gridlab-d/forums/forum/842562/topic/6750419 for details.

,

nikhilgupta10 commented 7 years ago

nikhilgupta10 imported these comments from Sourceforge: "dchassin": * status changed from new to accepted

Started work at r3852.

,

"dchassin": * owner changed from dchassin to jcfuller

This is fixed in r3853 (see /ticket/700).

,

"dchassin":It is worth noting that this type of problem is likely to come up for any module that attempts to write to a file using a file pointer given to it by the core. I don't know off hand of any other cases where this might happen, but in the future file operations should either be done on file handles obtained in the same DLL/EXE or through callback functions to the DLL/EXE they were opened in. It is obvious why when running on multiple hosts.

,

"dchassin":Once this is validated in 2.3, it will have to be promoted to trunk where the same problem is likely to occur.

,

"mhauer":\stream\ is sufficiently similar to the C++ STD \istream\, \ostream\, and related classes that another variable name should be selected to avoid name collisions in the future. An email discussion concluded that a different variable name shall be used in 3.0/trunk when this bug is fixed.

,

"jcfuller": * status changed from assigned to closed

Validated. This appears to fix it.

,