| 00:21.28 | *** part/#asterisk-dev Deeewayne (n=Deeewayn@nat/digium/x-64c0ca5622bf1b97) |
| 00:44.16 | *** join/#asterisk-dev sprite-- (n=sprite@12.228.1.97) |
| 00:44.45 | sprite-- | http://rafb.net/p/mhadml39.html : Possible 1.6 bug? Everything worked fine in 1.4. I've tried on 2 different computers now with the same result. |
| 01:13.10 | Corydon76-dig | sprite--: did you read the file that is referenced in there? |
| 01:14.04 | *** join/#asterisk-dev ZaVoid (n=zavoid@nj-76-6-39-193.dhcp.embarqhsd.net) |
| 01:18.51 | *** join/#asterisk-dev anthm (n=anthm@freeswitch/developer/anthm) |
| 01:22.19 | *** join/#asterisk-dev sasargen (n=chatzill@68-245-101-122.pools.spcsdns.net) |
| 01:30.44 | *** join/#asterisk-dev moy (n=moy@201.54.232.194) |
| 01:39.01 | sprite-- | Corydon76-dig: Yes. I've read through it. My settings haven't changed and they worked fine in 1.4. |
| 01:39.19 | seanbright | 1.6 != 1.4 |
| 01:39.35 | seanbright | ^^^ i hate people who say things like that |
| 01:39.40 | seanbright | <-- is one of those people |
| 01:40.14 | sprite-- | Well. [TK]D-Fender is logged into my box now and can't figure it out either. From what I can tell there shouldn't be any NAT setting changes between 1.4 and 1.6? |
| 01:40.37 | seanbright | i wouldn't know. i was just trolling. |
| 02:06.03 | *** join/#asterisk-dev sprite-- (n=sprite@12.228.1.131) |
| 02:09.00 | *** join/#asterisk-dev kamanashisroy (n=kamanash@119.30.34.3) |
| 03:41.53 | *** join/#asterisk-dev ZaVoid (n=zavoid@nj-76-6-39-193.dhcp.embarqhsd.net) |
| 03:48.08 | *** join/#asterisk-dev sasargen (n=chatzill@68.244.136.19) |
| 04:00.43 | *** join/#asterisk-dev snuff-work (n=jsewell@203.19.141.34) |
| 04:00.43 | *** mode/#asterisk-dev [+o snuff-work] by ChanServ |
| 04:25.19 | *** join/#asterisk-dev tschafer (n=tschafer@mason-wan-if0.carsoncity.k12.mi.us) |
| 04:27.59 | tschafer | Sorry to interrupt. I have a question about patching libpri... if it isn't too much trouble... |
| 04:30.50 | tschafer | Specifically, handling the absence of octet 5 in the bearer capability per Q.931 page 60, note 6 |
| 04:32.05 | seanbright | tschafer: better to ask on the asterisk-dev mailing list |
| 04:32.21 | seanbright | tschafer: the libpri maintainer isn't around at the moment |
| 04:32.39 | seanbright | there may even be a libpri mailing list |
| 04:33.16 | seanbright | there is, but only for commits... so yeah, try the asterisk-dev mailing list |
| 04:33.38 | tschafer | Thanks! No problem! |
| 06:13.58 | wedhorn | anyone around that can talk threads? |
| 06:14.45 | wedhorn | how expensive are they in terms of memory/cpu? if I want to delay something for a period of time and not block execution, should I start a new thread? |
| 06:51.34 | *** join/#asterisk-dev mattzerah (n=matt@ozvoip.dsl.onthenet.net) |
| 06:54.38 | *** join/#asterisk-dev philipp64 (n=chatzill@63.224.43.239) |
| 07:09.33 | *** join/#asterisk-dev sergee (n=serg@voip1.west-call.com) |
| 07:14.25 | *** join/#asterisk-dev oej (n=olle@ns.webway.se) |
| 07:14.25 | *** mode/#asterisk-dev [+o oej] by ChanServ |
| 07:34.24 | *** join/#asterisk-dev philipp64 (n=chatzill@63.224.43.239) |
| 07:38.13 | *** join/#asterisk-dev oej_ (n=olle@ns.webway.se) |
| 08:36.55 | *** join/#asterisk-dev oej (n=olle@ns.webway.se) |
| 08:36.55 | *** mode/#asterisk-dev [+o oej] by ChanServ |
| 09:40.11 | *** join/#asterisk-dev CrashHD (n=CrashHD@65.74.156.108) |
| 09:47.38 | *** join/#asterisk-dev eliel (n=Eliel@200.183.52.134) |
| 09:47.38 | *** mode/#asterisk-dev [+o eliel] by ChanServ |
| 11:19.28 | *** join/#asterisk-dev caio1982 (i=caio1982@CAcert-br/caio1982) |
| 11:59.22 | *** join/#asterisk-dev JBenden76_ (n=JBenden7@cblmdm72-240-122-140.buckeyecom.net) |
| 12:03.59 | *** join/#asterisk-dev shazaum (n=shazaum@unaffiliated/shazaum) |
| 12:10.11 | *** join/#asterisk-dev ZaVoid (n=zavoid@nj-76-6-39-193.dhcp.embarqhsd.net) |
| 12:21.52 | *** join/#asterisk-dev DarkRift (n=dark@bas1-sthubert21-1096590415.dsl.bell.ca) |
| 12:35.38 | *** join/#asterisk-dev ccesario (n=ccesario@189-19-9-100.dsl.telesp.net.br) |
| 12:51.01 | *** join/#asterisk-dev ZaVoid (n=zavoid@75.147.121.177) |
| 13:01.08 | *** join/#asterisk-dev sasargen (n=chatzill@173.100.18.91) |
| 13:42.35 | eliel | I am getting: 403 Forbidden (https://origsvn.digium.com) |
| 13:43.04 | eliel | Is there any general problem, or just my problem? |
| 13:43.16 | russellb | i haven't heard of that from anyone else ... |
| 13:43.30 | russellb | what URL |
| 13:43.46 | eliel | the one in the last msg https origsvn. |
| 13:43.58 | russellb | i meant, what command are you doing |
| 13:44.00 | eliel | while trying to commit |
| 13:44.04 | eliel | sorry |
| 13:44.07 | russellb | what branch |
| 13:44.16 | eliel | asterisk/trunk |
| 13:44.54 | russellb | fixed |
| 13:45.01 | russellb | someone typoed something in the access file |
| 13:45.02 | russellb | :) |
| 13:45.52 | eliel | russellb: thanks! |
| 13:46.42 | eliel | I now need to block the revisions in 1.6.1 and 1.6.0, or continue blocking just in 1.6.1? |
| 13:47.30 | russellb | good question |
| 13:48.23 | russellb | block in both to be safe i guess |
| 13:48.51 | eliel | ok |
| 13:49.47 | *** join/#asterisk-dev jtodd (n=jtodd@nat/digium/x-73b8f3e658be92d0) |
| 14:06.33 | *** join/#asterisk-dev bbryant (n=brett@c-68-59-20-153.hsd1.sc.comcast.net) |
| 14:06.33 | *** mode/#asterisk-dev [+o bbryant] by ChanServ |
| 14:12.37 | *** join/#asterisk-dev lmadsen (n=Leif@asterisk/documenteur-extraordinaire/blitzrage) |
| 14:12.37 | *** mode/#asterisk-dev [+o lmadsen] by ChanServ |
| 14:18.14 | *** join/#asterisk-dev ctooley (n=ctooley@68.25.194.170) |
| 14:18.14 | *** mode/#asterisk-dev [+o ctooley] by ChanServ |
| 14:22.29 | lmadsen | if anyone has any bugs they need me to look at today, please feel free to list them out! |
| 14:30.48 | ctooley | lmadsen, I've got a few, but MuffinMan doesn't have access to our internal mantis installation, you might be able to look at them. |
| 14:43.22 | *** join/#asterisk-dev asteriskmonkey (n=philip@69.77.169.14) |
| 14:45.15 | lmadsen | :) |
| 14:45.59 | eliel | lmadsen: M13836 |
| 14:46.26 | eliel | M13836 |
| 14:46.27 | MuffinMan | [assigned] [Asterisk] Channels/chan_dahdi 0013836: [patch] Use comma as the parameter separator. reported by eliel (Karma: +181.50) http://bugs.digium.com/view.php?id=13836 |
| 14:47.15 | lmadsen | thanks eliel, added to the list |
| 14:47.49 | eliel | thanks you |
| 15:06.06 | *** join/#asterisk-dev anthm (n=anthm@freeswitch/developer/anthm) |
| 15:19.50 | *** join/#asterisk-dev mog (n=mog@nat/digium/x-ffcc371e13a9c840) |
| 15:19.50 | *** mode/#asterisk-dev [+o mog] by ChanServ |
| 15:24.00 | *** join/#asterisk-dev Marquis42 (n=bfhbmw0@65-127-126-34.dia.static.qwest.net) |
| 15:35.32 | *** join/#asterisk-dev Deeewayne (n=Deeewayn@nat/digium/x-a45525bbf5756855) |
| 15:35.32 | *** mode/#asterisk-dev [+o Deeewayne] by ChanServ |
| 15:36.00 | *** join/#asterisk-dev ctooley (n=ctooley@207.160.210.210) |
| 15:36.00 | *** mode/#asterisk-dev [+o ctooley] by ChanServ |
| 15:36.54 | *** join/#asterisk-dev sruffell (n=sruffell@nat/digium/x-977c07f349e05686) |
| 15:36.55 | *** mode/#asterisk-dev [+o sruffell] by ChanServ |
| 15:37.16 | *** join/#asterisk-dev qdk (n=qdk@212.27.24.104.bredband.3.dk) |
| 15:44.38 | *** join/#asterisk-dev Corydon76-dig (i=peuce@pdpc/supporter/bronze/Corydon76-home) |
| 15:44.38 | *** mode/#asterisk-dev [+o Corydon76-dig] by ChanServ |
| 16:01.04 | *** join/#asterisk-dev psy0nid3 (n=chatzill@bookit-dev.com) |
| 16:17.45 | *** join/#asterisk-dev putnopvut (n=putnopvu@nat/digium/x-2478ca248281529e) |
| 16:17.45 | *** mode/#asterisk-dev [+o putnopvut] by ChanServ |
| 16:18.13 | *** join/#asterisk-dev Marquis42 (n=bfhbmw0@65-127-126-34.dia.static.qwest.net) |
| 16:21.10 | *** join/#asterisk-dev frawd (n=francois@59.Red-83-57-205.dynamicIP.rima-tde.net) |
| 16:24.56 | *** join/#asterisk-dev outtolunc (n=me@adsl-66-218-53-172.dslextreme.com) |
| 16:45.52 | *** join/#asterisk-dev CunningPike (n=arodgers@204.239.10.119) |
| 16:55.12 | *** join/#asterisk-dev ctooley (n=ctooley@doc-24-32-196-69.concordia.ks.cebridge.net) |
| 16:55.12 | *** mode/#asterisk-dev [+o ctooley] by ChanServ |
| 17:09.53 | *** join/#asterisk-dev oej (n=olle@ns.webway.se) |
| 17:09.53 | *** mode/#asterisk-dev [+o oej] by ChanServ |
| 17:16.16 | *** join/#asterisk-dev flujan (n=flujan@internet.nube.com.br) |
| 17:31.31 | *** part/#asterisk-dev asteriskmonkey (n=philip@69.77.169.14) |
| 17:31.36 | *** join/#asterisk-dev macli (n=macli@nmc.brc.ubc.ca) |
| 18:24.16 | *** join/#asterisk-dev linuxer_igor (n=linuxer_@mlsrj200152096p202.mls.com.br) |
| 18:27.12 | *** join/#asterisk-dev eliel (n=Eliel@200.183.52.134) |
| 18:27.12 | *** mode/#asterisk-dev [+o eliel] by ChanServ |
| 19:01.49 | *** join/#asterisk-dev eliel_ (n=Eliel@200.183.52.134) |
| 19:02.21 | *** mode/#asterisk-dev [+o eliel] by ChanServ |
| 19:40.23 | *** part/#asterisk-dev DagMoller (n=aguirre@unaffiliated/dagmoller) |
| 19:51.17 | *** join/#asterisk-dev rene- (n=renemend@200.34.66.137) |
| 20:10.08 | *** join/#asterisk-dev jasonwoot (n=some@bookit-dev.com) |
| 20:20.17 | lmadsen | eliel: ping |
| 20:20.27 | eliel | pong |
| 20:20.44 | lmadsen | DAHDISendCallreroutingFacility <-- this is a dialplan application? |
| 20:21.34 | eliel | yes |
| 20:21.46 | *** join/#asterisk-dev shazaum (n=shazaum@unaffiliated/shazaum) |
| 20:21.50 | lmadsen | hmmmm... I don't see that after installing dahdi and asterisk trunk |
| 20:22.06 | eliel | dahdi-tools also |
| 20:22.23 | lmadsen | maybe I need to double check my menuselect in asterisk |
| 20:22.29 | lmadsen | I'm installing linux-complete from dahdi |
| 20:22.47 | eliel | sorry, libpri also |
| 20:22.55 | lmadsen | oh, maybe that's what I'm missing then |
| 20:22.55 | lmadsen | thx |
| 20:22.59 | eliel | np |
| 20:29.10 | *** part/#asterisk-dev rene- (n=renemend@200.34.66.137) |
| 20:53.11 | eliel | lmadsen: thanks! what is the procedure when deprecating the | to use , |
| 20:54.25 | lmadsen | hmmmmmmm... that is an interesting question, because we have been moving to deprecate that usage between 1.4 -> 1.6, but my understanding is that we're not supposed to be deprecating anything, and that we are supposed to maintain backwards compatibility where possible. So I'm not sure if that means we need to support both syntaxes? Or if you just put a note in CHANGES ? |
| 20:59.00 | russellb | well, in 1.6, |
| 20:59.03 | russellb | | isn't supported anymore |
| 20:59.19 | russellb | however, IIRC, there is a backwards compat option available |
| 20:59.37 | russellb | we can't just support both like we used to ... these changes were made so that escaping was no longer a nightmare |
| 20:59.45 | russellb | and that stuff wasn't magically getting changed on you in the code |
| 21:02.27 | lmadsen | oh right, I forgot about that part |
| 21:02.44 | lmadsen | so I suppose it needs to be able to handle that configuration option |
| 21:03.05 | lmadsen | like the other applications |
| 21:06.05 | eliel | asterisk.conf [compat] ? |
| 21:13.14 | *** join/#asterisk-dev snuff-work (n=jsewell@203.176.96.250) |
| 21:13.14 | *** mode/#asterisk-dev [+o snuff-work] by ChanServ |
| 21:15.37 | russellb | i think, don't remember exactly |
| 21:16.15 | lmadsen | eliel: sounds right |
| 21:17.38 | eliel | what if instead of just making this dahdi app backward compatibility we set a app_delim=1.6 option in [compat] and modify STANDARD_APP_ARGS based on this option to use "," or "|" in every app? |
| 21:18.37 | russellb | thought it already did that |
| 21:18.42 | russellb | guess i'm wrong |
| 21:18.45 | lmadsen | ya... I think it should already be like that? |
| 21:19.18 | eliel | ok, i am blind or the code is not there :) |
| 21:20.17 | russellb | heh, I believe you |
| 21:20.35 | eliel | trust no one mr russell |
| 21:20.56 | eliel | ok, I will start working on that |
| 21:20.57 | russellb | no longer believes eliel |
| 21:21.02 | eliel | ups |
| 21:21.04 | Corydon76-dig | eliel: Happy belated birthday |
| 21:22.00 | eliel | Corydon76-dig: thanks! 26 now |
| 21:22.35 | eliel | and in brasil :( alone |
| 21:23.56 | russellb | eliel: come hang out in Huntsville :) |
| 21:25.09 | Corydon76-dig | eliel: Don't fret! You'll find a husband soon, I'm sure. |
| 21:25.33 | russellb | heh |
| 21:25.35 | eliel | heh |
| 21:26.04 | Corydon76-dig | russellb: would you agree that tab-completing a subset of available options is worse than not tab-completing at all? |
| 21:26.19 | russellb | good question |
| 21:26.44 | russellb | I suppose so |
| 21:27.02 | russellb | I'm just concerned that people use tab to see what's available, and if it shows something, but not everything ... |
| 21:27.02 | Corydon76-dig | M13969 |
| 21:27.04 | MuffinMan | [assigned] [Asterisk] General 0013969: Need tab completion of "core set debug X filename.c" reported by jtodd (Karma: +26.75) http://bugs.digium.com/view.php?id=13969 |
| 21:27.13 | Corydon76-dig | I've been trying to convince jtodd of that |
| 21:27.32 | russellb | what can't be tab completed in this case? |
| 21:27.36 | russellb | filenames? |
| 21:27.39 | snuff-work | likes new opera 10 alpha.. (if anyones interested) http://my.opera.com/desktopteam/blog/ |
| 21:27.40 | eliel | numbers |
| 21:27.48 | eliel | verbose levels |
| 21:27.49 | Corydon76-dig | All filenames available for 'core set debug' |
| 21:27.51 | eliel | and filenames |
| 21:28.17 | Corydon76-dig | because some of the filenames are not directly available in the binary, but only by searching the source directory |
| 21:28.20 | eliel | the problem is also when the next parameter is a number |
| 21:28.41 | Corydon76-dig | Right, because the number can be anywhere from 0 to 2.1 billion |
| 21:28.53 | Corydon76-dig | (all of which are reasonable) |
| 21:28.59 | russellb | ok, so clearly the number isn't going to fly |
| 21:29.04 | russellb | filenames we might be able to do |
| 21:29.12 | russellb | since we already have the ASTERISK_REGISTER_FILE_VERSION macro |
| 21:29.16 | russellb | it's registering filenames into a list |
| 21:29.26 | russellb | so you can pull from that |
| 21:29.34 | Corydon76-dig | That's a good hint. |
| 21:29.38 | russellb | :) |
| 21:29.55 | eliel | wow... i miss that one |
| 21:30.12 | russellb | check out "core show file version" |
| 21:30.38 | eliel | exactly what we need |
| 21:30.41 | russellb | yay |
| 21:31.19 | russellb | it just means we have to make sure every file has that macro in it ... |
| 21:31.27 | russellb | which I'm sure a number don't |
| 21:32.14 | Corydon76-dig | but some of those files don't have ast_debug() in them, so it probably doesn't matter |
| 21:32.20 | russellb | true |
| 21:32.24 | russellb | (or ast_verb()) |
| 21:32.54 | Corydon76-dig | You can set verbose only for a single file, too? |
| 21:33.04 | russellb | in trunk/1.6, yeah |
| 21:45.01 | lmadsen | M11485 |
| 21:45.03 | MuffinMan | [assigned] [Asterisk] Core/Portability 0011485: Alignment issues building on solaris sparc reported by snuffy (Karma: +63.25) http://bugs.digium.com/view.php?id=11485 |
| 21:45.07 | lmadsen | snuff-work: is that still an issue? |
| 21:45.30 | snuff-work | yes.. kpfleming is working with me |
| 21:45.40 | snuff-work | bit stale atm but its being looked at :) |
| 21:45.44 | lmadsen | ok :) |
| 21:45.58 | lmadsen | for some reason it looks like it is close :) |
| 21:46.54 | snuff-work | well there are 2 related ones.. one of em can be closed say refer to other.. if it hasn't been done.. |
| 21:47.58 | snuff-work | lmadsen: close that one.. just leave the 12932 open |
| 21:48.13 | lmadsen | oh ok, will do that. thanks! |
| 22:01.10 | *** join/#asterisk-dev eliel_ (n=Eliel@200.183.52.134) |
| 22:18.02 | *** join/#asterisk-dev JunK-Y (n=junky@modemcable156.137-20-96.mc.videotron.ca) |
| 22:19.01 | seanbright | thought... |
| 22:19.07 | russellb | counter-thought... |
| 22:19.11 | JunK-Y | hi guys |
| 22:19.12 | seanbright | M14001 |
| 22:19.13 | MuffinMan | [new] [Asterisk] Formats/format_pcm 0014001: [patch] Ability to use alaw files with extension ".alw" reported by henrikw (Karma: neutral) http://bugs.digium.com/view.php?id=14001 |
| 22:19.35 | seanbright | how about we allow format extensions to be overriden in configuration files? |
| 22:19.46 | russellb | eh .. |
| 22:19.50 | seanbright | worst idea ever? (tm) |
| 22:20.00 | Qwell | I really don't like that bug, heh |
| 22:20.01 | russellb | no, it's not terrible... it just seems like over-configurability |
| 22:20.10 | Qwell | not sure why, just seems very silly |
| 22:20.23 | seanbright | i don't disagree |
| 22:20.30 | russellb | Asterisk, in general, IMO, has _way_ too many freaking options |
| 22:20.35 | jtodd | well, .au is sometimes used as a suffix for alaw G711 I think. |
| 22:20.38 | russellb | many of which are necessary ... |
| 22:20.51 | russellb | some of which probably shouldn't exist |
| 22:20.53 | tzafrir_laptop | seanbright, where do you actually need it? |
| 22:20.59 | seanbright | tzafrir_laptop: i do not need it |
| 22:20.59 | jtodd | err.. perhaps used as alaw and ulaw, now that I think about it. |
| 22:21.03 | seanbright | tzafrir_laptop: at all |
| 22:21.09 | tzafrir_laptop | I suspect that "convert" is such a special case |
| 22:21.22 | JunK-Y | i agree, too much flexible in certain ways |
| 22:21.24 | jtodd | I'm not advocating additional format extensions, by the way. :-) |
| 22:21.41 | snuff-work | while all this activity is going on.. someone want to comment on this bug.. |
| 22:21.45 | snuff-work | M13783 |
| 22:21.47 | MuffinMan | [confirmed] [Asterisk] Channels/chan_sip/Registration 0013783: [patch] Cannot register with sip providers that require '@' in the username reported by navkumar (Karma: +1.00) http://bugs.digium.com/view.php?id=13783 |
| 22:21.48 | jtodd | quick! Everyone hide! |
| 22:22.02 | snuff-work | guy seems to have a point.. |
| 22:22.05 | Qwell | snuff-home: is that the macro change? that seemed reasonable |
| 22:22.08 | russellb | anyway, my opinion on that patch is fine, commit it, it's not a big deal |
| 22:22.16 | russellb | but make similar changes for ulaw |
| 22:22.18 | russellb | and move on |
| 22:22.19 | seanbright | i guess what i am saying... if we commit 14001, we should allow it to be overriden in conf. if we don't commit it, no other change required. |
| 22:22.23 | russellb | a minute wasted on that bug is a minute too much |
| 22:22.28 | seanbright | bleh |
| 22:22.44 | seanbright | wabout about .aLw? |
| 22:22.46 | seanbright | ducks |
| 22:22.59 | seanbright | what about* |
| 22:25.06 | seanbright | eliel_: 13991 and 13990 look fine to me, for what it's worth. |
| 22:25.13 | Marquis42 | Hrmmm... Working on a bug/feature/behavior in app_read, and wondering if anybody has an opinion on how to fix it: If you pass multiple ampersand-delimited filenames into app_read, then the usual # to end inputting digits does not work if you haven't yet put any digits in. I see *why* that is (the implementation of ast_readstring_full), but how should that get fixed? Any takers? I'm happy to code it up, just looking for |
| 22:25.14 | Marquis42 | direction. |
| 22:25.40 | russellb | sounds like you have plenty of direction already, heh |
| 22:25.44 | seanbright | yeha |
| 22:25.45 | Marquis42 | :) |
| 22:25.46 | russellb | not sure what more you can get other than the patch |
| 22:25.58 | Qwell | WELL |
| 22:25.59 | eliel_ | seanbright: thanks |
| 22:26.05 | Qwell | Marquis42: what's happening? Is read returning a #? |
| 22:26.30 | Qwell | that's one of those disagreements in required behavior |
| 22:26.38 | Marquis42 | Qwell: Nope, it skips through the files one at a time until it reaches the end of the list, then returns your variable with nothing in it. |
| 22:26.51 | Qwell | "but I want my extensions to start with a #!" |
| 22:26.59 | seanbright | yeah... sigh |
| 22:27.30 | Qwell | Marquis42: I don't understand |
| 22:27.34 | Marquis42 | russellb: There seems to be a couple of ways of fixing it. One is to change the way ast_readstring_full works a bit and then fix all uses of it (not a huge task, seems to be app_read, app_voicemail, app_directory, and *maybe* one other). |
| 22:28.08 | Qwell | If you dial 1#, it immediately stops playing files, and returns 1, but if you just hit #, it plays the next file? |
| 22:28.27 | Marquis42 | Correct |
| 22:29.40 | Qwell | yeah, I think # should just stop right away and return nothing |
| 22:29.53 | Qwell | that's what happens if you just play 1 file, right? |
| 22:29.53 | Marquis42 | Qwell: That's what I thought too. |
| 22:29.58 | Marquis42 | Yes |
| 22:30.30 | russellb | agreed |
| 22:35.35 | Marquis42 | russellb: OK, the other way I was thinking about fixing this seems much less reasonable and would require some duplication of code. So do you think that the aformentioned method of resolving this is ok? |
| 22:36.08 | russellb | sounds good to me |
| 22:36.29 | Marquis42 | ok, thanks. I'll code it up and submit it on the bug tracker. |
| 22:37.23 | Marquis42 | Oh, one more thing: Should this be fixed for 1.4 and then merged up through? Just wondering what branch I should base the diff from. |
| 22:37.39 | russellb | sounds like a bug to me |
| 22:38.10 | Marquis42 | That was my thinking. Alrighty, I'll base it off 1.4. Thanks again! |
| 22:55.49 | *** join/#asterisk-dev Deeewayne (n=Deeewayn@nat/digium/x-b133150c8aeea363) |
| 22:55.49 | *** mode/#asterisk-dev [+o Deeewayne] by ChanServ |
| 23:07.42 | kpfleming | huh... C99 allows for variable declarations in for loops, just like C++ |
| 23:08.10 | Qwell | for (int i = 0; ... ) ? |
| 23:08.16 | kpfleming | yep |
| 23:08.28 | Qwell | I used that quite a bit with .NET |
| 23:09.13 | Qwell | err, I mean, I would have, if I had ever used .NET. |
| 23:09.24 | Qwell | (...good save) |
| 23:12.52 | seanbright | and my other fav: foo((int[]){1, 2, 3, 4}); |
| 23:13.13 | kpfleming | yow |
| 23:13.16 | seanbright | heh |
| 23:13.50 | Qwell | what? |
| 23:14.35 | seanbright | i forget what the name is |
| 23:14.46 | seanbright | "compound literals" |
| 23:15.13 | Qwell | what does it...do? |
| 23:15.31 | seanbright | http://gcc.gnu.org/onlinedocs/gcc-4.3.2/gcc/Compound-Literals.html#Compound-Literals |
| 23:17.35 | seanbright | so can we do the 'for(int i = 0; ...' from now on then where applicable? |
| 23:17.40 | seanbright | or is that discouraged |
| 23:17.56 | putnopvut | I think you'll get a compiler warning based on Asterisk's settings. |
| 23:18.09 | putnopvut | I can give it a try though. |
| 23:20.37 | putnopvut | Yeah. |
| 23:20.40 | putnopvut | "app_dial.c:813: error: âforâ loop initial declaration used outside C99 mode |
| 23:20.40 | putnopvut | " |
| 23:20.48 | seanbright | "outside c99 mode"? |
| 23:21.13 | seanbright | i thought we were using c99 flags |
| 23:21.26 | kpfleming | i don't believe we are forcibly enabling c99 mode |
| 23:21.31 | seanbright | ahh |
| 23:34.53 | *** join/#asterisk-dev Deeewayne (n=Deeewayn@nat/digium/x-2f8d81f2b8350d23) |
| 23:34.53 | *** mode/#asterisk-dev [+o Deeewayne] by ChanServ |