| 00:06.55 | Notify | 03GCI:Arjun * Create an utility library (LIBBU) API unit test for any file: Task Claimed - I would like to work on this task. |
| 00:18.06 | Notify | 03GCI:Arjun * Create an utility library (LIBBU) API unit test for any file: Quick Questions - Hi, I decided to write a unit test for libbu/color.c, and I came accross a few concerns. First, am I... |
| 00:40.37 | *** join/#brlcad arjunyg (43bcf432@gateway/web/freenode/ip.67.188.244.50) |
| 01:25.54 | *** join/#brlcad arjunyg (~arjunyg@c-67-188-244-50.hsd1.ca.comcast.net) |
| 01:26.55 | arjunyg | anyone know anything about gcibot? |
| 02:00.15 | maths22 | what about it? |
| 02:08.25 | Notify | 03GCI:Gauravjeet Singh * Create unit tests for plane.c "dist" functions: Task Assigned - This task has been assigned to Andromeda Galaxy. You have 72 hours to complete this task, good luck! |
| 02:08.40 | Notify | 03GCI:Gauravjeet Singh * Create an utility library (LIBBU) API unit test for any file: Task Assigned - This task has been assigned to Arjun. You have 72 hours to complete this task, good luck! |
| 02:22.52 | *** join/#brlcad gjeet (~androirc@117.199.98.132) |
| 03:04.43 | brlcad | arjunyg: a little bit, why? |
| 04:22.47 | *** join/#brlcad YuGin (~IceChat77@182.55.146.247) |
| 05:58.36 | Notify | 03GCI:Melange * Implement a volume function for extruded bitmaps (EBM): No more Work can be submitted - Melange has detected that the deadline has passed and no more work can be submitted. The... |
| 06:01.01 | Notify | 03GCI:Sean * Creating Motion Typography video for BRL-CAD: Task Closed - Congratulations, this task has been completed successfully. |
| 06:03.27 | Notify | 03GCI:Sean * Implement a volume function for extruded bitmaps (EBM): Task Closed - Congratulations, this task has been completed successfully. |
| 06:05.32 | Notify | 03GCI:Sean * Implement a volume function for extruded bitmaps (EBM): either way, excellent work - Whether it's right or not, this is outstanding work. You've gone above and beyond by comparing to... |
| 06:33.29 | *** join/#brlcad caen23 (~caen23@92.81.177.20) |
| 07:03.56 | *** join/#brlcad caen23_ (~caen23@92.83.190.22) |
| 08:22.55 | *** join/#brlcad arjunyg (~arjunyg@c-67-188-244-50.hsd1.ca.comcast.net) |
| 09:05.23 | *** join/#brlcad YuGin (~IceChat77@182.55.146.247) |
| 09:59.41 | *** join/#brlcad kesha (~kesha@14.139.122.114) |
| 10:23.32 | *** join/#brlcad arjunyg (~arjunyg@c-67-188-244-50.hsd1.ca.comcast.net) |
| 10:41.35 | *** join/#brlcad merzo (~merzo@user-94-45-58-138-1.skif.com.ua) |
| 10:55.05 | *** join/#brlcad garnett (~Osiris_@195.24.220.16) |
| 11:10.07 | *** join/#brlcad Ch3ck_ (~Osiris_@195.24.220.16) |
| 12:23.52 | *** join/#brlcad arjunyg (~arjunyg@c-67-188-244-50.hsd1.ca.comcast.net) |
| 13:07.08 | *** join/#brlcad kesha (~kesha@14.139.122.114) |
| 14:09.18 | *** join/#brlcad microchip_sac (~microchip@223.239.199.70) |
| 14:11.39 | *** join/#brlcad kesha (~kesha@14.139.122.114) |
| 14:12.04 | *** join/#brlcad kesha (~kesha@14.139.122.114) |
| 14:24.16 | *** join/#brlcad arjunyg (~arjunyg@c-67-188-244-50.hsd1.ca.comcast.net) |
| 14:32.12 | *** join/#brlcad arjunyg (~arjunyg@c-67-188-244-50.hsd1.ca.comcast.net) |
| 14:32.59 | Notify | 03BRL-CAD:starseeker * 58843 brlcad/trunk/misc/CMake/BRLCAD_Summary.cmake: Shorten toplevel summary line slightly to avoid line wrap in 80x24 |
| 14:53.52 | Notify | 03BRL-CAD:starseeker * 58844 brlcad/trunk/misc/CMake/BRLCAD_Summary.cmake: Remove extra line |
| 14:55.38 | Notify | 03GCI:Shardul Chiplunkar * Add missing documentation (for any ONE command) #2: Ready for review - The work on this task is ready to be reviewed. |
| 14:56.29 | Notify | 03GCI:Shardul Chiplunkar * Add missing documentation (for any ONE command) #2: None - Please see the second uploaded patch file, it contains the changes. Thanks! |
| 15:02.11 | Notify | 03BRL-CAD:carlmoore * 58845 brlcad/trunk/src/librt/primitives/nmg/nmg_brep.cpp: fix spelling |
| 15:08.09 | Notify | 03BRL-CAD:starseeker * 58846 brlcad/trunk/misc/CMake/BRLCAD_Summary.cmake: Add a BRLCAD_SUMMARIZE_DEV_SETTINGS breakout that provides a place for more developer oriented summary printing entries - they won't be printed by default, but can be enabled by passing -DBRLCAD_SUMMARIZE_DEV_SETTINGS=ON to CMake |
| 15:22.24 | Notify | 03BRL-CAD:starseeker * 58847 brlcad/trunk/misc/CMake/BRLCAD_Summary.cmake: Whoops - do need a return when the terminal is actually wide enough for the top line |
| 15:25.08 | starseeker | brlcad: I'm not sure how to get the summary back down to 80x24 without a radical restructure - just the compilation flags take up almost that much space |
| 15:34.16 | Notify | 03BRL-CAD:starseeker * 58848 brlcad/trunk/misc/CMake/BRLCAD_Summary.cmake: Automate handling the width of the initial summary line. |
| 15:46.55 | Notify | 03BRL-CAD:starseeker * 58849 brlcad/trunk/misc/CMake/BRLCAD_Summary.cmake: Might as well make this configurable rather than hard coding it, especially since the rest of the summary logic is already more flexible. |
| 16:02.35 | Notify | 03BRL-CAD:carlmoore * 58850 brlcad/trunk/src/librt/primitives/nmg/nmg_brep.cpp: fix format specification due to warning message |
| 16:05.43 | Notify | 03BRL-CAD:tbrowder2 * 58851 (brlcad/trunk/src/lgt/reflect.c brlcad/trunk/src/libbn/plane.c and 16 others): various fixes from compiling with C++ compiler:correct function signaturesuse explict casts from void*use NULL for zeroing pointersremove unneeded 'auto' from variables (also conflicts with use in C++)change function format from K&R to ansiother fixes:ws, styletested with debug build on Debian 7, 64-bitmake |
| 16:05.45 | Notify | regress successfulmake test passed all but one (known to fail):254/289 Test #254: bn_plane_3pts_collinear_partial_same_2 ..............***Failed 0.00 sec |
| 17:33.26 | Notify | 03BRL-CAD:brlcad * 58852 brlcad/trunk/src/libtermio/termio.c: logic changed, should check if HAVE_TERMIOS_H is defined. |
| 19:20.25 | Notify | 03BRL-CAD:starseeker * 58853 (brlcad/trunk/src/libbu/kill.c brlcad/trunk/src/libbu/temp.c): Don't do the declarations if we're doing C++ |
| 19:20.35 | Notify | 03BRL-CAD:brlcad * 58854 brlcad/trunk/src/remrt/remrt.c: begin some cleanup to remove forward declarations and remove function titles |
| 19:26.16 | brlcad | 58853 feels like compounding levels of wrong ... that decl is dubious to begin with |
| 19:26.43 | brlcad | not to mention all the preprocessing goo now surrounding it based on feature testing, platforms, and now compiler type |
| 19:27.07 | brlcad | gone from bad to plaid |
| 19:31.06 | brlcad | starseeker: reducing summary display is secondary priority to discouraging it from growing more .. it's just getting to be TMI, info that's questionably actionable |
| 19:35.23 | Notify | 03BRL-CAD:brlcad * 58855 brlcad/trunk/src/remrt/remrt.c: move pkgswitch to the end, avoid all the forward decls |
| 19:48.50 | Notify | 03BRL-CAD:n_reed * 58856 (brlcad/trunk/src/libsysv/CMakeLists.txt brlcad/trunk/src/libsysv/memset.c and 4 others): pragma ignore of -Wunused-const-variable triggers -Wunknown-pragmas for older clang. Condition pragma on having the warning. |
| 20:06.45 | starseeker | brlcad: fair enough - didn't know if the decl was one of those "necessary" evils or what |
| 20:07.57 | starseeker | brlcad: I've proposed before that I convert the Tcl package lines in the summary from one-line-per to some more compact list - should I give it a go? |
| 20:08.10 | starseeker | that would be a decent haircut for the length |
| 20:09.01 | Notify | 03BRL-CAD:brlcad * 58857 brlcad/trunk/src/remrt/remrt.c: reorder to avoid all the forward declarations. one of the intended side-effects, it unveiled 3 functions that were unused. deleted tvsub, string2int, and mathtab_constant. made the callback funcs static to boot. |
| 20:09.11 | starseeker | brlcad: to be honest, that's why I figured the C++ clean-up might not be GCI material - too many ways to "solve" problems that don't solve underlying issues |
| 20:13.44 | Notify | 03BRL-CAD:brlcad * 58858 brlcad/trunk/src/remrt/remrt.c: ws comment cleanup, remove internal tabs |
| 20:14.51 | brlcad | starseeker: hardly anything is ever "necessary" .. just usually what was easiest or (sometimes) what was the best solution at a given point in time |
| 20:15.13 | brlcad | the decl might have been okay, but the wrapping it's needing says it's probably not any more .. there's a better place for that |
| 20:16.02 | brlcad | i'd hold on the tcl summary listing for the moment .. it's not really that annoying just yet -- how long is the listing and how much will that reduce? |
| 20:16.25 | brlcad | there's other ways we can display that info that might help too |
| 20:17.17 | brlcad | as for GCI tasking, I think there are categories of fixes that would be appropriate, like all of the (void*) conversions that are implicit in C but require a cast in C++ -- those are easy and generally won't introduce a bug |
| 20:17.23 | brlcad | (unless they get the type wrong) |
| 20:17.43 | brlcad | maybe can pull a complete make -k log to see what kinds of patterns there are and counts for how many |
| 20:19.26 | starseeker | currently there are 6 tcl/tk packages being listed individually, counting tk |
| 20:19.54 | starseeker | could probably get it down to 2 lines... compiled vs. system or some such |
| 20:22.00 | starseeker | kinda want to stick SMP-capable and verbose compilation flags in the developer summary category (off by default unless a specific setting is enabled...) |
| 20:22.17 | starseeker | ditto for debuggable release, since that's almost always on |
| 20:22.27 | starseeker | double ditto for profile |
| 20:22.54 | starseeker | is run-time debugging *ever* turned off? |
| 21:18.54 | Notify | 03BRL-CAD:tbrowder2 * 58859 brlcad/trunk/src/libbu/mappedfile.c: use bu_srtdup as advertised for the struct member; ws, style |
| 21:20.29 | Notify | 03BRL-CAD:tbrowder2 * 58860 (brlcad/trunk/src/rt/heatgraph.c brlcad/trunk/src/rt/view.c brlcad/trunk/src/util/ttcp.c): quell const string warnings; ws |
| 21:32.34 | Notify | 03BRL-CAD:tbrowder2 * 58861 brlcad/trunk/src/libbu/parse.c: ws, style |
| 22:05.26 | Notify | 03BRL-CAD:n_reed * 58862 (brlcad/trunk/CHANGES brlcad/trunk/CMakeLists.txt and 4 others): Carbon process-related functions used in dm_applicationfocus are deprecated in OS X 10.9. Removing dm_applicationfocus, since it only applies to OS X, and XQuartz has a "Focus On New Windows" preference that allows you to accomplish the same thing (e.g. giving mged focus on startup). Removing dm_applicationfocus eliminates the |
| 22:05.28 | Notify | need to check for Carbon in the first-party build logic. |
| 22:06.13 | maths22 | which cmake file would be best for a BRLCAD_ENABLE_COVERAGE option to be added to? |
| 22:10.54 | *** join/#brlcad FOSScookie (~brian@107-200-34-111.lightspeed.tulsok.sbcglobal.net) |
| 22:14.38 | Notify | 03BRL-CAD:n_reed * 58863 brlcad/trunk/src/rt/view.c: fix accidental char casts on fastf_t buffers |
| 22:24.05 | Notify | 03BRL-CAD:n_reed * 58864 brlcad/trunk/src/libbn/CMakeLists.txt: ignore bn_private.h |
| 22:52.48 | *** join/#brlcad FOSScookie (~brian@107-200-34-111.lightspeed.tulsok.sbcglobal.net) |
| 23:03.15 | starseeker | maths22: that's probably a top level option - sounds a little like BRLCAD_ENABLE_PROFILING |
| 23:16.15 | maths22 | I want to create it, but I don't know what file to add it to |
| 23:18.53 | maths22 | It looks like toplevel cmakelists + misc/CMake/BRLCAD_CompilerFlags.cmake |
| 23:49.59 | Notify | 03BRL-CAD:starseeker * 58865 brlcad/trunk/misc/CMake/BRLCAD_Summary.cmake: Move a few more items into the 'developers only' summary list |