To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=114706
User dtardon changed the following: What |Old value |New value ================================================================================ CC|'' |'dtardon' -------------------------------------------------------------------------------- ------- Additional comments from dtar...@openoffice.org Fri Sep 24 06:49:16 +0000 2010 ------- dtardon->hdu: If I understand it correctly, the function is supposed to add a font and all it's aliases. In that case the comparison should be the other way around, i.e. aMapNames.Len() <= nMapNameIndex, otherwise no alias would be ever added. The condition in the for loop is wrong too, because GetNextFontToken sets nMapNameIndex to STRING_NOTFOUND on the last token already, so the last alias would be always missed. Attached is a patch that does what I think is the right thing. dtardon->cmc: Thanks for digging into the autocorrect stuff. You were quicker than me, as usual :) --------------------------------------------------------------------- Please do not reply to this automatically generated notification from Issue Tracker. Please log onto the website and enter your comments. http://qa.openoffice.org/issue_handling/project_issues.html#notification --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@gsl.openoffice.org For additional commands, e-mail: issues-h...@gsl.openoffice.org --------------------------------------------------------------------- To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org For additional commands, e-mail: allbugs-h...@openoffice.org