Community
Participate
Working Groups
Build Identifier: I20110613-1736 When you have workspace encoding different than utf-8 and checkout org.eclipse.birt.report.engine.tests project from CVS you have the following error message: Invalid character constant in FontConfigReaderTest.java at line 63 It is a defect - if a project depends on specific encoding, it should be set in project preferences. Another but safer fix is to replace character from line 63 with \u624b symbol which is hex value of that character. The second option is simpler and it is lower chance to introduce regression. See also FontConfigReaderTest.java at line 49 where the same character is used. Reproducible: Always Steps to Reproduce: 1. In Windows->Preferences->General->Workspace set Text file encoding to iso-8859-1 2. Checkout FontConfigReaderTest from CVS repository You will have Error in FontConfigReaderTest.java at line 63
Created attachment 208920 [details] Patch
The patch applies for both CVS and git repository
The test has already used \u634b http://git.eclipse.org/c/birt/org.eclipse.birt.git/tree/engine/org.eclipse.birt.report.engine.tests/test/org/eclipse/birt/report/engine/layout/pdf/font/FontConfigReaderTest.java
Xiaoying, this has been fixed in line 49 but the line 63 still contains the issue. Could you review it again. please?
reopening
(In reply to comment #4) > Xiaoying, this has been fixed in line 49 but the line 63 still contains the > issue. Could you review it again. please? Sorry for the confusion. Yes the file in master branch is still not correct. We will update in the next milestone.
Set as fixed.
