combine icon indicating copy to clipboard operation
combine copied to clipboard

Offset parser

Open r4v3n6101 opened this issue 2 years ago • 1 comments

Hello! Previously, I asked to make an offset function parsing data located by some offset. Finally, I've found time to do this one. But we should discuss the feature because look_ahead(skip_count(count).with(parser)) does same thing as offset. Does the combine really need the new parser? If so then it closes #337.

r4v3n6101 avatar Mar 08 '22 06:03 r4v3n6101

First, my thanks to @Marwes for approving first PR of mine. Second topic I'd like to discuss is naming of this parser. offset seems to me as very abstract and general word: it may be shadowed with everything - other function named offset or some random variable. What about by_offset or parse_by_offset?

r4v3n6101 avatar Mar 14 '22 12:03 r4v3n6101