Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [cdt-dev] Codan preferences

Please make sure to sync to get my latest changes for https://bugs.eclipse.org/bugs/show_bug.cgi?id=345793.

-sergey

On Fri, May 13, 2011 at 6:25 PM, Alena Laskavaia <elaskavaia.cdt@xxxxxxxxx> wrote:
Sounds like a plan. I am glad it is "internal" :)

On Fri, May 13, 2011 at 9:21 PM, Sergey Prigogin
<eclipse.sprigogin@xxxxxxxxx> wrote:
>
>
> On Fri, May 13, 2011 at 6:14 PM, Alena Laskavaia <elaskavaia.cdt@xxxxxxxxx>
> wrote:
>>
>> Not sure. They are in core because they suppose to be used by headless
>> build. What do you mean initialized?
>
> org.eclipse.cdt.codan.internal.ui.preferences.PreferenceInitializer lives
> in org.eclipse.cdt.codan.ui. I think it should be moved
> to org.eclipse.cdt.codan.core.
> -sergey
>>
>> On Fri, May 13, 2011 at 8:54 PM, Sergey Prigogin
>> <eclipse.sprigogin@xxxxxxxxx> wrote:
>> > Hi Alena,
>> > I've noticed that Codan preferences that are defined in
>> > org.eclipse.cdt.codan.core are initialized in org.eclipse.cdt.codan.ui.
>> > This
>> > seems pretty dangerous since the preferences can potentially be accessed
>> > from org.eclipse.cdt.codan.core before being initialized. Could please
>> > shed
>> > some light on this. Thanks.
>> > -sergey
>
>


Back to the top