#general
- e
enough-analyst-54434
10/07/2015, 11:21 PMI think we agree today is broken? - w
witty-crayon-22786
10/07/2015, 11:22 PMwasn't sure, heh =P - w
witty-crayon-22786
10/07/2015, 11:22 PMcc @bored-art-40741 : ^ - e
enough-analyst-54434
10/07/2015, 11:23 PMI love types too - I think you have maybe a totally wrong impression. Have I pointed out my preconditions decorator RB before? - b
bored-art-40741
10/07/2015, 11:23 PMSorry missing the context here - w
witty-crayon-22786
10/07/2015, 11:23 PM@bored-art-40741: it's all there... scroll up to where i mentioned your name - w
witty-crayon-22786
10/07/2015, 11:24 PM@bored-art-40741: https://pantsbuild.slack.com/archives/general/p1444259121000634 - e
enough-analyst-54434
10/07/2015, 11:25 PMI assume their concern was valid though, ie: what you want takes more work, and what cody did would have needed to expand in scope - e
enough-analyst-54434
10/07/2015, 11:26 PMtradeoffs, lack of a continuous effort to get to the final solution - b
bored-art-40741
10/07/2015, 11:26 PMAh, not doing resolution inside of PayloadField fingerprinting - b
bored-art-40741
10/07/2015, 11:26 PMYeah, I stand by that decision - b
bored-art-40741
10/07/2015, 11:26 PMMaybe there is a general framework where that makes sense - e
enough-analyst-54434
10/07/2015, 11:26 PMI think I agree, you want a prior stage to resolve - b
bored-art-40741
10/07/2015, 11:26 PMBut I haven't heard it proposed, and that particular change was just going to be a trojan horse for badness - e
enough-analyst-54434
10/07/2015, 11:26 PMbut the effort to get there is large - e
enough-analyst-54434
10/07/2015, 11:27 PMit'll fall out of fine-grained execution / new engine, but certainly not doable in Cody's RB - b
bored-art-40741
10/07/2015, 11:27 PM=you - e
enough-analyst-54434
10/07/2015, 11:27 PMIE: I blew it in my RB, but it should not be overblown - e
enough-analyst-54434
10/07/2015, 11:27 PMWe all agree on the right path - w
witty-crayon-22786
10/07/2015, 11:27 PMok. then i misunderstood the reason. but not generalizing there has cost us this bug (edit: er, not really. that's a bit dramatic.) - e
enough-analyst-54434
10/07/2015, 11:27 PMwe just need to do it, and stick to it - b
bored-art-40741
10/07/2015, 11:28 PMWhich bug? - w
witty-crayon-22786
10/07/2015, 11:30 PMwas referring to https://rbcommons.com/s/twitter/r/2937/ ... but it doesn't really have anything to do with this. just got distracted - e
enough-analyst-54434
10/07/2015, 11:30 PMI trashed a magic option type - e
enough-analyst-54434
10/07/2015, 11:30 PMit was list of strings - but really list of Addresses - b
bored-art-40741
10/07/2015, 11:31 PMAh, yeah - e
enough-analyst-54434
10/07/2015, 11:31 PMI do think that was the real problem, pretending the Addresses were strings - e
enough-analyst-54434
10/07/2015, 11:32 PMwell, 3rd in line after my change and reviewes missing it - w
witty-crayon-22786
10/07/2015, 11:32 PMi think i agree. - b
bored-art-40741
10/07/2015, 11:32 PMThis is something I labored to clean up in the original target refactor, but when it's so easy to translate a spec to an address to a target things tend to slip