fix: reduce memory footprint when loading TLDs

[nandgator]

No description provided.

[salty-horse]

This significantly slows down TLD lookup. Opening a file and scanning it for every email validation is very inefficient. Is the memory footprint that much of a concern?

Would you be open to changing it back to something like this, which is 10 times faster?

_iana_tld_set=Nonedef_iana_tld():
global_iana_tld_setif_iana_tld_set:
return_iana_tld_setwithPath(__file__).parent.joinpath("_tld.txt").open() astld_f:
_=next(tld_f)
_iana_tld_set= {line.strip() forlineintld_f}
return_iana_tld_set

[nandgator]

Opening a file and scanning it for every email validation is very inefficient.

That's true for repeated validations.

Is the memory footprint that much of a concern?

Yes, if the file is too, large and/or, system memory is insufficient.

---

What about a load_iana_tld() method?

It will load and store the TLDs once. If that method isn't called, it'll lookup the file every time. Associate that method with a dataclass, instead of using global variables.

A PR is welcome.