Re: [Dynapi-Dev] Updated CVS - API Docs - Release?

[Michael P. <mp...@ph...>]

I've got a suggestion for the code before it gets released.  It is not bug
related, but it does bug me that it is there.  I find no reason for the
?true:false component in many of the evaluations in the API (eg.
events.js).  There is no need to evaluate the condition and then select
true or false to match the condition.  Just use the condition directly.

Again, I know that it is not a bug, but if there is no reason for it being
there, it should be removed.  Every bit of erelevant code it just another
spot for a bug to pop up.

Robert Rainwater wrote:

> I've updated CVS with some API documentation. In there you will also
> find the config files used to generate the API documentation. It uses
> Doxygen to create the files which is freely available under GPL. Look
> in the config directory under docs for more info.
>
> Also, I fixed a bug in DynDocument that was over writing the dyndocs
> array each time you added a document.  I'm still not sure how frames
> ever worked with that bug.
>
> Also, I feel like we are ready for a release.  If so I propose that
> todays build (dynapi_2001_01_20.zip) be considered a release candidate.
> If anyone has any changes to make to the build, please post them here.
>
> Here's the new snapshot page: http://dynapi.sourceforge.net/snapshot/
>
> --
> // Robert Rainwater
> ----------------------
> DynAPI Snapshots: http://dynapi.sourceforge.net/snapshot/
> DynAPI Homepage: http://dynapi.sourceforge.net/
>
> _______________________________________________
> Dynapi-Dev mailing list
> Dyn...@li...
> http://lists.sourceforge.net/lists/listinfo/dynapi-dev

--
Michael Pemberton
mp...@ph...
ICQ: 12107010