[
https://issues.apache.org/jira/browse/WAVE-271?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13037500#comment-13037500
]
Michael MacFadden commented on WAVE-271:
----------------------------------------
I agree with Dave's suggestions. I will make the changes.
> Review / rename ConversationBlip.appendInlineReplyThread(int location)
> ----------------------------------------------------------------------
>
> Key: WAVE-271
> URL: https://issues.apache.org/jira/browse/WAVE-271
> Project: Wave
> Issue Type: Task
> Reporter: Michael MacFadden
> Assignee: Michael MacFadden
> Priority: Minor
>
> The ConversationBlip defines these two methods:
> ConversationThread appendReplyThread();
> ConversationThread appendInlineReplyThread(int location);
> Since we have largely removed the distinction between inline threads and
> non-inline threads. These methods need to be reviewed. At first I had
> though of just renaming the second method to "ConversationThread
> appendInlineReplyThread(int location)", thereby just overloading the method.
> That still might be and option, but I thought we should actually review what
> these two methods do to see if we need to do anything other than renaming
> them. At the very least some better JavaDoc is probably a good idea.
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
