| 00:04.19 | tofu_ | starseeker: see what just a little reminder e-mail can do... |
| 00:05.39 | brlcad | some issues aren't worth looking into until the second (or third) squeaking of the wheel |
| 01:04.09 | *** join/#brlcad stevegt_ (~stevegt@c-69-181-134-76.hsd1.ca.comcast.net) | |
| 01:30.14 | *** join/#brlcad juanman (~quassel@unaffiliated/juanman) | |
| 02:04.47 | starseeker | huh? Pro/ENGINEER is now Creo Element/Pro? |
| 02:06.20 | starseeker | and Unigraphics decides NX sounds cooler... sigh |
| 02:06.38 | brlcad | latest version has been Creo for nearly a year |
| 02:06.59 | starseeker | <snort> another selling point to add to our list |
| 02:07.13 | starseeker | BRL-CAD - consistent naming since the 1980s! |
| 02:08.00 | starseeker | 'cause you just can't get any cooler |
| 02:17.28 | starseeker | talk about a candidate for the Ig Nobel: http://idle.slashdot.org/story/12/04/15/1437222/japanese-researchers-create-a-crab-based-computer |
| 03:00.41 | *** join/#brlcad KimK (~Kim__@2001:470:1f0f:1042:4261:86ff:fe43:bcad) | |
| 03:55.51 | CIA-128 | BRL-CAD: 03brlcad * r50063 10/brlcad/trunk/src/librt/primitives/nmg/nmg_class.c: convert the goto retry logic into a do/while loop. this is in response to coverity cid 168 even though it looks like it was a bonefide false positive. |
| 04:03.12 | CIA-128 | BRL-CAD: 03brlcad * r50064 10/brlcad/trunk/src/conv/g-var.c: default case is unreachable, so join it up with the first case. should resolve cid 173. |
| 04:06.13 | CIA-128 | BRL-CAD: 03brlcad * r50065 10/brlcad/trunk/src/librt/primitives/nmg/nmg_misc.c: quell dead code issue for the always-false block by commenting it out since that's what the true-setting block is commented out. cid 174. |
| 04:30.31 | *** join/#brlcad KimK (~Kim__@209.248.147.2.nw.nuvox.net) | |
| 04:38.40 | CIA-128 | BRL-CAD: 03brlcad * r50066 10/brlcad/trunk/src/librt/primitives/nmg/nmg_misc.c: |
| 04:38.40 | CIA-128 | BRL-CAD: little tricky to see it, but coverity seems to have correctly identified a |
| 04:38.40 | CIA-128 | BRL-CAD: deadcode condition where the loop never iterated more than once (found_face gets |
| 04:38.40 | CIA-128 | BRL-CAD: set to true on first pass). cid 175. converted to an if-clause in order to |
| 04:38.40 | CIA-128 | BRL-CAD: preserve existing behavior while allowing for code simplification. |
| 04:45.15 | *** join/#brlcad Stattrav_ (~Stattrav@61.12.114.82) | |
| 05:01.11 | CIA-128 | BRL-CAD: 03brlcad * r50067 10/brlcad/trunk/src/libged/wdb_obj.c: |
| 05:01.12 | CIA-128 | BRL-CAD: fix three different deadcode coverity detections (cid 205, 206, 207), two of |
| 05:01.12 | CIA-128 | BRL-CAD: which are obvious, but the third not so much.. it looks like the arb8 block was |
| 05:01.12 | CIA-128 | BRL-CAD: completely replicated but not kept in sync. remove the seemingly weaker second |
| 05:01.12 | CIA-128 | BRL-CAD: version matching the non-wdb_obj version. |
| 05:27.06 | CIA-128 | BRL-CAD: 03brlcad * r50068 10/brlcad/trunk/src/liborle/rle.c: just what are the parens supposed to be doing here |
| 05:37.40 | CIA-128 | BRL-CAD: 03brlcad * r50069 10/brlcad/trunk/src/liborle/rle.c: unlikely to fix cid 405 (OVERRUN_STATIC), but it looks like false positive. simplify the pointer a bit to see if it has an effect on the detected defect. |
| 05:44.20 | CIA-128 | BRL-CAD: 03brlcad * r50070 10/brlcad/trunk/src/librt/primitives/brep/brep.cpp: fix coverity deadcode detection (cid 208) |
| 06:33.05 | *** join/#brlcad Tecan (~fsadf@ip-216-234-182-80.wildroseinternet.ca) | |
| 06:33.06 | *** join/#brlcad Tecan (~fsadf@unaffiliated/unit41) | |
| 06:33.19 | Tecan | !seen |
| 07:12.31 | *** join/#brlcad jbs103 (aefc2b83@gateway/web/freenode/ip.174.252.43.131) | |
| 08:37.07 | *** join/#brlcad Jak_o_Shadows (~Fake@unaffiliated/jak-o-shadows/x-0479135) | |
| 10:29.20 | *** join/#brlcad Jak_o_Shadows (~Fake@unaffiliated/jak-o-shadows/x-0479135) | |
| 11:39.39 | *** join/#brlcad stas (~stas@188.24.35.114) | |
| 11:55.02 | CIA-128 | BRL-CAD: 03tbrowder2 * r50071 10/brlcad/trunk/src/libbu/test_quote.c: make test_quote work and report failures |
| 11:56.24 | CIA-128 | BRL-CAD: 03tbrowder2 * r50072 10/brlcad/trunk/regress/CMakeLists.txt: add regression test for 'quote' (actually bu_vls_encode/decode) |
| 12:09.12 | ``Erik | heh, ptc and amd teaming up to get creo gpu aware http://tech.slashdot.org/story/12/04/15/2246257/amd-launches-partnership-with-cad-developer-ptc |
| 12:19.16 | *** join/#brlcad juanman (~quassel@unaffiliated/juanman) | |
| 12:35.23 | *** join/#brlcad Jak_o_Shadows (~Fake@unaffiliated/jak-o-shadows/x-0479135) | |
| 13:17.53 | *** join/#brlcad jordisayol (~jordisayo@unaffiliated/jordisayol) | |
| 13:21.11 | CIA-128 | BRL-CAD: 03brlcad * r50073 10/brlcad/trunk/src/librt/primitives/brep/brep.cpp: ws, indent, style cleanup |
| 13:35.58 | CIA-128 | BRL-CAD: 03brlcad * r50074 10/brlcad/trunk/src/librt/ (3 files in 2 dirs): |
| 13:35.58 | CIA-128 | BRL-CAD: looks like BVNode<BV>::isTrimmed() is trying to set a pointer, so we need a |
| 13:35.58 | CIA-128 | BRL-CAD: pointer to it for it to persist back to the caller. this fixes a deadcode |
| 13:35.58 | CIA-128 | BRL-CAD: condition detected by coverity (CID 209), which looks like a bonefide bug in the |
| 13:35.58 | CIA-128 | BRL-CAD: trimming code where it was keeping track of an adjacent trimmed face id (where |
| 13:35.59 | CIA-128 | BRL-CAD: it would always get set to -99). |
| 13:41.30 | brlcad | now that looks like the type of bug I was hoping to find at the offsite |
| 13:41.46 | *** join/#brlcad merzo (~merzo@134-0-133-95.pool.ukrtel.net) | |
| 13:42.06 | brlcad | if that adjacent face id is used anywhere, that was an obscure bug that would have screwed with trimming |
| 13:44.29 | *** join/#brlcad d_rossberg (~rossberg@BZ.BZFLAG.BZ) | |
| 14:20.26 | CIA-128 | BRL-CAD: 03brlcad * r50075 10/brlcad/trunk/src/mged/animedit.c: accfound is never set to true, so conditional is always true. eliminate the var, even though it looks like it implies bad logic.. |
| 14:36.48 | CIA-128 | BRL-CAD: 03brlcad * r50076 10/brlcad/trunk/src/mged/animedit.c: |
| 14:36.48 | CIA-128 | BRL-CAD: looks like coverity deadcode defect CID 214 is a false positive, complaining |
| 14:36.48 | CIA-128 | BRL-CAD: about curfound being always true, but try to be explicit anyways for code |
| 14:36.48 | CIA-128 | BRL-CAD: clarity. move the initial false case init out to the loop so it's clear that it |
| 14:36.48 | CIA-128 | BRL-CAD: starts false. |
| 14:40.09 | *** join/#brlcad merzo (~merzo@134-0-133-95.pool.ukrtel.net) | |
| 14:47.54 | CIA-128 | BRL-CAD: 03brlcad * r50077 10/brlcad/trunk/src/mged/chgview.c: quell a deadcode defect detected by coverity (cid 217) by cleaning up the logic to match the messages being printed. if it introduces a bug, then the message being printed is wrong/misleading. |
| 14:56.24 | CIA-128 | BRL-CAD: 03brlcad * r50078 10/brlcad/trunk/src/mged/polyif.c: since the original implementation commit in 1991, manually calculating normals for export has been disabled, so remove the code for it. fixes the deadcode issue reported in cid 218. |
| 15:00.21 | *** join/#brlcad ibot (~ibot@rikers.org) | |
| 15:00.21 | *** topic/#brlcad is BRL-CAD Open Source Solid Modeling || http://brlcad.org || logs: http://ibot.rikers.org/%23brlcad/ || BRL-CAD release 7.22.0 is forthcoming (eta: end of March) || BRL-CAD has applied to participate in GSoC 2012! | |
| 15:01.25 | CIA-128 | BRL-CAD: 03brlcad * r50079 10/brlcad/trunk/src/mged/share.c: turn SHARE_RESOURCE macro into a do/while loop so we can always include a trailing semicolon so code looks like valid code. |
| 15:02.40 | CIA-128 | BRL-CAD: 03brlcad * r50080 10/brlcad/trunk/src/mged/share.c: ws indent cleanup now that there are semicolons |
| 15:13.05 | CIA-128 | BRL-CAD: 03brlcad * r50081 10/brlcad/trunk/src/mged/share.c: prevent a deadcode condition detected by coverity by only calling SHARE_RESOURCE once outside the uflag if/else conditional. cid 219. |
| 15:37.33 | CIA-128 | BRL-CAD: 03brlcad * r50082 10/brlcad/trunk/src/mged/tedit.c: editor is set to jove as a fallback so it can't be null. just use it. cid 220. |
| 15:46.28 | CIA-128 | BRL-CAD: 03brlcad * r50083 10/brlcad/trunk/src/libbu/bitv.c: prevent a null dereference if we pass a null bitvector around. however, return an empty bitvector from new() so we never propagate a null. should fix coverity forward_null cid 242. |
| 15:55.22 | CIA-128 | BRL-CAD: 03brlcad * r50084 10/brlcad/trunk/src/librt/primitives/nmg/nmg_misc.c: make sure when duplicating a shell that we don't assume a face has a loop (crash). detected by coverity forward_null cid 244. |
| 16:06.38 | brlcad | woot, down to less than two dozen files |
| 16:06.49 | brlcad | 30 issues remaining |
| 16:15.50 | *** join/#brlcad jbschw (183b1a66@gateway/web/freenode/ip.24.59.26.102) | |
| 16:29.54 | *** join/#brlcad Al_Da_Best (~Al_Da_Bes@elvyn-248-109.halls.student.lut.ac.uk) | |
| 17:23.12 | CIA-128 | BRL-CAD: 03tbrowder2 * r50085 10/brlcad/trunk/src/libbu/test_quote.c: improve and correct test_quote function |
| 17:24.32 | CIA-128 | BRL-CAD: 03tbrowder2 * r50086 10/brlcad/trunk/src/libbu/test_quote.c: explain hack |
| 18:09.53 | *** join/#brlcad merzo (~merzo@8-65-132-95.pool.ukrtel.net) | |
| 18:19.53 | *** join/#brlcad merzo (~merzo@211-121-133-95.pool.ukrtel.net) | |
| 19:27.01 | *** join/#brlcad bhinesley (~bhinesley@adsl-108-79-235-86.dsl.bkfd14.sbcglobal.net) | |
| 19:33.13 | CIA-128 | BRL-CAD: 03n_reed * r50087 10/brlcad/trunk/regress/CMakeLists.txt: can ignore quote.sh once it's been commited |
| 20:48.53 | *** join/#brlcad stevegt_1 (~stevegt@cislunar.TerraLuna.Org) | |
| 20:52.07 | *** join/#brlcad DarkCalf (DC@173.231.40.98) | |
| 21:18.18 | CIA-128 | BRL-CAD: 03tbrowder2 * r50088 10/brlcad/trunk/regress/quote.sh: add missing script for vls_encode/decode test |
| 21:25.30 | CIA-128 | BRL-CAD: 03r_weiss * r50089 10/brlcad/trunk/src/libged/ (14 files): (log message trimmed) |
| 21:25.30 | CIA-128 | BRL-CAD: Updated files xpush.c list.c copyeval.c showmats.c find.c pathsum.c wcodes.c |
| 21:25.30 | CIA-128 | BRL-CAD: wdb_obj.c trace.c ged_private.h push.c get_obj_bounds.c prefix.c and edcodes.c |
| 21:25.30 | CIA-128 | BRL-CAD: within the libged library. Changed functions 'ged_list' (ie mged 'l' command, |
| 21:25.30 | CIA-128 | BRL-CAD: lowercase L) and ged_pathsum (ie mged 'paths' command) to accept an addition |
| 21:25.30 | CIA-128 | BRL-CAD: parameter '-t' meaning terse to reduce the amount of information displayed when |
| 21:25.31 | CIA-128 | BRL-CAD: using these commands. This is useful when you want to use the 'l' command on a |
| 21:27.41 | CIA-128 | BRL-CAD: 03r_weiss * r50090 10/brlcad/trunk/include/raytrace.h: Updated header file 'raytrace.h'. Changed the definition of function 'db_tree_funcleaf' by adding a fourth generic user pointer. |
| 21:32.31 | CIA-128 | BRL-CAD: 03r_weiss * r50091 10/brlcad/trunk/src/conv/iges/g-iges.c: Updated file 'g-iges.c' changing all references to function 'db_tree_funcleaf' to include a fourth generic user pointer. |
| 21:37.37 | CIA-128 | BRL-CAD: 03r_weiss * r50092 10/brlcad/trunk/src/librt/ (db_match.c db_tree.c): Updated files 'src/librt/db_match.c', 'src/librt/db_tree.c' changing all references to function 'db_tree_funcleaf' to include a fourth generic user pointer. |
| 21:39.37 | CIA-128 | BRL-CAD: 03r_weiss * r50093 10/brlcad/trunk/src/mged/utility1.c: Updated file 'src/mged/utility1.c' changing all references to function 'db_tree_funcleaf' to include a fourth generic user pointer. |
| 21:46.22 | *** join/#brlcad stas (~stas@188.24.35.114) | |