Enum refactoring plugin development

2 views
Skip to first unread message

Raffi Khatchadourian

unread,
Apr 14, 2010, 11:21:55 AM4/14/10
to convert_consta...@googlegroups.com
(apologies for the repeat)

I thought perhaps we could make some progress on the enum plugin while the GSoC applications are pending. Is there any issues in the list (http://code.google.com/p/constants-to-enum-eclipse-plugin/issues/list) that you are interested/comfortable working on? Probably the most important issue right now pertains to developing the refactoring wizard.

Benjamin, any thoughts?

Raffi

George Trigeorgis

unread,
Apr 14, 2010, 7:12:55 PM4/14/10
to convert_consta...@googlegroups.com
Hey Raffi,

Unfortunately I am very busy with exam period now, so I don't have much
time for the plugin, but while checking out the plugin, I found some
major bugs just by using some simple static final int to convert to
enums where it would convert each constant into a different enum entry.
The other one as you pointed is the wizard, which needs a lot of work
(also to let the user select which constants to convert etc.). I d love
to work on these two things first, maybe starting with the wizard to get
familiar with the plugin!

Regards,
George

Raffi Khatchadourian

unread,
Apr 15, 2010, 10:37:57 AM4/15/10
to convert_consta...@googlegroups.com
Hi George,

On Apr 14, 2010, at 7:12 PM, George Trigeorgis wrote:

> Unfortunately I am very busy with exam period now, so I don't have much
> time for the plugin, but while checking out the plugin, I found some
> major bugs just by using some simple static final int to convert to
> enums where it would convert each constant into a different enum entry.

What exactly do you mean by this? Do you mean that each constant would be a "singleton" in an individual enum type?

> The other one as you pointed is the wizard, which needs a lot of work

Agreed, but it's off to a good start thanks to Benjamin.

> (also to let the user select which constants to convert etc.).

Ah, I see the problem. You can do this by selecting multiple constants prior to starting the refactoring. Let me know if that helps.

> I d love
> to work on these two things first, maybe starting with the wizard to get
> familiar with the plugin!

Yes, that may be a good start. In fact, what would be very helpful would be to break down all the issues with the wizard into google code issues. Good luck with the exams!

Raffi

George Trigeorgis

unread,
Apr 18, 2010, 7:05:32 PM4/18/10
to convert_consta...@googlegroups.com
Firstly Im really sorry about the delayed response, but I was really
busy lately.

> Hi George,
>
> On Apr 14, 2010, at 7:12 PM, George Trigeorgis wrote:
>
> > Unfortunately I am very busy with exam period now, so I don't have much
> > time for the plugin, but while checking out the plugin, I found some
> > major bugs just by using some simple static final int to convert to
> > enums where it would convert each constant into a different enum entry.
>What exactly do you mean by this? Do you mean that each constant would be a "singleton" in an individual enum type?
Yes thats exactly what I mean :)

>
> > The other one as you pointed is the wizard, which needs a lot of work
>
> Agreed, but it's off to a good start thanks to Benjamin.
>
> > (also to let the user select which constants to convert etc.).
>
> Ah, I see the problem. You can do this by selecting multiple constants prior to starting the refactoring. Let me know if that helps.
I tried that but the result was that each constant would be a
"singleton" in an individual enum type.
>
> > I d love
> > to work on these two things first, maybe starting with the wizard to get
> > familiar with the plugin!
>
> Yes, that may be a good start. In fact, what would be very helpful would be to break down all the issues with the wizard into google code issues. Good luck with the exams!
>
Ok ! I will do that!

Regards George
> Raffi
>


--
You received this message because you are subscribed to the Google Groups "convert_constants_to_enum-dev" group.
To post to this group, send email to convert_consta...@googlegroups.com.
To unsubscribe from this group, send email to convert_constants_to...@googlegroups.com.
For more options, visit this group at http://groups.google.com/group/convert_constants_to_enum-dev?hl=en.

Raffi Khatchadourian

unread,
Apr 19, 2010, 9:17:27 PM4/19/10
to convert_consta...@googlegroups.com
George,

On Apr 18, 2010, at 7:05 PM, George Trigeorgis wrote:

> Firstly Im really sorry about the delayed response, but I was really
> busy lately.

Not a problem.

>> Hi George,
>>
>> On Apr 14, 2010, at 7:12 PM, George Trigeorgis wrote:
>>
>>> Unfortunately I am very busy with exam period now, so I don't have much
>>> time for the plugin, but while checking out the plugin, I found some
>>> major bugs just by using some simple static final int to convert to
>>> enums where it would convert each constant into a different enum entry.
>> What exactly do you mean by this? Do you mean that each constant would be a "singleton" in an individual enum type?
> Yes thats exactly what I mean :)
>
>>
>>> The other one as you pointed is the wizard, which needs a lot of work
>>
>> Agreed, but it's off to a good start thanks to Benjamin.
>>
>>> (also to let the user select which constants to convert etc.).
>>
>> Ah, I see the problem. You can do this by selecting multiple constants prior to starting the refactoring. Let me know if that helps.
> I tried that but the result was that each constant would be a
> "singleton" in an individual enum type.

I see. Could you post an issue about this on google code? Please include any steps needed to reproduce the problem. To note, the enum type creation is a bit ad hoc at the moment (see issue 16 http://tinyurl.com/y2zuv76), but multiple constants should be grouped together if all those constants are compatible. If you can, please let us know the test code you were using.

>>
>>> I d love
>>> to work on these two things first, maybe starting with the wizard to get
>>> familiar with the plugin!
>>
>> Yes, that may be a good start. In fact, what would be very helpful would be to break down all the issues with the wizard into google code issues. Good luck with the exams!
>>
> Ok ! I will do that!

Great! I look forward to reviewing the new issues!

George Trigeorgis

unread,
Apr 20, 2010, 7:54:48 PM4/20/10
to convert_consta...@googlegroups.com
On Mon, 2010-04-19 at 21:17 -0400, Raffi Khatchadourian wrote:
> George,
>
> On Apr 18, 2010, at 7:05 PM, George Trigeorgis wrote:
>
> > Firstly Im really sorry about the delayed response, but I was really
> > busy lately.
>
> Not a problem.
>
> >> Hi George,
> >>
> >> On Apr 14, 2010, at 7:12 PM, George Trigeorgis wrote:
> >>
> >>> Unfortunately I am very busy with exam period now, so I don't have much
> >>> time for the plugin, but while checking out the plugin, I found some
> >>> major bugs just by using some simple static final int to convert to
> >>> enums where it would convert each constant into a different enum entry.
> >> What exactly do you mean by this? Do you mean that each constant would be a "singleton" in an individual enum type?
> > Yes thats exactly what I mean :)
> >
> >>
> >>> The other one as you pointed is the wizard, which needs a lot of work
> >>
> >> Agreed, but it's off to a good start thanks to Benjamin.
> >>
> >>> (also to let the user select which constants to convert etc.).
> >>
> >> Ah, I see the problem. You can do this by selecting multiple constants prior to starting the refactoring. Let me know if that helps.
> > I tried that but the result was that each constant would be a
> > "singleton" in an individual enum type.
>
> I see. Could you post an issue about this on google code? Please include any steps needed to reproduce the problem. To note, the enum type creation is a bit ad hoc at the moment (see issue 16 http://tinyurl.com/y2zuv76), but multiple constants should be grouped together if all those constants are compatible. If you can, please let us know the test code you were using.
>

Done, although that this maybe a duplicate of another issue. I also
posted another issue, regarding refactoring history.
I will be able to provide more technical details on the bugs as soon we
are finished with the exams!

> >>
> >>> I d love
> >>> to work on these two things first, maybe starting with the wizard to get
> >>> familiar with the plugin!
> >>
> >> Yes, that may be a good start. In fact, what would be very helpful would be to break down all the issues with the wizard into google code issues. Good luck with the exams!
> >>
> > Ok ! I will do that!
>
> Great! I look forward to reviewing the new issues!
>
> Raffi
>

Looking forward to work with you during the summer!

Regards,
George

Raffi Khatchadourian

unread,
Apr 20, 2010, 8:29:03 PM4/20/10
to convert_consta...@googlegroups.com
Hi George,

On Apr 20, 2010, at 7:54 PM, George Trigeorgis wrote:

> Looking forward to work with you during the summer!

Thanks for submitting the issues on google code. If the SOC grant does not go through, are you still interested in working on the plug-in?

Raffi

George Trigeorgis

unread,
Apr 21, 2010, 4:07:41 AM4/21/10
to convert_consta...@googlegroups.com
On Tue, 2010-04-20 at 20:29 -0400, Raffi Khatchadourian wrote:
> Hi George,
>
> On Apr 20, 2010, at 7:54 PM, George Trigeorgis wrote:
>
> > Looking forward to work with you during the summer!
>
> Thanks for submitting the issues on google code. If the SOC grant does not go through, are you still interested in working on the plug-in?
>
Yes, sure! The SoC is just one more reason for working on the project
full-time, but I would be still interested, though!
Reply all
Reply to author
Forward
0 new messages