feat: More structured parsing of Il2cpp SourceInfo

[Swatinem]

No description provided.

[loewenheim]

Right now this only exposes the create_line_mapping_from_object function (and its return type, the type alias RawLineMapping). Arguably it would be cleaner to make the only function one that combines create_line_mapping_from_object with serialization to JSON so that RawLineMapping is entirely internal.

[codecov-commenter]

Codecov Report

Merging #517 (878650c) into master (8c9eabc) will decrease coverage by 0.19%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #517      +/-   ##
==========================================
- Coverage   66.96%   66.77%   -0.20%     
==========================================
  Files         101      102       +1     
  Lines       18885    18941      +56     
==========================================
+ Hits        12646    12647       +1     
- Misses       6239     6294      +55     

[Swatinem]

That would also be perfectly fine. You could parse that into an opaque struct, which has as its only method a to_write or serialize which writes stuff to a T: Write.

[Swatinem]

I can’t approve my own PR :-D
I think rust complains because you never pub use this in the main lib.rs.

[loewenheim]

That sounds like a good idea, yeah.

[loewenheim]

I can’t approve my own PR :-D I think rust complains because you never pub use this in the main lib.rs.

Ah, I was wondering about that.

[kamilogorek]

Can we add an example line here in the comment, so its easier to understand the parsing in the future?

[loewenheim]

I added an example.