This is one of steps to stop rebuilding .h/.cpp files generated by unchanged IDLs (bug 76836). As a refactoring, we are planning to remove finish() from all CodeGenerators. In this bug, we make finish() of CodeGeneratorJS.pm empty.
Created attachment 123585 [details] Patch
Comment on attachment 123585 [details] Patch Clearing flags on attachment: 123585 Committed r105683: <http://trac.webkit.org/changeset/105683>
All reviewed patches have been landed. Closing bug.
Comment on attachment 123585 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=123585&action=review > Source/WebCore/bindings/scripts/CodeGeneratorJS.pm:3251 > + my @includes = (); Here is the first declaration. > Source/WebCore/bindings/scripts/CodeGeneratorJS.pm:3290 > + my @includes = (); This change caused the following warning: "my" variable @includes masks earlier declaration in same scope at /home/oszi/WebKit/Source/WebCore/bindings/scripts/CodeGeneratorJS.pm line 3290.
Reopening to attach new patch.
Created attachment 123703 [details] proposed warning fix
Comment on attachment 123703 [details] proposed warning fix Landed in http://trac.webkit.org/changeset/105734
When I do incremental builds today, I see all IDL files being processed every time. This didn't used to happen.
(In reply to comment #8) > When I do incremental builds today, I see all IDL files being processed every time. This didn't used to happen. smfr: I was able to reproduce it. I'll fix it.
(In reply to comment #8) > When I do incremental builds today, I see all IDL files being processed every time. This didn't used to happen. smfr: Fixed it in bug 76970.