This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 121347 - APT is broken on Unicode symbols inside string tokens for modified document
Summary: APT is broken on Unicode symbols inside string tokens for modified document
Status: VERIFIED FIXED
Alias: None
Product: cnd
Classification: Unclassified
Component: Code Model (show other bugs)
Version: 6.x
Hardware: All All
: P1 blocker (vote)
Assignee: Vladimir Voskresensky
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-11-07 09:32 UTC by Vladimir Voskresensky
Modified: 2007-11-12 15:52 UTC (History)
1 user (show)

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments
proposed patch for 6.0 (2.86 KB, patch)
2007-11-08 23:54 UTC, Vladimir Voskresensky
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Vladimir Voskresensky 2007-11-07 09:32:46 UTC
int main(int argc, char**argv) {
   // № 1   in russian is the problem symbol inside string
   cout <<"Точка № 1  :\n";
}
Comment 1 Vladimir Voskresensky 2007-11-07 09:35:36 UTC
test file is:
Checking in 0x16.cc;
/shared/data/ccvs/repository/cnd/modelimpl/test/unit/data/org/netbeans/modules/cnd/modelimpl/trace/FileModelTest/0x16.cc,v
 <--  0x16.cc
initial revision: 1.1
done
Comment 2 Vladimir Voskresensky 2007-11-08 23:24:42 UTC
It seems to be problem for a lot of Unicode symboles when file is not saved (so FileBuffer is document based)
And in this case we have broken APT for whole file
Rising to P2
Comment 3 Vladimir Voskresensky 2007-11-08 23:54:52 UTC
Created attachment 52767 [details]
proposed patch for 6.0
Comment 4 Vladimir Voskresensky 2007-11-08 23:59:23 UTC
fixed in trunk:
- use standard ByteArrayInputStream to convert String into InputStream

cvs log:
Checking in FileBufferDoc.java;
/cvs/cnd/modelimpl/src/org/netbeans/modules/cnd/modelimpl/platform/FileBufferDoc.java,v  <--  FileBufferDoc.java
new revision: 1.9; previous revision: 1.8
done
Comment 5 Jesse Grodnik 2007-11-09 00:17:43 UTC
Escalated. Critical for ML release.
Comment 6 Vladimir Kvashin 2007-11-09 11:07:14 UTC
The fix was reviewed.
I believe it is correct and safe.
Comment 7 Alexander Pepin 2007-11-09 14:58:04 UTC
verified in trunk build 20071109
Comment 8 Alexander Pepin 2007-11-12 11:16:00 UTC
verified in RC1 build