00:13.27 |
CIA-128 |
BRL-CAD: 03starseeker * r49599
10/brlcad/trunk/src/other/tkhtml/src/htmldecode.c: quiet a
warning... |
00:30.26 |
*** join/#brlcad packrat
(~packrator@c-98-209-146-133.hsd1.mi.comcast.net) |
00:33.56 |
starseeker |
interesting... - reset_Tty in Cad_Exit in
btclsh is hanging during an XCode build... stuck in ioctl. Wonder
if that's because of how I'm trying to run things or a fundamental
conflict with xcodebuild... |
01:34.41 |
CIA-128 |
BRL-CAD: 03starseeker * r49600
10/brlcad/trunk/src/other/tkhtml/src/cssprop.tcl: Generate modern
functions, not old style... |
01:42.31 |
*** join/#brlcad juan_man
(~quassel@201.255.29.73) |
01:42.38 |
*** join/#brlcad juan_man
(~quassel@unaffiliated/juanman) |
04:13.20 |
*** join/#brlcad Stattrav
(~Stattrav@61.12.114.82) |
04:13.22 |
*** join/#brlcad Stattrav
(~Stattrav@unaffiliated/stattrav) |
05:18.50 |
*** join/#brlcad simion314
(~quassel@92.83.96.234) |
06:03.06 |
brlcad |
starseeker: that's a really low-level I/O
interface call, unlikely either |
06:03.37 |
brlcad |
probably some assumption in btclsh code (like
assuming a controlling terminal or something) that isn't
valid |
06:04.24 |
starseeker |
well, whatever it's up to removing that
reset_Tty call was the only thing that allowed me to get past that
point in the build with xcodebuild |
06:04.25 |
brlcad |
depends on what ioctl request was being
made |
06:05.13 |
starseeker |
src/bwish/tcl.c:180 |
06:05.42 |
brlcad |
at a glance, it's calling reset_Tty and using
fileno(stdin) .. that's probably not valid if stdin is
closed |
06:08.45 |
brlcad |
yeah, I'd bet the save_Tty() call is
failing |
06:09.23 |
brlcad |
stdin is probably closed however it's being
run through xcode |
06:09.41 |
brlcad |
non-blocking on the getp call, but then blocks
on the setp call |
06:10.37 |
brlcad |
bwish code there should make sure stdin is
actually open before calling save_Tty() and reset_Tty()
both |
06:12.01 |
brlcad |
starseeker: try wrapping them in: if
(isatty(fileno(stdin))) { |
06:13.45 |
brlcad |
otherwise, if that doesn't work you'll
probably need to probe stdin directly with something like this:
http://stackoverflow.com/questions/1594251/how-to-check-if-stdin-is-still-opened-without-blocking |
06:14.46 |
starseeker |
brlcad: alrightie - if I get a chance tomorrow
I'll give it a go |
06:15.08 |
starseeker |
thanks :-) |
06:16.12 |
brlcad |
looks like bwish doesn't check if it's a tty
before making those tty calls, so a simple call to isatty()
beforehand just might do it |
08:40.22 |
*** join/#brlcad Stattrav
(~Stattrav@61.12.114.82) |
08:40.23 |
*** join/#brlcad Stattrav
(~Stattrav@unaffiliated/stattrav) |
11:56.09 |
*** join/#brlcad juanman
(~quassel@unaffiliated/juanman) |
12:04.53 |
*** join/#brlcad juan_man
(~quassel@unaffiliated/juanman) |
13:00.18 |
*** join/#brlcad Stattrav
(~Stattrav@61.12.114.82) |
13:00.18 |
*** join/#brlcad Stattrav
(~Stattrav@unaffiliated/stattrav) |
13:39.42 |
*** join/#brlcad juanman
(~quassel@unaffiliated/juanman) |
14:23.01 |
*** join/#brlcad juanman
(~quassel@unaffiliated/juanman) |
17:13.12 |
``Erik |
starseeker: how can I replicate your block
issue with xcode? |
17:13.26 |
starseeker |
just try xcodebuild |
17:13.43 |
starseeker |
it should go for a while then hang when you
try to run something that uses btclsh to build |
18:04.44 |
*** join/#brlcad Stattrav
(~Stattrav@61.12.114.82) |
18:04.45 |
*** join/#brlcad Stattrav
(~Stattrav@unaffiliated/stattrav) |
18:39.58 |
CIA-128 |
BRL-CAD: 03starseeker * r49601
10/brlcad/trunk/src/bwish/tcl.c: Make a stab at playing nice with
the tty when doing an xcode build, per
http://stackoverflow.com/questions/1594251/how-to-check-if-stdin-is-still-opened-without-blocking |
18:49.33 |
CIA-128 |
BRL-CAD: 03starseeker * r49602
10/brlcad/trunk/src/other/tkhtml/src/ (23 files): W4 warning level
on MSVC complains about old style definitions - use cproto to
auto-update most of them (may be a few it couldn't
parse...) |
19:01.59 |
CIA-128 |
BRL-CAD: 03r_weiss * r49603
10/brlcad/trunk/src/libbn/ (mat.c plane.c): |
19:01.59 |
CIA-128 |
BRL-CAD: Updated libbn functions
bn_angle_measure, bn_dist_pt3_lseg3, |
19:02.00 |
CIA-128 |
BRL-CAD: bn_distsq_line3_pt3,
bn_isect_line3_line3, bn_isect_line3_plane, |
19:02.00 |
CIA-128 |
BRL-CAD: bn_isect_line_lseg,
bn_isect_lseg3_lseg3, bn_isect_pt_lseg, bn_mat_fromto, |
19:02.00 |
CIA-128 |
BRL-CAD: bn_vec_ortho, bn_dist_pt3_line3.
Performed code cleanup, fixed float 0.0 |
19:02.00 |
CIA-128 |
BRL-CAD: compares and added the 'unlikely'
macro to improve performance. Updated files |
19:02.01 |
CIA-128 |
BRL-CAD: 'plane.c' and 'mat.c'. |
19:03.42 |
CIA-128 |
BRL-CAD: 03starseeker * r49604
10/brlcad/trunk/src/other/tkhtml/src/cssprop.tcl: One more
old-style declarator in the template |
19:05.24 |
CIA-128 |
BRL-CAD: 03starseeker * r49605
10/brlcad/trunk/src/other/tkhtml/CMakeLists.txt: cssprop.c depends
on cssprop.tcl |
19:06.54 |
CIA-128 |
BRL-CAD: 03r_weiss * r49606
10/brlcad/trunk/src/librt/primitives/nmg/nmg_pt_fu.c: Updated
function 'bn_distsq_pt3_lseg3' in file 'nmg_pt_fu.c'. Fixed a float
0.0 compare and did some code cleanup. |
19:19.34 |
brlcad |
starseeker: I pondered that briefly when I
sent in my analysis summary, but figured "surely, coverity will
convert this to whatever units they desire" :) |
19:20.07 |
brlcad |
and they did for the rate of fixes -- I'd
provided a fix rate for the week, per day, per hour, per
minute |
19:20.38 |
brlcad |
as well as per team and per person |
19:34.59 |
starseeker |
heh |
19:36.26 |
CIA-128 |
BRL-CAD: 03starseeker * r49607
10/brlcad/trunk/src/other/tkhtml/src/ (7 files): More old-style
declarations |
19:42.36 |
*** join/#brlcad simion314
(~quassel@92.83.96.234) |
19:56.28 |
CIA-128 |
BRL-CAD: 03starseeker * r49608
10/brlcad/trunk/src/other/tkhtml/src/ (8 files): More old-style
declarations |
20:11.56 |
*** join/#brlcad simion314_
(~quassel@92.83.96.234) |
20:20.09 |
*** join/#brlcad Stattrav
(~Stattrav@unaffiliated/stattrav) |
20:20.55 |
*** join/#brlcad merzo
(~merzo@226-197-200-46.pool.ukrtel.net) |
20:41.11 |
CIA-128 |
BRL-CAD: 03starseeker * r49609
10/brlcad/trunk/src/other/tkhtml/src/ (htmltcl.c htmltext.c
htmltree.c htmlutil.c restrack.c): More old-style declarator
updates |
21:05.55 |
CIA-128 |
BRL-CAD: 03starseeker * r49610
10/brlcad/trunk/src/other/lemon/ (lemon.c lempar.c): Update to the
latest lemon sources. |
21:40.56 |
CIA-128 |
BRL-CAD: 03starseeker * r49611
10/brlcad/trunk/src/other/tkhtml/src/ (css.c cssparser.c
cssprop.tcl htmldecode.c): Take a stab at fixing a few of the
compiler warnings on Linux... |
22:12.07 |
CIA-128 |
BRL-CAD: 03r_weiss * r49612
10/brlcad/trunk/src/mged/mged.c: Fixed a bug in function
'std_out_or_err' file 'mged.c'. The 'line' array was being written
one byte past its size when line length is RT_MAXLINE. This was
causing an intermittent seg fault. |
22:37.03 |
*** join/#brlcad juanman
(~quassel@unaffiliated/juanman) |
23:05.18 |
*** join/#brlcad juan_man
(~quassel@unaffiliated/juanman) |
23:11.42 |
CIA-128 |
BRL-CAD: 03starseeker * r49613
10/brlcad/trunk/src/other/tkhtml/src/css.c: |
23:11.42 |
CIA-128 |
BRL-CAD: Apply a fix found by Dan Kennedy
- |
23:11.42 |
CIA-128 |
BRL-CAD:
http://groups.google.com/group/tkhtml3/browse_thread/thread/fcee715c54fae86f
- |
23:11.42 |
CIA-128 |
BRL-CAD: this may be the show-stopper for 64
bit windows, although I'm pretty sure it's |
23:11.42 |
CIA-128 |
BRL-CAD: not the only potential issue based on
the W4 results... |
23:14.52 |
CIA-128 |
BRL-CAD: 03starseeker * r49614
10/brlcad/trunk/src/tclscripts/archer/ (Archer.tcl ArcherCore.tcl):
try turning help back on in 64bit MSVC |