[Gcjwebplugin-devel] Patch: Applet options functionality update (and more)

[Lillian Angel]

gcjappletviewer should not be able to take a --code arg as well as a
url. It should work like this:
       gcjappletviewer --code=filename.class | url(s)

So people need to either specify --code=filename.class or specify a list
of URLs.  If both are specified the --code option wins.

StandaloneAppletViewer now has 2 constructors: ne that takes a list of
urls, and one that takes the --code, --documentbase, etc. options.

Multiple urls with multiple applets each are supported, but only one 
applet can be loaded using --code (so --code=applet1.class 
--code=applet2.class isn't supported).

The --code option only takes a class filename, not a class name as well.
The man page has been updated to reflect this change. Therefore, the
--width --height overriding "feature" is gone. The tests have been
updated as well.

AppletTag has been updated again. It is much cleaner (and readable) now.
It is probably bug-free :)


This patch was also written by Tom Fitzsimmons.


2006-04-19  Lillian Angel  <address@hidden>
            Thomas Fitzsimmons  <address@hidden>

        * Makefile.am: Removed deleted file.
        * testsuite.at: Likewise.
        * src/gcjappletviewer.1: Updated man page and options
        to reflect changes in functionality. --code only
        works with a filename now.
        * src/gnu/gcjwebplugin/AppletTag.java
        (AppletTag): Fixed documentation.
        (AppletTag): Removed unused constructor.
        (AppletTag): Fixed documentation.
        (AppletTag): Made private, only used within class.
        (createApplet): Renamed to parseAppletTags.
        (EmbedTag): Removed Class.
        (AppletTag): New constructor. Parses a given
        DomHTMLEmbedElement.
        (ObjectTag): Removed class.
        (AppletTag): New constructor. Parses a given
        DomHTMLObjectElement.
        (recurseDocument): Made private.
        (parseParams): Made private. Removed String
        handling.
        (parseArchives): Made private.
        * src/gnu/gcjwebplugin/AppletViewer.java
        (main): Fixed option handling. --code only takes a
        filename. --width and --height no longer override.
        Also, added code to use new StandaloneAppletViewer
        constructor if a list of urls was passed in.
        (printArguments): New function.
        * src/gnu/gcjwebplugin/PluginAppletViewer.java
        (write): Changed to use System.err instead.
        (read): Likewise.
        * src/gnu/gcjwebplugin/PluginAppletWindow.java
        (setHandle): Fixed to use new function.
        * src/gnu/gcjwebplugin/StandaloneAppletViewer.java
        (StandaloneAppletViewer): New constructor. Handles
        a list of urls.
        (StandaloneAppletViewer): Reworked to use new functions
        in AppletTag, as well as create PARAM elements in the
        EMBED tag as needed.
        (printTags): New function.
        (createWindows): New function.
        * src/gnu/gcjwebplugin/locale/MessagesBundle.properties:
        Updated options.
        * tests/TestApplet.java
        (init): Fixed to use System.out.
        (start): Likewise.
        (stop): Likewise.
        * tests/TestApplet2.java        
        (init): Fixed to use System.out.
        (start): Likewise.
        (stop): Likewise.
        * tests/t_appletcontext.at: Updated test.
        * tests/t_appletcontextcodebase.at: Updated test.
        * tests/t_appletcontextnocodebase.at: Updated test.
        * tests/t_appletstub.at: Updated test.
        * tests/t_classfile.at: Updated test.
        * tests/t_cmdlinearchive.at: Updated test.
        * tests/t_cmdlinecode.at: Updated test.
        * tests/t_cmdlinecodebase.at: Updated test.
        * tests/t_cmdlineparam.at: Updated test.
        * tests/t_cmdlinewidthheight.at: Updated test.
        * tests/t_dimensionsnooverrideclass.at: Updated test.
        * tests/t_dimensionsnooverridecode.at: Updated test.
        * tests/t_dimensionsoverride.at: Removed.
        * tests/t_parameterswidthheight.at: Updated test.
        * tests/t_parseapplet.at: Updated test.
        * tests/t_parsearchives.at: Updated test.
        * tests/t_parseembed.at: Updated test.
        * tests/t_parseembedjavaattrs.at: Updated test.
        * tests/t_parseobject.at: Updated test.
        * tests/t_parseobjectjavaattrs.at: Updated test.
        * tests/t_parseparams.at: Updated test.
        * tests/t_parsepercentagedimension.at: Updated test.
        * tests/t_parsetwoapplets.at: Updated test.

gcjapp_patch.diff.gz
Description: GNU Zip compressed data