Bug 125825 - [typing] Camel case in case conversion
Summary: [typing] Camel case in case conversion
Status: ASSIGNED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: Text (show other bugs)
Version: 3.2   Edit
Hardware: PC Windows XP
: P4 enhancement with 2 votes (vote)
Target Milestone: ---   Edit
Assignee: Platform-Text-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords: helpwanted
Depends on:
Blocks:
 
Reported: 2006-01-31 08:58 EST by David Wright CLA
Modified: 2019-02-05 08:43 EST (History)
3 users (show)

See Also:


Attachments
fix (41.27 KB, patch)
2006-06-17 00:52 EDT, Sebastian Davids CLA
daniel_megert: review-
Details | Diff
tests (8.01 KB, patch)
2006-06-17 00:53 EDT, Sebastian Davids CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description David Wright CLA 2006-01-31 08:58:30 EST
Enhance or extend CaseAction to optionally convert camelCase to
CAMEL_CASE and vice-versa (ie to UC prepend _ to non-initial caps, to LC remove and keep following capitalised). This would be very useful for constants eg static final String MY_CONSTANT="myConstant"
Comment 1 Dani Megert CLA 2006-02-01 03:31:45 EST
Nice idea. This would be a good feature for someone to start contributing to Eclipse.

We currently don't plan to work on this.
Comment 2 Sebastian Davids CLA 2006-06-05 08:16:14 EDT
see 8518 for some discussion
Comment 3 Sebastian Davids CLA 2006-06-17 00:52:49 EDT
Created attachment 44721 [details]
fix

supports:

capitalize -> Capitalize
LOWER_CAMEL_CASE -> lowerCamelCase
UPPER_CAMEL_CASE -> UpperCamelCase
iNVERT cASE -> Invert Case
constantCase -> CONSTANT_CASE
Comment 4 Sebastian Davids CLA 2006-06-17 00:53:09 EDT
Created attachment 44722 [details]
tests
Comment 5 Sebastian Davids CLA 2006-06-17 01:00:55 EDT
I did not assign key shortcuts.

I did not create a new context menu ( Bug 8518 comment 23 ) because there is no declared sub-menu I could contribute it to.

I did not create a main menu entry because there is no declared sub-menu entry either -- the only existing menu these could be put on would be the edit menu.
Comment 6 Sebastian Davids CLA 2006-06-17 01:08:50 EDT
You might consider renaming class Strings and possibly push it down to jface.text or eclipse.text.

Or merge it w/ org.eclipse.jface.text.TextUtilities.

I've put it into org.eclipse.ui.internal.texteditor to make it non-API; the functionality might be useful to JFace's downstream plug-ins though.
Comment 7 Dani Megert CLA 2006-07-03 07:11:38 EDT
The problem with this patch is that it assumes Java (or more general: programming language) conventions to standard text editing and also introduces the term 'constant' which does not belong into a standard text editing plug-in.
Comment 8 Sebastian Davids CLA 2006-07-03 09:46:15 EDT
Rename as you like ...

I wasn't aware that there is a "general" text editor and "programming" text editor distinction within Eclipse frameworks.

Useful for "general" text editor:

capitalize -> Capitalize
iNVERT cASE -> Invert Case

Useful for "programming" text editor:

LOWER_CAMEL_CASE -> lowerCamelCase
UPPER_CAMEL_CASE -> UpperCamelCase
constantCase -> CONSTANT_CASE

So where does "text" stuff belong which applies for JDT, CDT, RDT, etc. and not "general" text editor stuff?

I am confused :|
Comment 9 Dani Megert CLA 2006-07-03 09:53:35 EDT
>So where does "text" stuff belong which applies for JDT, CDT, RDT, etc. and not
>"general" text editor stuff?
This does not yet exist, that's why we keep the current text editing framework free of such code. Discussions are underway whether to create an LTK layer for text editing support.