View Issue Details

Related ChangesetsJump to NotesJump to History
IDProjectCategoryView StatusDate SubmittedLast Update
0028381Open CASCADEOCCT:DRAWpublic2017-01-23 10:582017-09-29 16:24
ReporternbvAssigned Toapn  
PrioritynormalSeverityminor 
Status closedResolutionfixed 
Product Version7.1.0 
Target Version7.2.0Fixed in Version7.2.0 
Summary0028381: Draw Harness - DRAWEXE still hangs up immediatly when redirecting input stream on Windows
DescriptionThe fix for issue 0028350 does not solve the problem on my machine (DRAW hangs up as before - see hanging.png attached picture).
Steps To ReproduceTest case is not required.

Try in command window:

"draw.bat" - OK

"echo pload;exit | draw.bat" - HANGING
TagsNo tags attached.
Test case numberNot needed

Attached Files

  • hanging.png (18,759 bytes)

Relationships

related to 0028350 closedapn Draw Harness - DRAWEXE hangs up immediatly when redirecting input stream on Windows 

Activities

nbv

2017-01-23 10:58

developer  

hanging.png (18,759 bytes)

git

2017-01-23 11:05

administrator   ~0062886

Branch CR28381 has been created by nbv.

SHA-1: 8addee0d63a0b93bd27a4b572ca38ce64e54db78


Detailed log of new commits:

Author: nbv
Date: Mon Jan 23 11:05:16 2017 +0300

    0028381: Draw Harness - DRAWEXE hangs up immediatly when redirecting input stream on Windows

nbv

2017-01-23 11:06

developer   ~0062887

Dear Andrey,

Please review CR28381 branch. It fixes the problem on my machine.

kgv

2017-01-23 11:44

developer   ~0062889 

+      else if (fgetws(console_command, THE_COMMAND_SIZE, stdin))
+      {// fgetws() works only for characters within active locale (see setlocale())
+        console_semaphore = HAS_CONSOLE_COMMAND;
+      }

Please instead try removing check "if (anErr == ERROR_INVALID_HANDLE)" so that "isConsoleInput = false;" is set for any error.

On my system it was ERROR_INVALID_HANDLE, while on yours it is ERROR_INVALID_FUNCTION - have no idea why error code is different.

git

2017-01-23 12:17

administrator   ~0062891

Branch CR28381 has been updated by nbv.

SHA-1: 52d0d05f719488e391ce725bf76f0d1545bf81f2


Detailed log of new commits:

Author: nbv
Date: Mon Jan 23 12:17:26 2017 +0300

    Changes according to the last remarks.

nbv

2017-01-23 12:18

developer   ~0062892

Done.

Dear Kirill,

Please review the current state of CR28381 branch.

kgv

2017-01-23 12:24

developer   ~0062893

> Changes according to the last remarks.
This change should replace your original patch, not "include" it.
"else" statement will never occur because you are calling fgetws() before and GetLastError() has nothing to do with fgetws() return value.

git

2017-01-23 13:05

administrator   ~0062898

Branch CR28381 has been updated forcibly by nbv.

SHA-1: 17d79b5a25349bb33d5592e051aa1769af304527

nbv

2017-01-23 13:06

developer   ~0062899

Done.

Dear Kirill,

Please review the current state of CR28381 branch.

kgv

2017-01-23 13:45

developer   ~0062900

Please test the patch.

apv

2017-01-25 13:55

tester   ~0063002

Dear BugMaster,

Branch CR28381 from occt git-repository (and master from products git-repository) was compiled on Linux, MacOS and Windows platforms and tested.
SHA-1: 17d79b5a25349bb33d5592e051aa1769af304527

Number of compiler warnings:
occt component:
   Linux: 0 (0 on master)
   Windows: 0 (0 on master)
   MasOS: 0 (0 on master)
products component:
   Linux: 63
   Windows: 0
   MacOS: 1132

Regressions/Differences:
Not detected

Testing cases:
Not needed

Testing on Linux:
Total MEMORY difference: 92983626 / 92067637 [+0.99%]
Total CPU difference: 21075.62000000011 / 21062.120000000155 [+0.06%]

Testing on Windows:
Total MEMORY difference: 58324735 / 58324216 [+0.00%]
Total CPU difference: 19385.43306479856 / 19152.585972198493 [+1.22%]

git

2017-02-08 11:36

administrator   ~0063623

Branch CR28381 has been deleted by kgv.

SHA-1: 17d79b5a25349bb33d5592e051aa1769af304527

Related Changesets

occt: master 0f358da3

2017-01-23 08:05:16

nbv


Committer: apn Details Diff		 0028381: Draw Harness - DRAWEXE hangs up immediatly when redirecting input stream on Windows Affected Issues
0028381
mod - src/Draw/Draw_Window.cxx Diff File

Issue History

Date Modified Username Field Change
2017-01-23 10:58 nbv New Issue
2017-01-23 10:58 nbv Assigned To => apn
2017-01-23 10:58 nbv File Added: hanging.png
2017-01-23 11:05 git Note Added: 0062886
2017-01-23 11:06 nbv Note Added: 0062887
2017-01-23 11:06 nbv Status new => resolved
2017-01-23 11:06 nbv Steps to Reproduce Updated
2017-01-23 11:08 nbv Relationship added related to 0028350
2017-01-23 11:44 kgv Note Added: 0062889
2017-01-23 11:44 kgv Assigned To apn => nbv
2017-01-23 11:44 kgv Status resolved => assigned
2017-01-23 11:44 kgv Product Version => 7.1.0
2017-01-23 11:45 kgv Summary Draw Harness - DRAWEXE hangs up immediatly when redirecting input stream on Windows => Draw Harness - DRAWEXE still hangs up immediatly when redirecting input stream on Windows
2017-01-23 12:17 git Note Added: 0062891
2017-01-23 12:18 nbv Note Added: 0062892
2017-01-23 12:18 nbv Assigned To nbv => kgv
2017-01-23 12:18 nbv Status assigned => resolved
2017-01-23 12:24 kgv Note Added: 0062893
2017-01-23 12:25 kgv Assigned To kgv => nbv
2017-01-23 12:25 kgv Status resolved => assigned
2017-01-23 13:05 git Note Added: 0062898
2017-01-23 13:06 nbv Note Added: 0062899
2017-01-23 13:06 nbv Assigned To nbv => kgv
2017-01-23 13:06 nbv Status assigned => resolved
2017-01-23 13:45 kgv Note Added: 0062900
2017-01-23 13:45 kgv Assigned To kgv => bugmaster
2017-01-23 13:45 kgv Status resolved => reviewed
2017-01-24 17:24 apv Assigned To bugmaster => apv
2017-01-24 17:25 apv Test case number => Not needed
2017-01-25 13:55 apv Note Added: 0063002
2017-01-25 13:55 apv Assigned To apv => bugmaster
2017-01-25 13:55 apv Status reviewed => tested
2017-01-27 16:15 apn Changeset attached => occt master 0f358da3
2017-01-27 16:15 apn Assigned To bugmaster => apn
2017-01-27 16:15 apn Status tested => verified
2017-01-27 16:15 apn Resolution open => fixed
2017-02-08 11:36 git Note Added: 0063623
2017-09-29 16:21 aiv Fixed in Version => 7.2.0
2017-09-29 16:24 aiv Status verified => closed