New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[WebIDL] Cache extended attributes of WebIDL file #1769
[WebIDL] Cache extended attributes of WebIDL file #1769
Conversation
Is there a mechanism to invalidate the cache when the IDL changes? |
147c7e0
to
4b2c602
Compare
I implemented the same mechanism used by |
4b2c602
to
229e31a
Compare
229e31a
to
1f8c9e3
Compare
No. But itβs an in-memory cache during the runtime of the script, not an on-disk cache. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is fine.
https://bugs.webkit.org/show_bug.cgi?id=241960 Reviewed by Darin Adler. * Source/WebCore/bindings/scripts/CodeGenerator.pm: (GetInterfaceExtendedAttributesFromName): Store results into a cache indexeded by IDL filename, so we don't need to parse the same file again. (ComputeIsCallbackInterface): Use slurp function to read file contents. (ComputeIsCallbackFunction): Use slurp function to read file contents. Canonical link: https://commits.webkit.org/255617@main
1f8c9e3
to
c7da26f
Compare
Committed 255617@main (c7da26f): https://commits.webkit.org/255617@main Reviewed commits have been landed. Closing PR #1769 and removing active labels. |
c7da26f
https://github.com/False/commit/1f8c9e398af4817d65e6acf3a146a8015a4ee85c