Bugzilla – Full Text Bug Listing
| Summary: | [EditorMgmt] Editors should be able to treat large files specially (e.g. deny opening huge files) | ||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | [Eclipse Project] Platform | Reporter: | Prakash Rangaraj <prakash> | ||||||||
| Component: | UI | Assignee: | Platform UI Triaged <platform-ui-triaged> | ||||||||
| Status: | RESOLVED FIXED | QA Contact: | |||||||||
| Severity: | enhancement | ||||||||||
| Priority: | P3 | CC: | bokowski, daniel_megert, eclipse, elias, emoffatt, jamesblackburn+eclipse, markus.kell.r, mober.at+eclipse, ob1.eclipse, prakash, pwebster, remy.suen, sptaszkiewicz | ||||||||
| Version: | 3.7 | Flags: | prakash:
review?
bokowski: review+ remy.suen: review+ |
||||||||
| Target Milestone: | 3.7 RC2 | ||||||||||
| Hardware: | PC | ||||||||||
| OS: | Windows XP | ||||||||||
| Whiteboard: | |||||||||||
| Bug Depends on: | 117746, 344031, 344652, 344653 | ||||||||||
| Bug Blocks: | |||||||||||
| Attachments: |
|
||||||||||
|
Description
Prakash Rangaraj
Created attachment 194679 [details]
Patch v01
Patch v01 for HEAD
At this point changes in 3.7 should be minimal, I suggest we go with the same patch as was done in 3.4.2+ (bug 344031 ). Note also reversal of the changes from Label to Text (bug 344031 comment 18 ) which seems to be missing from the proposed patch. Created attachment 196027 [details]
The "forward-ported" patch from 3.4.2
(In reply to comment #3) > Created attachment 196027 [details] > The "forward-ported" patch from 3.4.2 The patch looks okay to me and does the job with the least amount of code required. One thing I noticed was that it doesn't "work" if you change the value at runtime but I suppose we're just expecting people to use plugin_customization.ini to alter the values on startup and not for them to be randomly changing the value while the application is up. If one were to change this value, they would need to spawn new workbench windows (so that a new EditorManager would get instantiated) for the new value to kick in. I think it would be worth adding a @noreference tag to the "upgraded" STORE_ID_INTERNAL_EXTERNAL static field. While the text notes that it probably won't be referenced (as clients are not supposed to extend the class), it doesn't hurt to add the tag in my opinion though I admit to being unfamiliar with our API tagging policies. Lastly, I'm wondering if these changes will come up as API errors during the build? I know that SWT has to update the API exclusion list for bug 279461 but that is a case of making a public class @noextend while we are lifting the 'final' requirement and replacing it with a @noextend. Are similar rules going to kick in for our API tooling tags here? Created attachment 196035 [details] The "forward-ported" patch from 3.4.2 updated (In reply to comment #4) > One thing I noticed was that it doesn't "work" if you change the value at > runtime but I suppose we're just expecting people to use > plugin_customization.ini to alter the values on startup and not for them to be > randomly changing the value while the application is up. That's correct. At present we do not expect the file size limit to change at runtime. > I think it would be worth adding a @noreference tag to the "upgraded" > STORE_ID_INTERNAL_EXTERNAL static field. Good point, I updated the patch. > Lastly, I'm wondering if these changes will come up as API errors during the > build? I checked with Olivier and he thinks that it should be fine for API tools. Of course there is always a chance that it won't :-). Comment on attachment 196035 [details]
The "forward-ported" patch from 3.4.2 updated
Marking the attachment as a patch.
+1 for 3.7.RC2 Updated patch applied to CVS Head. Thank you! Verified in I20110519-0800. Oleg, it looks like this didn't make it into 4.x. There is no EditorManager class in 4.2... *** This bug has been marked as a duplicate of bug 117746 *** Let's keep this as bug for the fix being in 3.7.x and 3.8.x. |