Bug 1575062 Comment 4 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

(In reply to Emilio Cobos Álvarez (:emilio) from comment #3)
> We may need some little extra code to add tests for it and such, but I think something like that keeps the performance impact to the minimum, wdyt?

Yap. Putting all parsing code together in the same pattern match makes sense to me to minimize the impact.

> Boris, does the comment above help? Does it make sense to you? If not I'm happy to talk or video-meet or what not :)

It's awesome and really helpful. Now I have a better concept for the implementation detail. Will let you know if I meet new problems. :)
(In reply to Emilio Cobos Álvarez (:emilio) from comment #3)
> We may need some little extra code to add tests for it and such, but I think something like that keeps the performance impact to the minimum, wdyt?

Yap. Putting all parsing code together in the same pattern match makes sense to me to minimize the impact.

> Boris, does the comment above help? Does it make sense to you? If not I'm happy to talk or video-meet or what not :)

It's awesome and really helpful. Now I have a better concept for the implementation detail. Will let you know if I meet new problems. Thanks a lot! :)

Back to Bug 1575062 Comment 4