[00:51:32] --- squinney has become available [05:18:57] --- mvitale has become available [05:33:40] --- abo has left [05:34:56] --- abo has become available [06:00:48] --- abo has left [06:02:16] --- abo has become available [06:22:36] --- ballbery has become available [06:53:48] --- Ken Dreyer has left [06:54:21] --- Ken Dreyer has become available [06:55:10] --- meffie has become available [07:08:18] --- deason has become available [07:18:48] --- Ken Dreyer has left [07:19:45] --- Ken Dreyer has become available [08:46:29] --- mvitale has left [08:49:35] --- mvitale has become available [09:26:43] --- mvitale has left [09:27:06] --- mvitale has become available [09:28:28] --- mvitale has left [09:28:28] --- mvitale has become available [09:30:15] --- mvitale has left [09:31:12] --- mvitale has become available [10:05:03] --- mvitale has left [10:05:03] --- mvitale has become available [10:10:29] --- mvitale has left [10:10:49] --- stephan.wiesand has become available [10:11:56] --- stephan.wiesand has left [10:13:53] --- mvitale has become available [11:08:46] --- meffie has left [11:24:54] --- mvitale has left [11:29:32] --- mvitale has become available [11:35:22] --- meffie has become available [11:35:33] --- mvitale has left [11:47:04] --- mvitale has become available [11:47:52] --- meffie has left [12:16:49] Style question about continuation lines: code = a_really_long_function_call(calling_lots_of_things, too_long_for_one_line); or code = a_really_long_function_call(calling_lots_of_things, too_long_for_one_line); Does it change when three lines are needed in either form? [12:16:50] --- squinney has left: Lost connection [12:23:03] I definitely prefer the latter [12:23:50] We have a bunch of (old?) code doing the former, and I don't have a great way to check for the latter. [12:24:24] (The latter is house style here at work, so I want another opinion before defaulting to it.) [12:24:26] "check for the latter"? [12:24:45] > check for the latter See how much code volume there is using the second form [12:25:47] there should be plenty of examples of both in the tree; I didn't think either one "wins", so I wouldn't bother checking [12:25:53] unless everything else in that file or package uses one style [12:30:16] grep '[^(]+)' **/*.c ? (modify for your shell…) [12:30:28] hm, wait, add -E to that grep [12:32:59] almost [12:33:55] grep -E '^[^(]+\)' **/*.c [12:34:18] switching to -E means more escaping, plus forgot the anchor [12:38:58] --- mvitale has left [12:41:24] --- mvitale has become available [14:58:56] --- mvitale has left [14:58:56] --- mvitale has become available [15:13:22] --- ballbery has left [15:13:22] --- mvitale has left [15:14:01] --- mvitale has become available [15:42:36] --- deason has left [16:04:40] --- mdionne has become available [16:16:12] --- mvitale has left [16:16:36] --- mvitale has become available [16:21:00] --- mvitale has left [16:21:00] --- mvitale has become available [16:23:30] --- mvitale has left [16:23:55] --- mvitale has become available [16:31:02] --- mvitale has left [16:31:02] --- mvitale has become available [16:38:32] --- mvitale has left [16:42:30] --- mvitale has become available [16:51:32] --- mvitale has left [16:54:54] --- mvitale has become available [17:05:15] --- mvitale has left [17:05:22] --- mvitale has become available [17:11:54] --- mvitale has left [17:15:27] --- mvitale has become available [17:17:52] --- mvitale has left [17:19:30] --- jaltman/FrogsLeap has left: Disconnected [17:22:04] --- mvitale has become available [17:27:12] --- jaltman/FrogsLeap has become available [17:38:43] --- mvitale has left [19:18:09] --- mvitale has become available [19:49:04] --- mvitale has left [19:50:16] --- mdionne has left [23:28:09] --- abo has left [23:28:09] --- abo has become available