#BUGFIX by Stefan Reise
authorsr
Thu, 13 Feb 2020 14:24:24 +0100
changeset 25259 c6a993f1ead2
parent 25258 58b39832286e
child 25260 cad9ab22bee7
#BUGFIX by Stefan Reise undo a fix, which did now block expecco ALM startup class: NonPositionableExternalStream changed: #nextPutBytes:from:startingAt:
NonPositionableExternalStream.st
--- a/NonPositionableExternalStream.st	Wed Feb 12 18:46:25 2020 +0100
+++ b/NonPositionableExternalStream.st	Thu Feb 13 14:24:24 2020 +0100
@@ -991,6 +991,13 @@
 
     |offset remaining wasBlocking|
 
+    "sr: ATTENTION: keep the following #blocking: call here,
+     dont move it under the #handle is nil check,
+     if you move it, expecco ALM will freeze with 100% cpu usage during startup...
+     and see the comment inside the #handle is nil check,
+     which seems to assume this #blocking: call before"
+    wasBlocking := self blocking:false.
+
     handle isNil ifTrue:[
         "self blocking: above may set handle to nil!!"
         self errorNotOpen.
@@ -1003,7 +1010,6 @@
         ^ 0.
     ].
 
-    wasBlocking := self blocking:false.
 
     offset := initialOffset.
     remaining := initialWriteCount.
@@ -1032,6 +1038,8 @@
     wasBlocking ifTrue:[self blocking:true].
 
     ^ offset - initialOffset.
+
+    "Modified (format): / 13-02-2020 / 14:23:44 / Stefan Reise"
 !
 
 nextPutLine:aString