--- Day changed Mon Oct 08 2018 10:24 < stevenroose> dongcarl: yeah that is next level 10:25 < stevenroose> The optional args are changed to an Option of the specified type. Later, mostly to save space 10:28 < stevenroose> andytoshi: I'm not using any `tt` captures anymore now. `expr` for method names is weird, as it will catch a ton of stuff like operators, brackets... I thought ident just did names (like anything that is characters numbers and underscores) 10:29 < stevenroose> To keep it more structured, I was thinking of having the macro wrap the method call in an `impl` block and then move the result types into the same file so that result types and the method that uses it are together 10:30 < stevenroose> The unit test could also be there in fact. So that you get , ... 10:30 < stevenroose> Not sure if that would obstruct readability too much. Would certainly make it easier to not have to go back and forth 10:30 < stevenroose> I'm digesting Scaling videos today :)