[#DIRMINA-111] MessageHandler.messageReceived(...) doesn't throw Exception

MINA

MessageHandler.messageReceived(...) doesn't throw Exception

Created: 03/Nov/05 10:08 AM Updated: 03/Nov/05 10:09 AM

Return to search

Component/s:

None

Affects Version/s:

0.8.0

Fix Version/s:

0.8.1

Time Tracking:

Not Specified

Resolution Date:

03/Nov/05 10:09 AM

Description

« Hide

Jose Alberto Fernandez wrote:

Is there any reason why org.apache.mina.protocol.handler.MessageHandler.messageReceived() does not allow throwing an Exception? I would have thought the signature should be similar to org.apache.mina.protocol.ProtocolHandler.messageReceived().

This means you need to treat error conditions differently depending if you use one implementation strategy or another.

Any thoughts about it?

Re) This is a flaw in API. I'll fix the method signature in 0.8.1.

All

Comments

Work Log

Change History

Subversion Commits

Sort Order:

Repository	Revision	Date	User	Message
ASF	#330401	Thu Nov 03 01:09:13 UTC 2005	trustin	Resolved issue: ~~DIRMINA-111~~ MessageHandler.messageReceived(...) doesn't throw Exception * Changed the method signature to throw any exceptions
				Files Changed
				MODIFY /directory/network/trunk/src/java/org/apache/mina/handler/DemuxingIoHandler.java MODIFY /directory/network/branches/0.8/src/java/org/apache/mina/protocol/handler/DemuxingProtocolHandler.java MODIFY /directory/network/trunk/src/java/org/apache/mina/handler/MessageHandler.java MODIFY /directory/network/branches/0.8/src/java/org/apache/mina/protocol/handler/MessageHandler.java

[ Permlink | « Hide ]

Trustin Lee added a comment - 03/Nov/05 10:09 AM

Changed the method signature just like ProtocolHandler.messageReceived()

Trustin Lee made changes - 03/Nov/05 10:09 AM

Field	Original Value	New Value
Status	Open [ 1 ]	Closed [ 6 ]
Resolution		Fixed [ 1 ]