rust-lang
Add column info to the Record?
We have this info when constructing, it's one of field provided by core::panic::Location.
Or even we can provide the Location as a field of the Record to make the fields more compact.
What do you think?
How valuable would this be? Because usually the line number is enough to locate where the message came from. Adding the column will increase binary size with questionable (to me) benefit.
How valuable would this be? Because usually the line number is enough to locate where the message came from. Adding the column will increase binary size with questionable (to me) benefit.
This is a good point. I agree that the line number is generally enough to locate the issue, unless deep nested call without line break, which, with the help of rustfmt, is not quite possible.
However, I'd like to learn more about the binary size issue. Could you elaborate a bit how the binary size would be affected? Is it per log! call? If so, how can I verify that? So far, I can see we always use Location::caller to get the location info and pass some of its fields.
Could you elaborate a bit how the binary size would be affected? Is it per log! call? If so, how can I verify that? So far, I can see we always use Location::caller to get the location info and pass some of its fields.
@Thomasdezeeuw I'll appreciate it if you can share the debugging/analyzing method here :D