odenix
odenix
> It would require a further update to NOTICE.txt, btw. done
There’s likely room for optimizations. Although I was tempted in a few places, I decided not to touch EconomicMap code in this PR to simplify review and updates. I believe...
Simple optimizations are avoiding wrapper calls (already the case) and reducing the number of boundary calls. I don’t understand why you’re pushing back against a vendoring that is both more...
At a minimum, it makes the code much cleaner (i.e., easier to read, write, and maintain) and less error-prone (currently, wrapper methods are used in some places but not in...
@HT154 What’s the use case? Unlike a regular property, a local property cannot be amended in place. I can’t think of many valid reasons to set a local property to...
> normal properties will default to the type's default value. True, except that many types don’t have default values and always need to be assigned explicitly. > Also, const and...
> I'm not sure what errors we might be preventing here When I see `local a: A`, I immediately wonder if the author forgot to set a value (and they...