html-agility-pack icon indicating copy to clipboard operation
html-agility-pack copied to clipboard
verified publisher icon zzzprojects

async/await API missing

Open Legends opened this issue 7 years ago • 5 comments

Any plans to implement the async api? I mean, why force people to use Task.run?

Legends avatar Jun 22 '18 17:06 Legends

Hello @Legends ,

We can always implement async but inside the method, they will only do Task.Run so it will not be real async. Unfortunately, supporting async all the way require a lot of development.

Best Regards,

Jonathan

JonathanMagnan avatar Jun 23 '18 16:06 JonathanMagnan

@JonathanMagnan Will you in principle support that change to API ? There modern tools for writing more performance oriented parsing, using for example Span<T> and System.Memory infrastructure in .NET Core. Even this would be project on it's own, that greatly help those who use HtmlAgilityPack for parsing HTML in large quantities.

kant2002 avatar May 07 '19 12:05 kant2002

Hello @kant2002,

There is no plan to change this at the moment.

Unfortunately, time is missing on our side as well, so we cannot afford this kind of development at this moment.

JonathanMagnan avatar May 07 '19 13:05 JonathanMagnan

@JonathanMagnan I'm more asking if you are up to accepting contribution which could help you move in that direction?

kant2002 avatar May 07 '19 13:05 kant2002

Oh sure ;)

JonathanMagnan avatar May 07 '19 14:05 JonathanMagnan