| 01:51.14 | *** join/#asterisk-dev spatel (~spatel@pool-96-237-230-175.bstnma.fios.verizon.net) |
| 02:58.24 | *** join/#asterisk-dev gtjoseph (~gtjoseph@unaffiliated/gtj) |
| 02:58.24 | *** mode/#asterisk-dev [+o gtjoseph] by ChanServ |
| 05:05.29 | *** join/#asterisk-dev gtjoseph (~gtjoseph@unaffiliated/gtj) |
| 05:05.29 | *** mode/#asterisk-dev [+o gtjoseph] by ChanServ |
| 07:08.08 | *** join/#asterisk-dev pchero_work (~pchero@2a02:a210:2241:6480:6cc0:a060:a892:eb7) |
| 07:18.23 | *** join/#asterisk-dev pchero_work (~pchero@2a02:a210:2241:6480:6cc0:a060:a892:eb7) |
| 10:18.47 | *** join/#asterisk-dev spatel (~spatel@pool-96-237-230-175.bstnma.fios.verizon.net) |
| 12:28.02 | *** join/#asterisk-dev bford (uid283514@gateway/web/irccloud.com/x-rmzfpkayvxdfgwax) |
| 12:28.02 | *** mode/#asterisk-dev [+o bford] by ChanServ |
| 12:32.55 | *** join/#asterisk-dev pchero_work (~pchero@51.247.195.35.bc.googleusercontent.com) |
| 13:14.11 | seanbright | file: can we just use ast_asprintf() and avoid all of that stuff |
| 13:14.13 | seanbright | ? |
| 13:14.21 | seanbright | since we're heap allocating anyway |
| 13:14.41 | file | hrm, yes! |
| 13:15.05 | seanbright | do you want me to say as much on the review or was this conversation sufficient? |
| 13:17.58 | file | the requested action was completed successfully |
| 13:54.10 | *** join/#asterisk-dev kharwell (uid358942@gateway/web/irccloud.com/x-ccpklgacxpebrhey) |
| 13:54.10 | *** mode/#asterisk-dev [+o kharwell] by ChanServ |
| 14:06.04 | seanbright | the filename variable is not strictly required, or could be declared in the block in which it is used |
| 14:06.12 | seanbright | but those are minor things |
| 14:39.42 | coreyfarrell | does anyone know of any specific AMI response where the order of keys in the response is significant? |
| 14:39.59 | file | nope |
| 14:46.35 | *** join/#asterisk-dev spatel (~spatel@pool-96-237-230-175.bstnma.fios.verizon.net) |
| 14:48.02 | *** join/#asterisk-dev CELYA_ (~Thunderbi@2a01cb0583d31d00f4d7ddd1bc0987bc.ipv6.abo.wanadoo.fr) |
| 14:53.53 | seanbright | but changing the order is problematic if memory serves |
| 14:54.44 | file | the AMI v2 spec I think defines no guaranteed ordering |
| 14:57.10 | seanbright | right, but i tried to change the order one time and someone nixed it, trying to find the logs |
| 14:58.34 | seanbright | meh, can't find it. |
| 14:58.36 | seanbright | carry on. |
| 14:59.36 | coreyfarrell | thanks for the details.. I was asking because I wrote a node.js AMI client and was documenting the property which provides an array of name/value pairs in the original packet order.. wasn't sure if there was a known legitimate use for that or if it was 'just in case'. |
| 15:01.54 | seanbright | hmm |
| 15:02.04 | coreyfarrell | so if you have a packet with 'type: t1\nname: v1\ntype: t2\nname: v2\n' this would normally be decoded as {type: ['t1','t2'], name: ['v1','v2']}.. |
| 15:02.47 | seanbright | can messages have multiple headers with the same name? |
| 15:02.52 | coreyfarrell | I have a second property in the packet class which would provide `[['type', 't1'], ['name', 'v1'], ['type', 't2'], ['name', 'v2']]` |
| 15:03.17 | coreyfarrell | don't think the ordered array form is ever needed but whatever. |
| 15:03.22 | seanbright | https://issues.asterisk.org/jira/browse/ASTERISK-27072 |
| 15:03.54 | seanbright | i knew i remembered something |
| 15:04.26 | coreyfarrell | the originate request can take multiple `variable` headers but there is no significance of order between `variable` and other headers. |
| 15:06.40 | coreyfarrell | seanbright: thanks, that's a legit example of where you'd have to use the ordered array property of my packet class to understand the result. |
| 15:09.45 | seanbright | surely |
| 15:25.51 | *** join/#asterisk-dev pchero_work (~pchero@2a02:a210:2241:6480:2495:93ff:235d:754c) |
| 15:34.06 | file | seanbright: noice |
| 15:36.58 | seanbright | what'd i do now? |
| 15:37.37 | file | finding the ASTERISK Issue |
| 15:38.09 | seanbright | ah yeah |
| 15:38.12 | file | we were having our sprint review/retrospective/planning and then I had a quick call with the ever delightful malcolmd, so my response was delayed :P |
| 16:03.20 | *** join/#asterisk-dev CELYA_1 (~Thunderbi@lfbn-nan-1-2297-47.w92-139.abo.wanadoo.fr) |
| 16:41.33 | seanbright | elon musk tweeted 'tesla stock is too high imo' an hour ago and it's now down 10% |
| 16:41.53 | seanbright | he's lost his mind |
| 16:43.39 | file | vroom vroom |
| 16:43.52 | seanbright | i feel like he's one of those rare people you could say 'elon said...' or 'musk said...' and either way you would know who was being talked about |
| 17:12.29 | *** join/#asterisk-dev tyler (~tmoore@50-253-243-17-static.hfc.comcastbusiness.net) |
| 22:10.24 | *** join/#asterisk-dev jkroon (~jkroon@165.16.203.102) |
| 22:10.31 | *** join/#asterisk-dev coreyfarrell (~coreyfarr@rrcs-66-57-14-122.midsouth.biz.rr.com) |
| 22:10.41 | *** mode/#asterisk-dev [+o coreyfarrell] by ChanServ |