| 00:03.54 | *** join/#brlcad jbschw (~jbschw@ool-4355ee10.dyn.optonline.net) | |
| 01:40.04 | *** join/#brlcad crdueck_ (~cdk@d173-238-127-19.home4.cgocable.net) | |
| 02:29.26 | DarkCalf | pokes brlcad with a stick |
| 02:51.50 | brlcad | moos |
| 03:07.45 | starseeker | cristina: I put that CMake logic together |
| 03:08.16 | starseeker | part of some earlier testing - I figured it might save you time |
| 03:11.50 | starseeker | cristina: this might be a useful hit for those errors - http://stackoverflow.com/questions/5269957/looser-throw-specifier-error-in-c |
| 03:19.28 | starseeker | cristina: if we did end up using goblin (or, for that matter, adaptagrams) we'd eventually want to look at integrating the necessary bits into src/other |
| 03:20.40 | starseeker | cristina: that's not something you need to worry about at the moment, unless it makes it easier for you to work with goblin |
| 03:22.20 | starseeker | cristina: my initial impression when I looked at goblin was that there wasn't a whole lot of activity on the project in the last few years |
| 03:25.16 | starseeker | That's one of the advantages adaptagrams has - they seem to be a project with current activity |
| 03:25.47 | starseeker | I mainly didn't try a CMake build with them because it's a more complex proposition |
| 03:37.56 | *** join/#brlcad xth1 (~thiago@187.106.53.173) | |
| 07:59.01 | *** join/#brlcad kane__ (~kane@g226127211.adsl.alicedsl.de) | |
| 08:18.50 | *** join/#brlcad ksuzee (~ksu@193.151.107.42) | |
| 08:27.19 | ksuzee | starseeker: Hello, Cliff! Thanks for suggestion in my patch. I've never heard about astyle before. I've installed it and made a formatting with patch but I don't understand in which way I should use configuration file. Explain me please) |
| 08:52.05 | *** join/#brlcad stas (~stas@82.208.133.12) | |
| 08:52.24 | *** join/#brlcad merzo (~merzo@242-120-132-95.pool.ukrtel.net) | |
| 09:09.20 | brlcad | ksuzee: er, when you run astyle, it has about a hundred different options for specifying how to format the file(s) |
| 09:09.53 | brlcad | we specify our style/format in an astyle configuration file so you don't have to guess |
| 09:10.09 | brlcad | you provide that to astyle when you run it (see their documentation on how) |
| 09:10.57 | brlcad | when in doubt, read our HACKING file and edit your files accordingly -- you should ALSO be doing it or know how to do it manually |
| 09:11.36 | brlcad | very basic skill to be able to conform to another projects' style exactly (with OR WITHOUT a tool helping) |
| 09:16.34 | ksuzee | thank you Sean) Now I understand |
| 09:17.49 | ksuzee | also I have one more question by my another patch |
| 09:18.23 | ksuzee | I ran lowp with parameters (4 files) |
| 09:18.38 | ksuzee | But it shows an error |
| 09:18.55 | ksuzee | the same - before and after reduction |
| 09:19.42 | ksuzee | I think, I use parameters in wrong way |
| 09:21.14 | ksuzee | But I don't know where's my mistake. Can you give me and example, how you use it? It'll help me with testing another commands |
| 09:21.16 | andrei | hello! |
| 09:22.39 | andrei | I m currently looking into the test_rbtree patch I wrote |
| 09:23.04 | andrei | should I create a new addition with it( when it's done) or modify the existent one? |
| 09:41.03 | *** part/#brlcad kane__ (~kane@g226127211.adsl.alicedsl.de) | |
| 09:45.51 | *** join/#brlcad kane__ (~kane@g226127211.adsl.alicedsl.de) | |
| 10:07.22 | *** join/#brlcad Al_Da_Best (~Al_Da_Bes@elvyn-248-109.halls.student.lut.ac.uk) | |
| 10:49.01 | *** join/#brlcad Al_DC_Best (~Al_Da_Bes@elvyn-248-109.halls.student.lut.ac.uk) | |
| 10:54.49 | andrei | how can I automatically indent code?( I don t remember the name of the script) |
| 10:59.42 | cristina | hey andrei! |
| 11:00.14 | andrei | Hello cristina :) |
| 11:00.29 | cristina | do you know about astyle? |
| 11:01.31 | andrei | if you mean the one on sourceforge, yes? |
| 11:01.56 | andrei | should I use it ? |
| 11:03.22 | cristina | I never used it but I've seen that Sean recommended it to someone. I guess you could give it a try |
| 11:03.48 | andrei | oh, good to know. Thank you! |
| 11:04.11 | ksuzee | hi, it was me) |
| 11:04.32 | andrei | hey ksuzee |
| 11:04.44 | cristina | andrei: you're welcome |
| 11:05.36 | ksuzee | there's the astyle config file in /misc |
| 11:06.10 | ksuzee | you should use it togather with astyle programm |
| 11:06.50 | ksuzee | I've learnt about it only today |
| 11:07.38 | andrei | ah, I m currently syncronizing with the AUR repo's which takes forever |
| 11:07.48 | andrei | will see in a moment, thanks :) |
| 11:08.47 | ksuzee | welcome) |
| 11:10.47 | ksuzee | And have you found how to use it together with astyle config. Because I haven't found in MAN yet |
| 11:11.11 | CIA-65 | BRL-CAD: 03tbrowder2 * r50696 10/brlcad/trunk/src/libbu/test_bitv.c: bu_strcpy not found, use libc version |
| 11:12.10 | andrei | I haven't had the chance to install it yet, bu when I ll find out I ll get back to you:) |
| 11:13.06 | ksuzee | okay) Will wait) I did just this - sudo apt-get install astyle |
| 11:13.27 | cristina | starseeker: I've seen some stackoverflow answers to this problem. They all suggest using 'noexcept' to indicate throw(). I also tried modifying the code by replacing throw() with noexcept but errors still exist |
| 11:14.44 | cristina | starseeker: yes, there hasn't beed new activity for goblin since 2010 |
| 11:20.23 | CIA-65 | BRL-CAD: 03tbrowder2 * r50697 10/brlcad/trunk/src/libbu/test_bitv.c: ws, style |
| 11:20.58 | andrei | ksuzee, not sure if this is exactly what you need, but there are a few usage examples here |
| 11:21.00 | andrei | http://astyle.sourceforge.net/astyle.html#_Usage |
| 11:22.35 | cristina | ksuzee: |
| 11:22.52 | andrei | I think what you are interested in is using astyle --options=path-to-options-file yourfile beautifiedfile. |
| 11:23.01 | cristina | I think the needed command is astyle --options=astyle.opt <source_file> |
| 11:23.10 | cristina | :) |
| 11:24.06 | ksuzee | Thank, Andrei, thanks Cristina! I'll try now |
| 11:25.22 | andrei | cristina, I do get a error when calling the above command tho |
| 11:25.26 | andrei | Invalid option file options: |
| 11:25.27 | andrei | indent=force-tab-x |
| 11:27.04 | andrei | and that is rather wierd |
| 11:28.59 | cristina | ksuzee: hm, I just tested on a file where I haven't used tab at all and it didn't give me any error. |
| 11:29.23 | cristina | I assume it's a problem with the tab indentation |
| 11:29.41 | cristina | *sorry, andrei :D |
| 11:30.21 | andrei | the issue seems to be in the astyle.opt, at least from what I understand |
| 11:30.40 | cristina | yes |
| 11:31.41 | ksuzee | I've just tested and I have the same error as Andrei |
| 11:31.52 | cristina | just a second, I want to check something |
| 11:31.53 | andrei | cristina must be special :o |
| 11:32.23 | cristina | ok |
| 11:32.35 | ksuzee | hah) Maybe) |
| 11:32.39 | cristina | I think the problem is that it should be indent=force-tab=x |
| 11:32.47 | cristina | http://astyle.sourceforge.net/astyle.html |
| 11:33.45 | cristina | nope |
| 11:34.17 | andrei | hm,could you upload a copy of your astyle.opt somewhere? |
| 11:34.29 | andrei | I m curious if there are any differences |
| 11:34.46 | cristina | it's the same as the one in the repo |
| 11:35.03 | ksuzee | cristina: I've put indent=force-tab=x, but it throws the same error |
| 11:35.47 | andrei | astyle doesn't depend on distro |
| 11:39.07 | cristina | if I change the x into a number(say 8) the error stops appearing |
| 11:39.28 | cristina | but then there's another error in --max-instatement-indent=120 |
| 11:39.42 | ksuzee | I have this error too |
| 11:44.48 | andrei | I have to go, goodbye! |
| 11:46.40 | cristina | bye! |
| 11:50.51 | ksuzee | bye |
| 11:54.15 | cristina | ksuzee: it's strange, it doesn't accept the option in the command line either even though this option is documented |
| 12:01.13 | CIA-65 | BRL-CAD: 03Tbrowder 07http://brlcad.org * r3735 10/wiki/Undoing-a-commit: show shorthand use of the dot for URL, add a correct svn response to 'svn ci' |
| 12:01.51 | ksuzee | yes. I've already tried different variants, but the error is anyway |
| 12:09.44 | cristina | ksuzee: I downloaded their latest source from here: http://astyle.sourceforge.net/subversion.html and it works now |
| 12:18.46 | cristina | ksuzee: now the option indent=force-tab-x works as well. I guess the ubuntu repository version was older than this one |
| 12:20.29 | ksuzee | cristina: it works!!! Thanks, thanks! |
| 12:20.47 | cristina | ksuzee: you're welcome :) |
| 12:21.41 | ksuzee | I'll e-mail Andrei what to do) |
| 12:22.09 | cristina | ok, but I think he will see our discussion as he is still connected to this channel |
| 12:26.02 | ksuzee | cristina: Cliff told me to take the latest version but I was sure that Ubuntu has the latest one |
| 12:43.46 | *** join/#brlcad cristina (~quassel@188.24.71.175) | |
| 13:16.50 | *** join/#brlcad cristina (~quassel@188.24.71.175) | |
| 13:20.42 | *** join/#brlcad cristina (~quassel@188.24.71.175) | |
| 13:30.04 | *** part/#brlcad kane__ (~kane@g226127211.adsl.alicedsl.de) | |
| 13:48.25 | *** join/#brlcad louipc (~louipc@archlinux/fellow/louipc) | |
| 14:17.55 | CIA-65 | BRL-CAD: 03brlcad * r50698 10/brlcad/trunk/src/libbu/ (test_bitv.c test_booleanize.c test_quote.c test_vls.c): ws cleanup, run sh/ws.sh |
| 14:25.28 | CIA-65 | BRL-CAD: 03brlcad * r50699 10/brlcad/trunk/src/libbu/test_bitv.c: there's not bu_strcpy() but there is bu_strlcpy(). using strcpy() will cause regression testing to fail due to HACKING restrictions on using unsafe libc functions. |
| 14:28.16 | CIA-65 | BRL-CAD: 03brlcad * r50700 10/brlcad/trunk/src/rt/rtexample.c: apply spelling corrections from sf patch 3529931 by anuragmurty |
| 15:13.56 | *** join/#brlcad KimK (~Kim__@2001:470:1f0f:1042:4261:86ff:fe43:bcad) | |
| 15:30.37 | andrei | I got my version of astyle from the aur repo's , as I m using arch linux . I wonder why it waasn't the newest version there |
| 15:31.27 | andrei | and yes it did work for me aswell |
| 15:38.39 | *** join/#brlcad jbschw (~jbschw@ool-4355ee10.dyn.optonline.net) | |
| 15:45.38 | louipc | andrei: PKGBUILDs that build against the scm usually don't need to be changed unless there's some change in the build system, or dependencies. It will automatically grab the latest head. |
| 15:46.19 | andrei | I got astyle using yaourt |
| 15:46.41 | andrei | and I didn't edit the PKGBUILD either |
| 15:46.48 | andrei | ( I m not sure if I fully understand what you said) |
| 15:48.20 | louipc | andrei: well, the only version of astyle in the aur is the svn version |
| 15:48.48 | louipc | if you don't have astyle-svn then you didn't get it through the aur |
| 15:49.11 | andrei | ah |
| 15:49.17 | louipc | andrei: you grabbed a binary version? |
| 15:49.17 | andrei | there are 3 versions in aur |
| 15:49.35 | andrei | astyle , astyle-svn and astyle-doc |
| 15:49.39 | andrei | I got 1 and 3 |
| 15:49.50 | andrei | I didn't know about the atyle-svn part. |
| 15:50.24 | andrei | ah, I am sorry |
| 15:50.28 | andrei | it s from "community" |
| 15:50.32 | louipc | ah right.. community repo is technically the binary part of the aur... |
| 15:51.23 | andrei | so in similar situations I should avoid using community repo's and stick to the aur? |
| 15:51.39 | louipc | depends on what you want |
| 15:52.07 | louipc | I would grab binary packages unless they don't satisfy my need |
| 15:52.26 | andrei | good to know, thanks for the advice |
| 15:52.41 | andrei | I m pretty much an archlinux noob , but I love it compared to other distro's |
| 15:53.03 | louipc | yeah I find it's the least sucky hehe |
| 15:53.18 | andrei | I mean , it is harder to use than others, but the arch wiki provides help for many issues |
| 15:55.11 | louipc | I find it's easier, less confusing... simple |
| 16:34.16 | andrei | I have modified the red-black tree testing unit aswell, taking into account everything that was mentioned into the feedback comment. When they will be reviewed, please let me know if there is anything I need to correct. |
| 16:58.51 | *** join/#brlcad KimK (~Kim__@209.248.147.2.nw.nuvox.net) | |
| 17:44.27 | *** join/#brlcad cristina (~quassel@188.24.71.175) | |
| 17:45.45 | *** join/#brlcad andrei (andrei@5-12-88-164.residential.rdsnet.ro) | |
| 17:45.59 | *** join/#brlcad anuragmurty (~anurag@14.139.128.12) | |
| 17:55.59 | CIA-65 | BRL-CAD: 03tbrowder2 * r50701 10/brlcad/trunk/doc/README.Linux: update requirements based on more testing with Linux Mint LWDE 32-bit |
| 18:24.00 | CIA-65 | BRL-CAD: 03tbrowder2 * r50702 10/brlcad/trunk/src/libbu/vls.c: converted all flag variables in function bu_vls_vprintf to be contained in a special struct; added a reset function for them |
| 18:25.22 | CIA-65 | BRL-CAD: 03tbrowder2 * r50703 10/brlcad/trunk/src/libbu/vls.c: removed obsolete bu_vls %S format specifier |
| 18:26.46 | CIA-65 | BRL-CAD: 03tbrowder2 * r50704 10/brlcad/trunk/src/libbu/test_vls.c: commented out %S test, retained for use in soon-to-come bu_vls_vprintf mods |
| 18:27.48 | CIA-65 | BRL-CAD: 03tbrowder2 * r50705 10/brlcad/trunk/NEWS: notified users of removal of formerly deprecated and now obsolete %S bu_vls format specifier |
| 18:33.03 | *** join/#brlcad jbschw (~jbschw@ool-4355ee10.dyn.optonline.net) | |
| 18:34.44 | *** join/#brlcad ksuzee (~ksuzee91@193.151.107.42) | |
| 19:00.25 | CIA-65 | BRL-CAD: 03brlcad * r50706 10/brlcad/trunk/NEWS: NEWS is only for (non-developer) user-visible changes. annotate %S change to obsolete in doc/deprecation.txt |
| 19:00.37 | CIA-65 | BRL-CAD: 03brlcad * r50707 10/brlcad/trunk/doc/deprecation.txt: NEWS is only for (non-developer) user-visible changes. annotate %S change to obsolete in doc/deprecation.txt |
| 19:03.37 | CIA-65 | BRL-CAD: 03brlcad * r50708 10/brlcad/trunk/src/libbu/parse.c: remove the struct parse table support for '%S' since vls was just removed. |
| 19:11.22 | brlcad | ksuzee: if I just give you an example on how to use a filter tool like that, you wouldn't exactly learn how to test the next tool you modify |
| 19:11.42 | brlcad | and I can't give you an example on all the edits you might make, there are hundreds of tools ... |
| 19:12.11 | brlcad | see if there's a manual page, see if there's a usage statement, or read the source code when in doubt |
| 19:12.23 | brlcad | lowp is about as simple as code gets |
| 19:13.39 | brlcad | andrei: using astyle is good but don't let it be a crutch -- you should know how to make style conform even if you don't have a tool to help you, and you end up having to do it manually |
| 19:14.10 | andrei | I do have some issues regarding that matter, indeed. |
| 19:14.22 | brlcad | "M-x indent-region" in emacs with also do cleanup for you automatically |
| 19:14.33 | andrei | However, I did manage to get it working, have already submitted it |
| 19:14.41 | brlcad | plus we have two scripts to clean up files: sh/ws.sh and sh/indent.sh (the latter requires emacs) |
| 19:14.56 | andrei | ah, that s what I was looking for. |
| 19:17.30 | andrei | Aside of the two patches, what other patches or fixes should I attempt? |
| 19:17.41 | andrei | or should I just look on the TODO's list and see what I can handle? |
| 19:17.55 | CIA-65 | BRL-CAD: 03tbrowder2 * r50709 10/brlcad/trunk/doc/deprecation.txt: clarify instructions for the obsolete step |
| 19:18.48 | brlcad | as for astyle, you do have to use the latest sources -- I had to modify astyle to support our format, and those changes were just recently integrated |
| 19:20.36 | brlcad | andrei: since you have to work with libpkg, I suggest maybe writing either a unit test or a performance/scaling test |
| 19:20.47 | brlcad | where you either create your own client/server application (like tpkg) or write a script that runs tpkg with different parameters |
| 19:21.01 | brlcad | the latter makes the most sense for performance testing |
| 19:21.48 | brlcad | a unit test could be interesting and teach you how to use the library |
| 19:22.38 | andrei | sorry for the rather wierd question, but what exactly implies "performance" , speed only? |
| 19:24.38 | brlcad | ~dict 6 performance |
| 19:25.07 | brlcad | describing how it's functioning or operating .. speed would certainly be a factor in that |
| 19:27.05 | brlcad | what I'd probably do would be write a script that invokes tpkg client-server pairings testing package size vs data size vs time |
| 19:27.25 | brlcad | from that, I'd have a 3d graph of performance |
| 19:28.54 | brlcad | for each data size { for each package size { time sending from client to server } } |
| 19:30.00 | andrei | that sounds interesting :) |
| 19:30.10 | brlcad | alternatively, could test those factors independently: data size vs time package size vs time package count vs time .. etc |
| 19:31.04 | brlcad | yeah, that third one might matter: |
| 19:32.05 | brlcad | for some number of packages { for each data size { for each package size { time sending X packages of size Y in Z-size chunks from client to server } } } |
| 19:32.12 | brlcad | that's probably best |
| 19:32.54 | brlcad | could almost write the whole thing just like that as a shell script |
| 19:33.06 | brlcad | andrei: do you know posix shell scripting? |
| 19:33.23 | brlcad | or any scripting for that matter |
| 19:39.21 | andrei | the only thing I ever scripted in was bash |
| 19:39.34 | andrei | but I would like to learn shell scripting |
| 19:39.47 | louipc | bash is shell scripting ;) |
| 19:39.52 | andrei | I know |
| 19:40.20 | andrei | but I don t know what exactly to say |
| 19:40.38 | andrei | for example I wrote shell scripts to start / stop daemons, change different parameters |
| 19:40.53 | andrei | not very complicated |
| 19:42.40 | brlcad | but it sounds like it's at least an attainable and perhaps interesting approach to use for this task |
| 19:43.28 | brlcad | andrei: so maybe give that a go -- if you've already learned how to use tpkg, try invoking it within a script |
| 19:43.41 | brlcad | then wrap it in a loop that iterates over package sizes |
| 19:43.52 | brlcad | then wrap that in another loop that iterates over data sizes |
| 19:44.09 | brlcad | then wrap that in another loop that iterates over a certain number of packages... |
| 19:44.13 | brlcad | etc |
| 19:44.23 | brlcad | example to get you started: http://brlcad.org/wiki/Animation#Fly-around_animation |
| 19:45.00 | CIA-65 | BRL-CAD: 03Sean 07http://brlcad.org * r3736 10/wiki/Animation: /* Fly-around animation */ |
| 19:46.29 | brlcad | another example with a simple timer: http://brlcad.svn.sourceforge.net/svnroot/brlcad/brlcad/trunk/sh/orbit.sh |
| 19:47.28 | brlcad | submit an updated patch for each one of those steps as you make progress |
| 21:12.28 | *** join/#brlcad jbschw (~jbschw@ool-4355ee10.dyn.optonline.net) | |
| 21:24.45 | *** part/#brlcad ksuzee (~ksuzee91@193.151.107.42) | |
| 22:56.13 | CIA-65 | BRL-CAD: 03Cprecup 07http://brlcad.org * r3737 10/wiki/User:Cprecup/GSoC2012_progress: Adaptagrams samples - 26/05/2012 |
| 22:56.51 | CIA-65 | BRL-CAD: 03Cprecup 07http://brlcad.org * r3738 10/wiki/User:Cprecup/GSoC2012_progress: formatting error corrected |
| 23:59.17 | CIA-65 | BRL-CAD: 03tbrowder2 * r50710 10/brlcad/trunk/HACKING: fix typo |