Bug 125462 - Remove docs/make-bytecode-docs.pl
Summary: Remove docs/make-bytecode-docs.pl
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
Depends on:
Reported: 2013-12-09 11:42 PST by Joseph Pecoraro
Modified: 2013-12-09 12:17 PST (History)
3 users (show)

See Also:

[PATCH] Proposed Fix (2.42 KB, patch)
2013-12-09 11:44 PST, Joseph Pecoraro
joepeck: review-
Details | Formatted Diff | Diff
[PATCH] Proposed Fix (2.58 KB, patch)
2013-12-09 11:45 PST, Joseph Pecoraro
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Joseph Pecoraro 2013-12-09 11:42:13 PST
This script is generating a basically empty HTML file, hardcoded in the script:

    <!-- Generated from Interpreter.cpp by make-bytecode-docs.pl. -->
    <style>p code { font-size: 14px; }</style>

The script is very old. It looks like the op codes are no longer defined in Interpreter.cpp, and the "documentation" is not being used by anyone if the file has been empty for so long.
Comment 1 Joseph Pecoraro 2013-12-09 11:44:05 PST
Created attachment 218783 [details]
[PATCH] Proposed Fix
Comment 2 Joseph Pecoraro 2013-12-09 11:44:43 PST
Comment on attachment 218783 [details]
[PATCH] Proposed Fix

Missed a reference to byte code.html
Comment 3 Joseph Pecoraro 2013-12-09 11:45:09 PST
Created attachment 218784 [details]
[PATCH] Proposed Fix
Comment 4 WebKit Commit Bot 2013-12-09 12:17:11 PST
Comment on attachment 218784 [details]
[PATCH] Proposed Fix

Clearing flags on attachment: 218784

Committed r160325: <http://trac.webkit.org/changeset/160325>
Comment 5 WebKit Commit Bot 2013-12-09 12:17:13 PST
All reviewed patches have been landed.  Closing bug.