RE: [cdt-dev] Warnings in ui-plugin

["Krasilnikov, Oleg" <oleg.krasilnikov@xxxxxxxxx>]

Remaining org.eclipse.managedbuilder.ui 
warnings are generated due to following reasons:

- actions/ConvertTargetDialog.java:
- newui/NewVarDialog.java:
- newui/ToolSelectionDialog.java:
      usage of org.eclipse.cdt.internal.ui.dialogs/StatusDialog. Will be
eliminated soon.

- newui/BuilderSettingsTab.java:
      usage of deprecated methods IMakeBuilderInfo.set***BuildTarget().
Will be eliminated soon.

- newui/BuildToolSettingUI.java:
- newui/DiscoveryTab.java:
      usage of org.eclipse.cdt.internal.ui.util.PixelConverter. I have
now idea yet how to replace it.

- newui/FileListControl.java:
     usage of org.eclipse.cdt.internal.ui.dialogs.StatusInfo and
TypedViewerFilter
     This is derived code from old-style UI. I have not touch it yet,
because it works.
     Anyway, it will be rewritten soon to remove warnings.

- wizards/CMainWizardPage.java
    usage of
org.eclipse.ui.internal.ide.dialogs.ProjectContentsLocationArea.
    This is a common habit, at least old-style wizard pages used the
same thing.

- wizards/NewModelProjectWizard.java
     usage of org.eclipse.ui.internal.dialogs.PropertyDialog. Have no
idea how to replace.

That's all. 
In general, 4 files in cdt.managedbuilder.ui (+ 2 in cdt.ui) seem to be
left in warning state. 
Other warnings will be removed soon, I hope.
-----------------------------------
With best regards, Oleg Krasilnikov
Software designer, Eclipse team. 
Intel corp.
+7 8312 162 444 ext. 2587
(Russia, Nizhny Novgorod)



-----Original Message-----
From: cdt-dev-bounces@xxxxxxxxxxx [mailto:cdt-dev-bounces@xxxxxxxxxxx]
On Behalf Of Schorn, Markus
Sent: Wednesday, February 21, 2007 4:21 PM
To: CDT General developers list.
Subject: [cdt-dev] Warnings in ui-plugin

Hi,
Half a year ago, I invested into fixing all the warnings in the
ui-plugin. Recently by adding the language-mappings and the new project
model new warnings were introduced.

I think it is a good habit to keep the code clean and once more I ask
for your support in doing this. Especially new functionality added
should be free of warnings.

Please fix the warnings or explain to me why I should not care.

Markus.
_______________________________________________
cdt-dev mailing list
cdt-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/cdt-dev