yes, will change it tonight. Thanks for the review ! Frédéric THOMAS
> From: [email protected] > To: [email protected] > Subject: RE: git commit: [flex-tlf] [refs/heads/develop] - FLEX-34592: Can't > use optionnal importer for TLF plainTextFormat and TextLayoutFormat > Date: Mon, 13 Oct 2014 17:18:02 -0600 > > +1 > > > -----Original Message----- > From: Justin Mclean [mailto:[email protected]] > Sent: October-13-14 4:33 PM > To: [email protected] > Subject: Re: git commit: [flex-tlf] [refs/heads/develop] - FLEX-34592: Can't > use optionnal importer for TLF plainTextFormat and TextLayoutFormat > > Hi, > > Wouldn't a better patch have been this? > > if (textFlow) { > textScrap = new TextScrap(textFlow); > > if (format == TextConverter.PLAIN_TEXT_FORMAT) > textScrap.setPlainText(true); > else if (format == TextConverter.TEXT_LAYOUT_FORMAT) > textScrap.setPlainText(false); > } > > Thanks, > Justin >
