You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@mina.apache.org by "Emmanuel Lecharny (JIRA)" <ji...@apache.org> on 2010/01/13 01:48:54 UTC

[jira] Resolved: (DIRMINA-754) Javadoc omission in IoFilterChain#fireMessageSent()

     [ https://issues.apache.org/jira/browse/DIRMINA-754?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Emmanuel Lecharny resolved DIRMINA-754.
---------------------------------------

    Resolution: Fixed

Fixed in http://svn.apache.org/viewvc?rev=898585&view=rev

I completely reviewed the doco for the whole class. What a mess it was !

> Javadoc omission in IoFilterChain#fireMessageSent()
> ---------------------------------------------------
>
>                 Key: DIRMINA-754
>                 URL: https://issues.apache.org/jira/browse/DIRMINA-754
>             Project: MINA
>          Issue Type: Improvement
>          Components: Core
>    Affects Versions: 2.0.0-RC1
>         Environment: N/A
>            Reporter: Bogdan Pistol
>            Assignee: Emmanuel Lecharny
>            Priority: Trivial
>             Fix For: 2.0.0-RC2
>
>
> The javadoc of the org.apache.mina.core.filterchain.IoFilterChain#fireMessageSent() is
> /**
>  * Fires a {@link IoHandler#sessionOpened(IoSession)} event.  Most users don't need to call
>  * this method at all.  Please use this method only when you implement a new transport or
>  * fire a virtual event.
>  */
> public void fireMessageSent(WriteRequest request);
> And instead a better javadoc would be:
> /**
>  * Fires a {@link IoHandler#messageSent(IoSession, Object)} event.  Most users don't need to call
>  * this method at all.  Please use this method only when you implement a new transport or
>  * fire a virtual event.
>  */
> public void fireMessageSent(WriteRequest request);

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.