CalendarBuilder should not asume nothing about underline CalendarParser implementation, and should not automatically wrap all streams into UnfoldingReader. Also this puts an end to other usefull ReaderFilter implementations that parser could use.
This patch is just a proposal for change since it breaks compatibility by removing parse(UnfoldingReader method) , witch is rarely called directly by clients I think, but real fix should leave all methods or maybe add method like
parse(Reader in, boolean unfold);
Log in to post a comment.