diff options
Diffstat (limited to 'bundles/org.eclipse.equinox.p2.ui.discovery')
19 files changed, 308 insertions, 413 deletions
diff --git a/bundles/org.eclipse.equinox.p2.ui.discovery/.settings/org.eclipse.jdt.core.prefs b/bundles/org.eclipse.equinox.p2.ui.discovery/.settings/org.eclipse.jdt.core.prefs index 654a73733..c65423da1 100644 --- a/bundles/org.eclipse.equinox.p2.ui.discovery/.settings/org.eclipse.jdt.core.prefs +++ b/bundles/org.eclipse.equinox.p2.ui.discovery/.settings/org.eclipse.jdt.core.prefs @@ -1,13 +1,13 @@ -#Tue May 12 21:05:03 PDT 2009 +#Wed Feb 17 14:07:15 PST 2010 eclipse.preferences.version=1 -org.eclipse.jdt.core.codeComplete.argumentPrefixes= -org.eclipse.jdt.core.codeComplete.argumentSuffixes= -org.eclipse.jdt.core.codeComplete.fieldPrefixes= -org.eclipse.jdt.core.codeComplete.fieldSuffixes= -org.eclipse.jdt.core.codeComplete.localPrefixes= -org.eclipse.jdt.core.codeComplete.localSuffixes= -org.eclipse.jdt.core.codeComplete.staticFieldPrefixes= -org.eclipse.jdt.core.codeComplete.staticFieldSuffixes= +org.eclipse.jdt.core.builder.cleanOutputFolder=clean +org.eclipse.jdt.core.builder.duplicateResourceTask=warning +org.eclipse.jdt.core.builder.invalidClasspath=abort +org.eclipse.jdt.core.builder.recreateModifiedClassFileInOutputFolder=ignore +org.eclipse.jdt.core.builder.resourceCopyExclusionFilter=*.launch +org.eclipse.jdt.core.circularClasspath=error +org.eclipse.jdt.core.classpath.exclusionPatterns=enabled +org.eclipse.jdt.core.classpath.multipleOutputLocations=enabled org.eclipse.jdt.core.compiler.codegen.inlineJsrBytecode=enabled org.eclipse.jdt.core.compiler.codegen.targetPlatform=1.5 org.eclipse.jdt.core.compiler.codegen.unusedLocal=preserve @@ -15,70 +15,83 @@ org.eclipse.jdt.core.compiler.compliance=1.5 org.eclipse.jdt.core.compiler.debug.lineNumber=generate org.eclipse.jdt.core.compiler.debug.localVariable=generate org.eclipse.jdt.core.compiler.debug.sourceFile=generate +org.eclipse.jdt.core.compiler.doc.comment.support=enabled +org.eclipse.jdt.core.compiler.maxProblemPerUnit=1000 org.eclipse.jdt.core.compiler.problem.annotationSuperInterface=warning org.eclipse.jdt.core.compiler.problem.assertIdentifier=error org.eclipse.jdt.core.compiler.problem.autoboxing=ignore +org.eclipse.jdt.core.compiler.problem.comparingIdentical=warning +org.eclipse.jdt.core.compiler.problem.deadCode=warning org.eclipse.jdt.core.compiler.problem.deprecation=warning org.eclipse.jdt.core.compiler.problem.deprecationInDeprecatedCode=disabled org.eclipse.jdt.core.compiler.problem.deprecationWhenOverridingDeprecatedMethod=enabled org.eclipse.jdt.core.compiler.problem.discouragedReference=warning -org.eclipse.jdt.core.compiler.problem.emptyStatement=ignore +org.eclipse.jdt.core.compiler.problem.emptyStatement=warning org.eclipse.jdt.core.compiler.problem.enumIdentifier=error org.eclipse.jdt.core.compiler.problem.fallthroughCase=ignore -org.eclipse.jdt.core.compiler.problem.fatalOptionalError=enabled -org.eclipse.jdt.core.compiler.problem.fieldHiding=ignore -org.eclipse.jdt.core.compiler.problem.finalParameterBound=warning +org.eclipse.jdt.core.compiler.problem.fatalOptionalError=disabled +org.eclipse.jdt.core.compiler.problem.fieldHiding=warning +org.eclipse.jdt.core.compiler.problem.finalParameterBound=ignore org.eclipse.jdt.core.compiler.problem.finallyBlockNotCompletingNormally=warning org.eclipse.jdt.core.compiler.problem.forbiddenReference=error org.eclipse.jdt.core.compiler.problem.hiddenCatchBlock=warning org.eclipse.jdt.core.compiler.problem.incompatibleNonInheritedInterfaceMethod=warning org.eclipse.jdt.core.compiler.problem.incompleteEnumSwitch=ignore -org.eclipse.jdt.core.compiler.problem.indirectStaticAccess=ignore -org.eclipse.jdt.core.compiler.problem.localVariableHiding=ignore +org.eclipse.jdt.core.compiler.problem.indirectStaticAccess=warning +org.eclipse.jdt.core.compiler.problem.invalidJavadoc=error +org.eclipse.jdt.core.compiler.problem.invalidJavadocTagsVisibility=private +org.eclipse.jdt.core.compiler.problem.localVariableHiding=warning org.eclipse.jdt.core.compiler.problem.methodWithConstructorName=warning org.eclipse.jdt.core.compiler.problem.missingDeprecatedAnnotation=ignore +org.eclipse.jdt.core.compiler.problem.missingHashCodeMethod=ignore +org.eclipse.jdt.core.compiler.problem.missingJavadocComments=ignore +org.eclipse.jdt.core.compiler.problem.missingJavadocCommentsOverriding=enabled +org.eclipse.jdt.core.compiler.problem.missingJavadocCommentsVisibility=public +org.eclipse.jdt.core.compiler.problem.missingJavadocTags=ignore +org.eclipse.jdt.core.compiler.problem.missingJavadocTagsOverriding=enabled +org.eclipse.jdt.core.compiler.problem.missingJavadocTagsVisibility=public org.eclipse.jdt.core.compiler.problem.missingOverrideAnnotation=ignore +org.eclipse.jdt.core.compiler.problem.missingOverrideAnnotationForInterfaceMethodImplementation=enabled org.eclipse.jdt.core.compiler.problem.missingSerialVersion=warning +org.eclipse.jdt.core.compiler.problem.missingSynchronizedOnInheritedMethod=ignore org.eclipse.jdt.core.compiler.problem.noEffectAssignment=warning org.eclipse.jdt.core.compiler.problem.noImplicitStringConversion=warning org.eclipse.jdt.core.compiler.problem.nonExternalizedStringLiteral=warning -org.eclipse.jdt.core.compiler.problem.nullReference=error +org.eclipse.jdt.core.compiler.problem.nullReference=warning org.eclipse.jdt.core.compiler.problem.overridingPackageDefaultMethod=warning org.eclipse.jdt.core.compiler.problem.parameterAssignment=ignore -org.eclipse.jdt.core.compiler.problem.possibleAccidentalBooleanAssignment=ignore -org.eclipse.jdt.core.compiler.problem.potentialNullReference=warning +org.eclipse.jdt.core.compiler.problem.possibleAccidentalBooleanAssignment=warning +org.eclipse.jdt.core.compiler.problem.potentialNullReference=ignore org.eclipse.jdt.core.compiler.problem.rawTypeReference=warning org.eclipse.jdt.core.compiler.problem.redundantNullCheck=ignore org.eclipse.jdt.core.compiler.problem.redundantSuperinterface=ignore org.eclipse.jdt.core.compiler.problem.specialParameterHidingField=disabled org.eclipse.jdt.core.compiler.problem.staticAccessReceiver=warning org.eclipse.jdt.core.compiler.problem.suppressWarnings=enabled -org.eclipse.jdt.core.compiler.problem.syntheticAccessEmulation=ignore +org.eclipse.jdt.core.compiler.problem.syntheticAccessEmulation=warning org.eclipse.jdt.core.compiler.problem.typeParameterHiding=warning org.eclipse.jdt.core.compiler.problem.uncheckedTypeOperation=warning -org.eclipse.jdt.core.compiler.problem.undocumentedEmptyBlock=ignore +org.eclipse.jdt.core.compiler.problem.undocumentedEmptyBlock=warning org.eclipse.jdt.core.compiler.problem.unhandledWarningToken=warning -org.eclipse.jdt.core.compiler.problem.unnecessaryElse=ignore -org.eclipse.jdt.core.compiler.problem.unnecessaryTypeCheck=ignore +org.eclipse.jdt.core.compiler.problem.unnecessaryElse=warning +org.eclipse.jdt.core.compiler.problem.unnecessaryTypeCheck=warning org.eclipse.jdt.core.compiler.problem.unqualifiedFieldAccess=ignore -org.eclipse.jdt.core.compiler.problem.unusedDeclaredThrownException=ignore +org.eclipse.jdt.core.compiler.problem.unusedDeclaredThrownException=warning org.eclipse.jdt.core.compiler.problem.unusedDeclaredThrownExceptionExemptExceptionAndThrowable=enabled org.eclipse.jdt.core.compiler.problem.unusedDeclaredThrownExceptionIncludeDocCommentReference=enabled -org.eclipse.jdt.core.compiler.problem.unusedDeclaredThrownExceptionWhenOverriding=disabled -org.eclipse.jdt.core.compiler.problem.unusedImport=warning +org.eclipse.jdt.core.compiler.problem.unusedDeclaredThrownExceptionWhenOverriding=enabled +org.eclipse.jdt.core.compiler.problem.unusedImport=error org.eclipse.jdt.core.compiler.problem.unusedLabel=warning org.eclipse.jdt.core.compiler.problem.unusedLocal=warning +org.eclipse.jdt.core.compiler.problem.unusedObjectAllocation=ignore org.eclipse.jdt.core.compiler.problem.unusedParameter=ignore org.eclipse.jdt.core.compiler.problem.unusedParameterIncludeDocCommentReference=enabled -org.eclipse.jdt.core.compiler.problem.unusedParameterWhenImplementingAbstract=disabled -org.eclipse.jdt.core.compiler.problem.unusedParameterWhenOverridingConcrete=disabled -org.eclipse.jdt.core.compiler.problem.unusedPrivateMember=warning +org.eclipse.jdt.core.compiler.problem.unusedParameterWhenImplementingAbstract=enabled +org.eclipse.jdt.core.compiler.problem.unusedParameterWhenOverridingConcrete=enabled +org.eclipse.jdt.core.compiler.problem.unusedPrivateMember=error org.eclipse.jdt.core.compiler.problem.unusedWarningToken=warning org.eclipse.jdt.core.compiler.problem.varargsArgumentNeedCast=warning org.eclipse.jdt.core.compiler.source=1.5 -org.eclipse.jdt.core.compiler.taskCaseSensitive=enabled -org.eclipse.jdt.core.compiler.taskPriorities=NORMAL,HIGH,NORMAL -org.eclipse.jdt.core.compiler.taskTags=TODO,FIXME,XXX org.eclipse.jdt.core.formatter.align_type_members_on_columns=false org.eclipse.jdt.core.formatter.alignment_for_arguments_in_allocation_expression=16 org.eclipse.jdt.core.formatter.alignment_for_arguments_in_enum_constant=16 @@ -94,7 +107,7 @@ org.eclipse.jdt.core.formatter.alignment_for_expressions_in_array_initializer=16 org.eclipse.jdt.core.formatter.alignment_for_multiple_fields=16 org.eclipse.jdt.core.formatter.alignment_for_parameters_in_constructor_declaration=16 org.eclipse.jdt.core.formatter.alignment_for_parameters_in_method_declaration=16 -org.eclipse.jdt.core.formatter.alignment_for_selector_in_method_invocation=80 +org.eclipse.jdt.core.formatter.alignment_for_selector_in_method_invocation=16 org.eclipse.jdt.core.formatter.alignment_for_superclass_in_type_declaration=16 org.eclipse.jdt.core.formatter.alignment_for_superinterfaces_in_enum_declaration=16 org.eclipse.jdt.core.formatter.alignment_for_superinterfaces_in_type_declaration=16 @@ -102,7 +115,7 @@ org.eclipse.jdt.core.formatter.alignment_for_throws_clause_in_constructor_declar org.eclipse.jdt.core.formatter.alignment_for_throws_clause_in_method_declaration=16 org.eclipse.jdt.core.formatter.blank_lines_after_imports=1 org.eclipse.jdt.core.formatter.blank_lines_after_package=1 -org.eclipse.jdt.core.formatter.blank_lines_before_field=1 +org.eclipse.jdt.core.formatter.blank_lines_before_field=0 org.eclipse.jdt.core.formatter.blank_lines_before_first_class_body_declaration=0 org.eclipse.jdt.core.formatter.blank_lines_before_imports=1 org.eclipse.jdt.core.formatter.blank_lines_before_member_type=1 @@ -123,18 +136,18 @@ org.eclipse.jdt.core.formatter.brace_position_for_method_declaration=end_of_line org.eclipse.jdt.core.formatter.brace_position_for_switch=end_of_line org.eclipse.jdt.core.formatter.brace_position_for_type_declaration=end_of_line org.eclipse.jdt.core.formatter.comment.clear_blank_lines_in_block_comment=false -org.eclipse.jdt.core.formatter.comment.clear_blank_lines_in_javadoc_comment=true +org.eclipse.jdt.core.formatter.comment.clear_blank_lines_in_javadoc_comment=false org.eclipse.jdt.core.formatter.comment.format_block_comments=false org.eclipse.jdt.core.formatter.comment.format_header=false org.eclipse.jdt.core.formatter.comment.format_html=true -org.eclipse.jdt.core.formatter.comment.format_javadoc_comments=true +org.eclipse.jdt.core.formatter.comment.format_javadoc_comments=false org.eclipse.jdt.core.formatter.comment.format_line_comments=false org.eclipse.jdt.core.formatter.comment.format_source_code=true -org.eclipse.jdt.core.formatter.comment.indent_parameter_description=true -org.eclipse.jdt.core.formatter.comment.indent_root_tags=true +org.eclipse.jdt.core.formatter.comment.indent_parameter_description=false +org.eclipse.jdt.core.formatter.comment.indent_root_tags=false org.eclipse.jdt.core.formatter.comment.insert_new_line_before_root_tags=insert -org.eclipse.jdt.core.formatter.comment.insert_new_line_for_parameter=insert -org.eclipse.jdt.core.formatter.comment.line_length=120 +org.eclipse.jdt.core.formatter.comment.insert_new_line_for_parameter=do not insert +org.eclipse.jdt.core.formatter.comment.line_length=80 org.eclipse.jdt.core.formatter.compact_else_if=true org.eclipse.jdt.core.formatter.continuation_indentation=2 org.eclipse.jdt.core.formatter.continuation_indentation_for_array_initializer=2 @@ -148,11 +161,8 @@ org.eclipse.jdt.core.formatter.indent_empty_lines=false org.eclipse.jdt.core.formatter.indent_statements_compare_to_block=true org.eclipse.jdt.core.formatter.indent_statements_compare_to_body=true org.eclipse.jdt.core.formatter.indent_switchstatements_compare_to_cases=true -org.eclipse.jdt.core.formatter.indent_switchstatements_compare_to_switch=false +org.eclipse.jdt.core.formatter.indent_switchstatements_compare_to_switch=true org.eclipse.jdt.core.formatter.indentation.size=4 -org.eclipse.jdt.core.formatter.insert_new_line_after_annotation_on_local_variable=insert -org.eclipse.jdt.core.formatter.insert_new_line_after_annotation_on_member=insert -org.eclipse.jdt.core.formatter.insert_new_line_after_annotation_on_parameter=do not insert org.eclipse.jdt.core.formatter.insert_new_line_after_opening_brace_in_array_initializer=do not insert org.eclipse.jdt.core.formatter.insert_new_line_at_end_of_file_if_missing=do not insert org.eclipse.jdt.core.formatter.insert_new_line_before_catch_in_try_statement=do not insert @@ -161,7 +171,7 @@ org.eclipse.jdt.core.formatter.insert_new_line_before_else_in_if_statement=do no org.eclipse.jdt.core.formatter.insert_new_line_before_finally_in_try_statement=do not insert org.eclipse.jdt.core.formatter.insert_new_line_before_while_in_do_statement=do not insert org.eclipse.jdt.core.formatter.insert_new_line_in_empty_annotation_declaration=insert -org.eclipse.jdt.core.formatter.insert_new_line_in_empty_anonymous_type_declaration=insert +org.eclipse.jdt.core.formatter.insert_new_line_in_empty_anonymous_type_declaration=do not insert org.eclipse.jdt.core.formatter.insert_new_line_in_empty_block=insert org.eclipse.jdt.core.formatter.insert_new_line_in_empty_enum_constant=insert org.eclipse.jdt.core.formatter.insert_new_line_in_empty_enum_declaration=insert @@ -204,7 +214,7 @@ org.eclipse.jdt.core.formatter.insert_space_after_ellipsis=insert org.eclipse.jdt.core.formatter.insert_space_after_opening_angle_bracket_in_parameterized_type_reference=do not insert org.eclipse.jdt.core.formatter.insert_space_after_opening_angle_bracket_in_type_arguments=do not insert org.eclipse.jdt.core.formatter.insert_space_after_opening_angle_bracket_in_type_parameters=do not insert -org.eclipse.jdt.core.formatter.insert_space_after_opening_brace_in_array_initializer=insert +org.eclipse.jdt.core.formatter.insert_space_after_opening_brace_in_array_initializer=do not insert org.eclipse.jdt.core.formatter.insert_space_after_opening_bracket_in_array_allocation_expression=do not insert org.eclipse.jdt.core.formatter.insert_space_after_opening_bracket_in_array_reference=do not insert org.eclipse.jdt.core.formatter.insert_space_after_opening_paren_in_annotation=do not insert @@ -233,7 +243,7 @@ org.eclipse.jdt.core.formatter.insert_space_before_binary_operator=insert org.eclipse.jdt.core.formatter.insert_space_before_closing_angle_bracket_in_parameterized_type_reference=do not insert org.eclipse.jdt.core.formatter.insert_space_before_closing_angle_bracket_in_type_arguments=do not insert org.eclipse.jdt.core.formatter.insert_space_before_closing_angle_bracket_in_type_parameters=do not insert -org.eclipse.jdt.core.formatter.insert_space_before_closing_brace_in_array_initializer=insert +org.eclipse.jdt.core.formatter.insert_space_before_closing_brace_in_array_initializer=do not insert org.eclipse.jdt.core.formatter.insert_space_before_closing_bracket_in_array_allocation_expression=do not insert org.eclipse.jdt.core.formatter.insert_space_before_closing_bracket_in_array_reference=do not insert org.eclipse.jdt.core.formatter.insert_space_before_closing_paren_in_annotation=do not insert @@ -250,9 +260,9 @@ org.eclipse.jdt.core.formatter.insert_space_before_closing_paren_in_switch=do no org.eclipse.jdt.core.formatter.insert_space_before_closing_paren_in_synchronized=do not insert org.eclipse.jdt.core.formatter.insert_space_before_closing_paren_in_while=do not insert org.eclipse.jdt.core.formatter.insert_space_before_colon_in_assert=insert -org.eclipse.jdt.core.formatter.insert_space_before_colon_in_case=do not insert +org.eclipse.jdt.core.formatter.insert_space_before_colon_in_case=insert org.eclipse.jdt.core.formatter.insert_space_before_colon_in_conditional=insert -org.eclipse.jdt.core.formatter.insert_space_before_colon_in_default=do not insert +org.eclipse.jdt.core.formatter.insert_space_before_colon_in_default=insert org.eclipse.jdt.core.formatter.insert_space_before_colon_in_for=insert org.eclipse.jdt.core.formatter.insert_space_before_colon_in_labeled_statement=do not insert org.eclipse.jdt.core.formatter.insert_space_before_comma_in_allocation_expression=do not insert @@ -321,19 +331,19 @@ org.eclipse.jdt.core.formatter.insert_space_between_empty_parens_in_constructor_ org.eclipse.jdt.core.formatter.insert_space_between_empty_parens_in_enum_constant=do not insert org.eclipse.jdt.core.formatter.insert_space_between_empty_parens_in_method_declaration=do not insert org.eclipse.jdt.core.formatter.insert_space_between_empty_parens_in_method_invocation=do not insert -org.eclipse.jdt.core.formatter.join_lines_in_comments=true -org.eclipse.jdt.core.formatter.join_wrapped_lines=true org.eclipse.jdt.core.formatter.keep_else_statement_on_same_line=false org.eclipse.jdt.core.formatter.keep_empty_array_initializer_on_one_line=false org.eclipse.jdt.core.formatter.keep_imple_if_on_one_line=false org.eclipse.jdt.core.formatter.keep_then_statement_on_same_line=false -org.eclipse.jdt.core.formatter.lineSplit=120 -org.eclipse.jdt.core.formatter.never_indent_block_comments_on_first_column=true -org.eclipse.jdt.core.formatter.never_indent_line_comments_on_first_column=true +org.eclipse.jdt.core.formatter.lineSplit=800 +org.eclipse.jdt.core.formatter.never_indent_block_comments_on_first_column=false +org.eclipse.jdt.core.formatter.never_indent_line_comments_on_first_column=false org.eclipse.jdt.core.formatter.number_of_blank_lines_at_beginning_of_method_body=0 org.eclipse.jdt.core.formatter.number_of_empty_lines_to_preserve=1 -org.eclipse.jdt.core.formatter.put_empty_statement_on_new_line=true +org.eclipse.jdt.core.formatter.put_empty_statement_on_new_line=false org.eclipse.jdt.core.formatter.tabulation.char=tab org.eclipse.jdt.core.formatter.tabulation.size=4 org.eclipse.jdt.core.formatter.use_tabs_only_for_leading_indentations=false org.eclipse.jdt.core.formatter.wrap_before_binary_operator=true +org.eclipse.jdt.core.incompatibleJDKLevel=ignore +org.eclipse.jdt.core.incompleteClasspath=error diff --git a/bundles/org.eclipse.equinox.p2.ui.discovery/.settings/org.eclipse.jdt.ui.prefs b/bundles/org.eclipse.equinox.p2.ui.discovery/.settings/org.eclipse.jdt.ui.prefs index 8d68e732b..8f3db4a0a 100644 --- a/bundles/org.eclipse.equinox.p2.ui.discovery/.settings/org.eclipse.jdt.ui.prefs +++ b/bundles/org.eclipse.equinox.p2.ui.discovery/.settings/org.eclipse.jdt.ui.prefs @@ -1,21 +1,17 @@ -#Thu Sep 11 16:27:18 PDT 2008 -cleanup_settings_version=2 +#Wed Feb 17 14:07:15 PST 2010 eclipse.preferences.version=1 editor_save_participant_org.eclipse.jdt.ui.postsavelistener.cleanup=true -formatter_profile=_Mylyn based on Eclipse +formatter_profile=_core formatter_settings_version=11 -internal.default.compliance=default -org.eclipse.jdt.ui.exception.name=e -org.eclipse.jdt.ui.gettersetter.use.is=true -org.eclipse.jdt.ui.javadoc=false -org.eclipse.jdt.ui.keywordthis=false -org.eclipse.jdt.ui.overrideannotation=true -org.eclipse.jdt.ui.text.custom_code_templates=<?xml version\="1.0" encoding\="UTF-8" standalone\="no"?><templates><template autoinsert\="true" context\="gettercomment_context" deleted\="false" description\="Comment for getter method" enabled\="true" id\="org.eclipse.jdt.ui.text.codetemplates.gettercomment" name\="gettercomment">/**\r\n * @return the ${bare_field_name}\r\n */</template><template autoinsert\="true" context\="settercomment_context" deleted\="false" description\="Comment for setter method" enabled\="true" id\="org.eclipse.jdt.ui.text.codetemplates.settercomment" name\="settercomment">/**\r\n * @param ${param} the ${bare_field_name} to set\r\n */</template><template autoinsert\="true" context\="constructorcomment_context" deleted\="false" description\="Comment for created constructors" enabled\="true" id\="org.eclipse.jdt.ui.text.codetemplates.constructorcomment" name\="constructorcomment">/**\r\n * ${tags}\r\n */</template><template autoinsert\="true" context\="filecomment_context" deleted\="false" description\="Comment for created Java files" enabled\="true" id\="org.eclipse.jdt.ui.text.codetemplates.filecomment" name\="filecomment">/**\r\n * \r\n */</template><template autoinsert\="false" context\="typecomment_context" deleted\="false" description\="Comment for created types" enabled\="true" id\="org.eclipse.jdt.ui.text.codetemplates.typecomment" name\="typecomment">/**\r\n * @author ${user}\r\n */</template><template autoinsert\="true" context\="fieldcomment_context" deleted\="false" description\="Comment for fields" enabled\="true" id\="org.eclipse.jdt.ui.text.codetemplates.fieldcomment" name\="fieldcomment">/**\r\n * \r\n */</template><template autoinsert\="true" context\="methodcomment_context" deleted\="false" description\="Comment for non-overriding methods" enabled\="true" id\="org.eclipse.jdt.ui.text.codetemplates.methodcomment" name\="methodcomment">/**\r\n * ${tags}\r\n */</template><template autoinsert\="false" context\="overridecomment_context" deleted\="false" description\="Comment for overriding methods" enabled\="true" id\="org.eclipse.jdt.ui.text.codetemplates.overridecomment" name\="overridecomment"/><template autoinsert\="false" context\="newtype_context" deleted\="false" description\="Newly created files" enabled\="true" id\="org.eclipse.jdt.ui.text.codetemplates.newtype" name\="newtype">/*******************************************************************************\r\n * Copyright (c) 2010 Tasktop Technologies and others.\r\n * All rights reserved. This program and the accompanying materials\r\n * are made available under the terms of the Eclipse Public License v1.0\r\n * which accompanies this distribution, and is available at\r\n * http\://www.eclipse.org/legal/epl-v10.html\r\n *\r\n * Contributors\:\r\n * Tasktop Technologies - initial API and implementation\r\n *******************************************************************************/\r\n\r\n${package_declaration}\r\n\r\n${typecomment}\r\n${type_declaration}</template><template autoinsert\="true" context\="classbody_context" deleted\="false" description\="Code in new class type bodies" enabled\="true" id\="org.eclipse.jdt.ui.text.codetemplates.classbody" name\="classbody">\r\n</template><template autoinsert\="true" context\="interfacebody_context" deleted\="false" description\="Code in new interface type bodies" enabled\="true" id\="org.eclipse.jdt.ui.text.codetemplates.interfacebody" name\="interfacebody">\r\n</template><template autoinsert\="true" context\="enumbody_context" deleted\="false" description\="Code in new enum type bodies" enabled\="true" id\="org.eclipse.jdt.ui.text.codetemplates.enumbody" name\="enumbody">\r\n</template><template autoinsert\="true" context\="annotationbody_context" deleted\="false" description\="Code in new annotation type bodies" enabled\="true" id\="org.eclipse.jdt.ui.text.codetemplates.annotationbody" name\="annotationbody">\r\n</template><template autoinsert\="false" context\="catchblock_context" deleted\="false" description\="Code in new catch blocks" enabled\="true" id\="org.eclipse.jdt.ui.text.codetemplates.catchblock" name\="catchblock">// ${todo} Auto-generated catch block\r\n${exception_var}.printStackTrace();</template><template autoinsert\="false" context\="methodbody_context" deleted\="false" description\="Code in created method stubs" enabled\="true" id\="org.eclipse.jdt.ui.text.codetemplates.methodbody" name\="methodbody">// ignore\r\n${body_statement}</template><template autoinsert\="false" context\="constructorbody_context" deleted\="false" description\="Code in created constructor stubs" enabled\="true" id\="org.eclipse.jdt.ui.text.codetemplates.constructorbody" name\="constructorbody">${body_statement}\r\n// ignore</template><template autoinsert\="true" context\="getterbody_context" deleted\="false" description\="Code in created getters" enabled\="true" id\="org.eclipse.jdt.ui.text.codetemplates.getterbody" name\="getterbody">return ${field};</template><template autoinsert\="true" context\="setterbody_context" deleted\="false" description\="Code in created setters" enabled\="true" id\="org.eclipse.jdt.ui.text.codetemplates.setterbody" name\="setterbody">${field} \= ${param};</template><template autoinsert\="true" context\="delegatecomment_context" deleted\="false" description\="Comment for delegate methods" enabled\="true" id\="org.eclipse.jdt.ui.text.codetemplates.delegatecomment" name\="delegatecomment">/**\r\n * ${tags}\r\n * ${see_to_target}\r\n */</template><template autoinsert\="true" context\="gettercomment_context" deleted\="false" description\="Comment for getter function" enabled\="true" id\="org.eclipse.wst.jsdt.ui.text.codetemplates.gettercomment" name\="gettercomment">/**\r\n * @return the ${bare_field_name}\r\n */</template><template autoinsert\="true" context\="settercomment_context" deleted\="false" description\="Comment for setter function" enabled\="true" id\="org.eclipse.wst.jsdt.ui.text.codetemplates.settercomment" name\="settercomment">/**\r\n * @param ${param} the ${bare_field_name} to set\r\n */</template><template autoinsert\="true" context\="constructorcomment_context" deleted\="false" description\="Comment for created constructors" enabled\="true" id\="org.eclipse.wst.jsdt.ui.text.codetemplates.constructorcomment" name\="constructorcomment">/**\r\n * ${tags}\r\n */</template><template autoinsert\="true" context\="filecomment_context" deleted\="false" description\="Comment for created JavaScript files" enabled\="true" id\="org.eclipse.wst.jsdt.ui.text.codetemplates.filecomment" name\="filecomment">/**\r\n * \r\n */</template><template autoinsert\="true" context\="typecomment_context" deleted\="false" description\="Comment for created types" enabled\="true" id\="org.eclipse.wst.jsdt.ui.text.codetemplates.typecomment" name\="typecomment">/**\r\n * @author ${user}\r\n *\r\n * ${tags}\r\n */</template><template autoinsert\="true" context\="fieldcomment_context" deleted\="false" description\="Comment for vars" enabled\="true" id\="org.eclipse.wst.jsdt.ui.text.codetemplates.fieldcomment" name\="fieldcomment">/**\r\n * \r\n */</template><template autoinsert\="true" context\="methodcomment_context" deleted\="false" description\="Comment for non-overriding function" enabled\="true" id\="org.eclipse.wst.jsdt.ui.text.codetemplates.methodcomment" name\="methodcomment">/**\r\n * ${tags}\r\n */</template><template autoinsert\="true" context\="overridecomment_context" deleted\="false" description\="Comment for overriding functions" enabled\="true" id\="org.eclipse.wst.jsdt.ui.text.codetemplates.overridecomment" name\="overridecomment">/* (non-Jsdoc)\r\n * ${see_to_overridden}\r\n */</template><template autoinsert\="true" context\="delegatecomment_context" deleted\="false" description\="Comment for delegate functions" enabled\="true" id\="org.eclipse.wst.jsdt.ui.text.codetemplates.delegatecomment" name\="delegatecomment">/**\r\n * ${tags}\r\n * ${see_to_target}\r\n */</template><template autoinsert\="true" context\="newtype_context" deleted\="false" description\="Newly created files" enabled\="true" id\="org.eclipse.wst.jsdt.ui.text.codetemplates.newtype" name\="newtype">${filecomment}\r\n${package_declaration}\r\n\r\n${typecomment}\r\n${type_declaration}</template><template autoinsert\="true" context\="classbody_context" deleted\="false" description\="Code in new class type bodies" enabled\="true" id\="org.eclipse.wst.jsdt.ui.text.codetemplates.classbody" name\="classbody">\r\n</template><template autoinsert\="true" context\="interfacebody_context" deleted\="false" description\="Code in new interface type bodies" enabled\="true" id\="org.eclipse.wst.jsdt.ui.text.codetemplates.interfacebody" name\="interfacebody">\r\n</template><template autoinsert\="true" context\="enumbody_context" deleted\="false" description\="Code in new enum type bodies" enabled\="true" id\="org.eclipse.wst.jsdt.ui.text.codetemplates.enumbody" name\="enumbody">\r\n</template><template autoinsert\="true" context\="annotationbody_context" deleted\="false" description\="Code in new annotation type bodies" enabled\="true" id\="org.eclipse.wst.jsdt.ui.text.codetemplates.annotationbody" name\="annotationbody">\r\n</template><template autoinsert\="true" context\="catchblock_context" deleted\="false" description\="Code in new catch blocks" enabled\="true" id\="org.eclipse.wst.jsdt.ui.text.codetemplates.catchblock" name\="catchblock">// ${todo} Auto-generated catch block\r\n${exception_var}.printStackTrace();</template><template autoinsert\="true" context\="methodbody_context" deleted\="false" description\="Code in created function stubs" enabled\="true" id\="org.eclipse.wst.jsdt.ui.text.codetemplates.methodbody" name\="methodbody">// ${todo} Auto-generated function stub\r\n${body_statement}</template><template autoinsert\="true" context\="constructorbody_context" deleted\="false" description\="Code in created constructor stubs" enabled\="true" id\="org.eclipse.wst.jsdt.ui.text.codetemplates.constructorbody" name\="constructorbody">${body_statement}\r\n// ${todo} Auto-generated constructor stub</template><template autoinsert\="true" context\="getterbody_context" deleted\="false" description\="Code in created getters" enabled\="true" id\="org.eclipse.wst.jsdt.ui.text.codetemplates.getterbody" name\="getterbody">return ${field};</template><template autoinsert\="true" context\="setterbody_context" deleted\="false" description\="Code in created setters" enabled\="true" id\="org.eclipse.wst.jsdt.ui.text.codetemplates.setterbody" name\="setterbody">${field} \= ${param};</template></templates> +org.eclipse.jdt.ui.ignorelowercasenames=true +org.eclipse.jdt.ui.importorder=; +org.eclipse.jdt.ui.ondemandthreshold=3 +org.eclipse.jdt.ui.staticondemandthreshold=3 +org.eclipse.jdt.ui.text.custom_code_templates=<?xml version\="1.0" encoding\="UTF-8" standalone\="no"?><templates/> sp_cleanup.add_default_serial_version_id=true sp_cleanup.add_generated_serial_version_id=false sp_cleanup.add_missing_annotations=true sp_cleanup.add_missing_deprecated_annotations=true -sp_cleanup.add_missing_methods=false sp_cleanup.add_missing_nls_tags=false sp_cleanup.add_missing_override_annotations=true sp_cleanup.add_serial_version_id=false @@ -23,29 +19,27 @@ sp_cleanup.always_use_blocks=true sp_cleanup.always_use_parentheses_in_expressions=false sp_cleanup.always_use_this_for_non_static_field_access=false sp_cleanup.always_use_this_for_non_static_method_access=false -sp_cleanup.convert_to_enhanced_for_loop=true -sp_cleanup.correct_indentation=true +sp_cleanup.convert_to_enhanced_for_loop=false sp_cleanup.format_source_code=true -sp_cleanup.format_source_code_changes_only=false sp_cleanup.make_local_variable_final=false sp_cleanup.make_parameters_final=false sp_cleanup.make_private_fields_final=true sp_cleanup.make_variable_declarations_final=true sp_cleanup.never_use_blocks=false sp_cleanup.never_use_parentheses_in_expressions=true -sp_cleanup.on_save_use_additional_actions=true +sp_cleanup.on_save_use_additional_actions=false sp_cleanup.organize_imports=true sp_cleanup.qualify_static_field_accesses_with_declaring_class=false sp_cleanup.qualify_static_member_accesses_through_instances_with_declaring_class=true sp_cleanup.qualify_static_member_accesses_through_subtypes_with_declaring_class=true -sp_cleanup.qualify_static_member_accesses_with_declaring_class=true +sp_cleanup.qualify_static_member_accesses_with_declaring_class=false sp_cleanup.qualify_static_method_accesses_with_declaring_class=false sp_cleanup.remove_private_constructors=true -sp_cleanup.remove_trailing_whitespaces=true +sp_cleanup.remove_trailing_whitespaces=false sp_cleanup.remove_trailing_whitespaces_all=true sp_cleanup.remove_trailing_whitespaces_ignore_empty=false sp_cleanup.remove_unnecessary_casts=true -sp_cleanup.remove_unnecessary_nls_tags=true +sp_cleanup.remove_unnecessary_nls_tags=false sp_cleanup.remove_unused_imports=false sp_cleanup.remove_unused_local_variables=false sp_cleanup.remove_unused_private_fields=true @@ -54,7 +48,7 @@ sp_cleanup.remove_unused_private_methods=true sp_cleanup.remove_unused_private_types=true sp_cleanup.sort_members=false sp_cleanup.sort_members_all=false -sp_cleanup.use_blocks=true +sp_cleanup.use_blocks=false sp_cleanup.use_blocks_only_for_return_and_throw=false sp_cleanup.use_parentheses_in_expressions=false sp_cleanup.use_this_for_non_static_field_access=false diff --git a/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/DiscoveryUi.java b/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/DiscoveryUi.java index dcf065fdb..e6edb58a1 100644 --- a/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/DiscoveryUi.java +++ b/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/DiscoveryUi.java @@ -12,7 +12,6 @@ package org.eclipse.equinox.internal.p2.ui.discovery; import java.lang.reflect.InvocationTargetException; import java.util.List; - import org.eclipse.core.runtime.IStatus; import org.eclipse.core.runtime.Status; import org.eclipse.equinox.internal.p2.discovery.model.CatalogItem; @@ -36,6 +35,7 @@ public abstract class DiscoveryUi { private static CommonColors commonColors; private DiscoveryUi() { + // don't allow clients to instantiate } public static boolean install(List<CatalogItem> descriptors, IRunnableContext context) { @@ -43,9 +43,7 @@ public abstract class DiscoveryUi { IRunnableWithProgress runner = new DiscoveryInstallOperation(descriptors); context.run(true, true, runner); } catch (InvocationTargetException e) { - IStatus status = new Status(IStatus.ERROR, DiscoveryUi.ID_PLUGIN, NLS.bind( - Messages.ConnectorDiscoveryWizard_installProblems, new Object[] { e.getCause().getMessage() }), - e.getCause()); + IStatus status = new Status(IStatus.ERROR, DiscoveryUi.ID_PLUGIN, NLS.bind(Messages.ConnectorDiscoveryWizard_installProblems, new Object[] {e.getCause().getMessage()}), e.getCause()); StatusManager.getManager().handle(status, StatusManager.SHOW | StatusManager.BLOCK | StatusManager.LOG); return false; } catch (InterruptedException e) { diff --git a/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/commands/ShowBundleCatalogCommandHandler.java b/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/commands/ShowBundleCatalogCommandHandler.java index 4614377f9..43aa1e02e 100644 --- a/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/commands/ShowBundleCatalogCommandHandler.java +++ b/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/commands/ShowBundleCatalogCommandHandler.java @@ -11,15 +11,11 @@ package org.eclipse.equinox.internal.p2.ui.discovery.commands; import java.util.Arrays; - import org.eclipse.core.commands.AbstractHandler; import org.eclipse.core.commands.ExecutionEvent; -import org.eclipse.core.commands.ExecutionException; import org.eclipse.equinox.internal.p2.discovery.Catalog; import org.eclipse.equinox.internal.p2.discovery.DiscoveryCore; -import org.eclipse.equinox.internal.p2.discovery.compatibility.BundleDiscoveryStrategy; -import org.eclipse.equinox.internal.p2.discovery.compatibility.ConnectorDiscoveryExtensionReader; -import org.eclipse.equinox.internal.p2.discovery.compatibility.RemoteBundleDiscoveryStrategy; +import org.eclipse.equinox.internal.p2.discovery.compatibility.*; import org.eclipse.equinox.internal.p2.ui.discovery.util.WorkbenchUtil; import org.eclipse.equinox.internal.p2.ui.discovery.wizards.CatalogConfiguration; import org.eclipse.equinox.internal.p2.ui.discovery.wizards.DiscoveryWizard; @@ -39,7 +35,7 @@ public class ShowBundleCatalogCommandHandler extends AbstractHandler { private static final String ID_P2_INSTALL_UI = "org.eclipse.equinox.p2.ui.sdk/org.eclipse.equinox.p2.ui.sdk.install"; //$NON-NLS-1$ - public Object execute(ExecutionEvent event) throws ExecutionException { + public Object execute(ExecutionEvent event) { // check to make sure that the p2 install ui is enabled if (WorkbenchUtil.allowUseOf(ID_P2_INSTALL_UI)) { Catalog catalog = new Catalog(); @@ -67,9 +63,7 @@ public class ShowBundleCatalogCommandHandler extends AbstractHandler { WizardDialog dialog = new WizardDialog(WorkbenchUtil.getShell(), wizard); dialog.open(); } else { - MessageDialog.openWarning(WorkbenchUtil.getShell(), - Messages.ShowConnectorDiscoveryWizardCommandHandler_Install_Connectors, - Messages.ShowConnectorDiscoveryWizardCommandHandler_Unable_To_Install_No_P2); + MessageDialog.openWarning(WorkbenchUtil.getShell(), Messages.ShowConnectorDiscoveryWizardCommandHandler_Install_Connectors, Messages.ShowConnectorDiscoveryWizardCommandHandler_Unable_To_Install_No_P2); } return null; diff --git a/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/commands/ShowRepositoryCatalogCommandHandler.java b/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/commands/ShowRepositoryCatalogCommandHandler.java index 483e8daf7..304cadba4 100644 --- a/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/commands/ShowRepositoryCatalogCommandHandler.java +++ b/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/commands/ShowRepositoryCatalogCommandHandler.java @@ -12,10 +12,7 @@ package org.eclipse.equinox.internal.p2.ui.discovery.commands; import java.net.URI; import java.net.URISyntaxException; - -import org.eclipse.core.commands.AbstractHandler; -import org.eclipse.core.commands.ExecutionEvent; -import org.eclipse.core.commands.ExecutionException; +import org.eclipse.core.commands.*; import org.eclipse.equinox.internal.p2.discovery.Catalog; import org.eclipse.equinox.internal.p2.discovery.DiscoveryCore; import org.eclipse.equinox.internal.p2.ui.discovery.repository.RepositoryDiscoveryStrategy; @@ -46,7 +43,7 @@ public class ShowRepositoryCatalogCommandHandler extends AbstractHandler { try { strategy.addLocation(new URI(DEFAULT_REPOSITORY_URL)); } catch (URISyntaxException e) { - throw new ExecutionException("Invalid location format", e); + throw new ExecutionException("Invalid location format", e); //$NON-NLS-1$ } catalog.getDiscoveryStrategies().add(strategy); @@ -60,9 +57,7 @@ public class ShowRepositoryCatalogCommandHandler extends AbstractHandler { WizardDialog dialog = new WizardDialog(WorkbenchUtil.getShell(), wizard); dialog.open(); } else { - MessageDialog.openWarning(WorkbenchUtil.getShell(), - Messages.ShowConnectorDiscoveryWizardCommandHandler_Install_Connectors, - Messages.ShowConnectorDiscoveryWizardCommandHandler_Unable_To_Install_No_P2); + MessageDialog.openWarning(WorkbenchUtil.getShell(), Messages.ShowConnectorDiscoveryWizardCommandHandler_Install_Connectors, Messages.ShowConnectorDiscoveryWizardCommandHandler_Unable_To_Install_No_P2); } return null; diff --git a/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/operations/DiscoveryInstallOperation.java b/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/operations/DiscoveryInstallOperation.java index d4e89620d..94ada5f27 100644 --- a/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/operations/DiscoveryInstallOperation.java +++ b/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/operations/DiscoveryInstallOperation.java @@ -81,22 +81,19 @@ public class DiscoveryInstallOperation implements IRunnableWithProgress { public void run(IProgressMonitor progressMonitor) throws InvocationTargetException, InterruptedException { try { - SubMonitor monitor = SubMonitor.convert(progressMonitor, Messages.InstallConnectorsJob_task_configuring, - 100); + SubMonitor monitor = SubMonitor.convert(progressMonitor, Messages.InstallConnectorsJob_task_configuring, 100); try { final IInstallableUnit[] ius = computeInstallableUnits(monitor.newChild(50)); checkCancelled(monitor); - final InstallOperation installOperation = resolve(monitor.newChild(50), ius, - repositoryLocations.toArray(new URI[0])); + final InstallOperation installOperation = resolve(monitor.newChild(50), ius, repositoryLocations.toArray(new URI[0])); checkCancelled(monitor); Display.getDefault().asyncExec(new Runnable() { public void run() { - provisioningUI.openInstallWizard(WorkbenchUtil.getShell(), Arrays.asList(ius), - installOperation, null); + provisioningUI.openInstallWizard(WorkbenchUtil.getShell(), Arrays.asList(ius), installOperation, null); } }); } finally { @@ -115,11 +112,9 @@ public class DiscoveryInstallOperation implements IRunnableWithProgress { } } - private InstallOperation resolve(IProgressMonitor monitor, final IInstallableUnit[] ius, URI[] repositories) - throws CoreException { + private InstallOperation resolve(IProgressMonitor monitor, final IInstallableUnit[] ius, URI[] repositories) throws CoreException { final InstallOperation installOperation = provisioningUI.getInstallOperation(Arrays.asList(ius), repositories); - IStatus operationStatus = installOperation.resolveModal(new SubProgressMonitor(monitor, - installableConnectors.size())); + IStatus operationStatus = installOperation.resolveModal(new SubProgressMonitor(monitor, installableConnectors.size())); if (operationStatus.getSeverity() > IStatus.WARNING) { throw new CoreException(operationStatus); } @@ -136,36 +131,34 @@ public class DiscoveryInstallOperation implements IRunnableWithProgress { checkForUnavailable(installableUnits); return installableUnits.toArray(new IInstallableUnit[installableUnits.size()]); -// MultiStatus status = new MultiStatus(DiscoveryUi.ID_PLUGIN, 0, Messages.PrepareInstallProfileJob_ok, null); -// ius = installableUnits.toArray(new IInstallableUnit[installableUnits.size()]); -// ProfileChangeRequest profileChangeRequest = InstallAction.computeProfileChangeRequest(ius, profileId, -// status, new SubProgressMonitor(monitor, installableConnectors.size())); -// if (status.getSeverity() > IStatus.WARNING) { -// throw new CoreException(status); -// } -// if (profileChangeRequest == null) { -// // failed but no indication as to why -// throw new CoreException(new Status(IStatus.ERROR, DiscoveryUi.ID_PLUGIN, -// Messages.PrepareInstallProfileJob_computeProfileChangeRequestFailed, null)); -// } -// PlannerResolutionOperation operation = new PlannerResolutionOperation( -// Messages.PrepareInstallProfileJob_calculatingRequirements, profileId, profileChangeRequest, null, -// status, true); -// IStatus operationStatus = operation.execute(new SubProgressMonitor(monitor, installableConnectors.size())); -// if (operationStatus.getSeverity() > IStatus.WARNING) { -// throw new CoreException(operationStatus); -// } -// -// plannerResolutionOperation = operation; + // MultiStatus status = new MultiStatus(DiscoveryUi.ID_PLUGIN, 0, Messages.PrepareInstallProfileJob_ok, null); + // ius = installableUnits.toArray(new IInstallableUnit[installableUnits.size()]); + // ProfileChangeRequest profileChangeRequest = InstallAction.computeProfileChangeRequest(ius, profileId, + // status, new SubProgressMonitor(monitor, installableConnectors.size())); + // if (status.getSeverity() > IStatus.WARNING) { + // throw new CoreException(status); + // } + // if (profileChangeRequest == null) { + // // failed but no indication as to why + // throw new CoreException(new Status(IStatus.ERROR, DiscoveryUi.ID_PLUGIN, + // Messages.PrepareInstallProfileJob_computeProfileChangeRequestFailed, null)); + // } + // PlannerResolutionOperation operation = new PlannerResolutionOperation( + // Messages.PrepareInstallProfileJob_calculatingRequirements, profileId, profileChangeRequest, null, + // status, true); + // IStatus operationStatus = operation.execute(new SubProgressMonitor(monitor, installableConnectors.size())); + // if (operationStatus.getSeverity() > IStatus.WARNING) { + // throw new CoreException(operationStatus); + // } + // + // plannerResolutionOperation = operation; } catch (URISyntaxException e) { // should never happen, since we already validated URLs. - throw new CoreException(new Status(IStatus.ERROR, DiscoveryUi.ID_PLUGIN, - Messages.InstallConnectorsJob_unexpectedError_url, e)); + throw new CoreException(new Status(IStatus.ERROR, DiscoveryUi.ID_PLUGIN, Messages.InstallConnectorsJob_unexpectedError_url, e)); } catch (MalformedURLException e) { // should never happen, since we already validated URLs. - throw new CoreException(new Status(IStatus.ERROR, DiscoveryUi.ID_PLUGIN, - Messages.InstallConnectorsJob_unexpectedError_url, e)); + throw new CoreException(new Status(IStatus.ERROR, DiscoveryUi.ID_PLUGIN, Messages.InstallConnectorsJob_unexpectedError_url, e)); } finally { monitor.done(); } @@ -210,8 +203,7 @@ public class DiscoveryInstallOperation implements IRunnableWithProgress { if (detailedMessage.length() > 0) { detailedMessage += Messages.InstallConnectorsJob_commaSeparator; } - detailedMessage += NLS.bind(Messages.PrepareInstallProfileJob_notFoundDescriptorDetail, new Object[] { - descriptor.getName(), unavailableIds.toString(), descriptor.getSiteUrl() }); + detailedMessage += NLS.bind(Messages.PrepareInstallProfileJob_notFoundDescriptorDetail, new Object[] {descriptor.getName(), unavailableIds.toString(), descriptor.getSiteUrl()}); } } @@ -221,14 +213,11 @@ public class DiscoveryInstallOperation implements IRunnableWithProgress { final String finalMessage = message; Display.getDefault().syncExec(new Runnable() { public void run() { - okayToProceed[0] = MessageDialog.openQuestion(WorkbenchUtil.getShell(), - Messages.InstallConnectorsJob_questionProceed, NLS.bind( - Messages.InstallConnectorsJob_questionProceed_long, new Object[] { finalMessage })); + okayToProceed[0] = MessageDialog.openQuestion(WorkbenchUtil.getShell(), Messages.InstallConnectorsJob_questionProceed, NLS.bind(Messages.InstallConnectorsJob_questionProceed_long, new Object[] {finalMessage})); } }); if (!okayToProceed[0]) { - throw new CoreException(new Status(IStatus.ERROR, DiscoveryUi.ID_PLUGIN, NLS.bind( - Messages.InstallConnectorsJob_connectorsNotAvailable, detailedMessage), null)); + throw new CoreException(new Status(IStatus.ERROR, DiscoveryUi.ID_PLUGIN, NLS.bind(Messages.InstallConnectorsJob_connectorsNotAvailable, detailedMessage), null)); } } } @@ -262,8 +251,7 @@ public class DiscoveryInstallOperation implements IRunnableWithProgress { * unlikely that the same feature id is available from more than one of the selected repositories, and we must * ensure that the user gets the one that they asked for. */ - private List<IInstallableUnit> queryInstallableUnits(SubMonitor monitor, List<IMetadataRepository> repositories) - throws URISyntaxException { + private List<IInstallableUnit> queryInstallableUnits(SubMonitor monitor, List<IMetadataRepository> repositories) throws URISyntaxException { final List<IInstallableUnit> installableUnits = new ArrayList<IInstallableUnit>(); monitor.setWorkRemaining(repositories.size()); @@ -293,9 +281,7 @@ public class DiscoveryInstallOperation implements IRunnableWithProgress { } private boolean isQualifyingFeature(final Set<String> installableUnitIdsThisRepository, String id) { - return id.endsWith(P2_FEATURE_GROUP_SUFFIX) - && installableUnitIdsThisRepository.contains(id.substring(0, - id.indexOf(P2_FEATURE_GROUP_SUFFIX))); + return id.endsWith(P2_FEATURE_GROUP_SUFFIX) && installableUnitIdsThisRepository.contains(id.substring(0, id.indexOf(P2_FEATURE_GROUP_SUFFIX))); } }; IQueryResult<IInstallableUnit> result = repository.query(query, monitor.newChild(1)); @@ -304,8 +290,7 @@ public class DiscoveryInstallOperation implements IRunnableWithProgress { return installableUnits; } - private List<IMetadataRepository> addRepositories(SubMonitor monitor) throws MalformedURLException, - URISyntaxException, ProvisionException { + private List<IMetadataRepository> addRepositories(SubMonitor monitor) throws MalformedURLException, URISyntaxException, ProvisionException { // tell p2 that it's okay to use these repositories ProvisioningSession session = ProvisioningUI.getDefaultUI().getSession(); RepositoryTracker repositoryTracker = ProvisioningUI.getDefaultUI().getRepositoryTracker(); @@ -316,9 +301,9 @@ public class DiscoveryInstallOperation implements IRunnableWithProgress { if (repositoryLocations.add(uri)) { checkCancelled(monitor); repositoryTracker.addRepository(uri, null, session); -// ProvisioningUtil.addMetaDataRepository(url.toURI(), true); -// ProvisioningUtil.addArtifactRepository(url.toURI(), true); -// ProvisioningUtil.setColocatedRepositoryEnablement(url.toURI(), true); + // ProvisioningUtil.addMetaDataRepository(url.toURI(), true); + // ProvisioningUtil.addArtifactRepository(url.toURI(), true); + // ProvisioningUtil.setColocatedRepositoryEnablement(url.toURI(), true); } monitor.worked(1); } @@ -328,8 +313,7 @@ public class DiscoveryInstallOperation implements IRunnableWithProgress { monitor.setWorkRemaining(repositories.size()); for (URI uri : repositoryLocations) { checkCancelled(monitor); - IMetadataRepository repository = session.getMetadataRepositoryManager().loadRepository(uri, - monitor.newChild(1)); + IMetadataRepository repository = session.getMetadataRepositoryManager().loadRepository(uri, monitor.newChild(1)); repositories.add(repository); } return repositories; diff --git a/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/repository/RepositoryDiscoveryStrategy.java b/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/repository/RepositoryDiscoveryStrategy.java index a2ca1e523..474cf0c64 100644 --- a/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/repository/RepositoryDiscoveryStrategy.java +++ b/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/repository/RepositoryDiscoveryStrategy.java @@ -12,18 +12,8 @@ package org.eclipse.equinox.internal.p2.ui.discovery.repository; import java.net.URI; -import java.util.ArrayList; -import java.util.Collection; -import java.util.HashMap; -import java.util.List; -import java.util.Map; - -import org.eclipse.core.runtime.CoreException; -import org.eclipse.core.runtime.IProgressMonitor; -import org.eclipse.core.runtime.IStatus; -import org.eclipse.core.runtime.OperationCanceledException; -import org.eclipse.core.runtime.Status; -import org.eclipse.core.runtime.SubMonitor; +import java.util.*; +import org.eclipse.core.runtime.*; import org.eclipse.equinox.internal.p2.discovery.AbstractCatalogSource; import org.eclipse.equinox.internal.p2.discovery.AbstractDiscoveryStrategy; import org.eclipse.equinox.internal.p2.discovery.model.CatalogCategory; @@ -31,9 +21,7 @@ import org.eclipse.equinox.internal.p2.discovery.model.CatalogItem; import org.eclipse.equinox.internal.p2.metadata.IRequiredCapability; import org.eclipse.equinox.internal.p2.metadata.TranslationSupport; import org.eclipse.equinox.p2.core.ProvisionException; -import org.eclipse.equinox.p2.metadata.IInstallableUnit; -import org.eclipse.equinox.p2.metadata.IProvidedCapability; -import org.eclipse.equinox.p2.metadata.IRequirement; +import org.eclipse.equinox.p2.metadata.*; import org.eclipse.equinox.p2.operations.ProvisioningSession; import org.eclipse.equinox.p2.operations.RepositoryTracker; import org.eclipse.equinox.p2.query.IQuery; @@ -85,7 +73,7 @@ public class RepositoryDiscoveryStrategy extends AbstractDiscoveryStrategy { queryInstallableUnits(monitor.newChild(50), repositories); connectCategories(); } catch (ProvisionException e) { - throw new CoreException(new Status(IStatus.ERROR, PLUGIN_ID, "Failed to process repository contents", e)); + throw new CoreException(new Status(IStatus.ERROR, PLUGIN_ID, "Failed to process repository contents", e)); //$NON-NLS-1$ } } @@ -122,8 +110,7 @@ public class RepositoryDiscoveryStrategy extends AbstractDiscoveryStrategy { // fetch meta-data for these repositories ArrayList<IMetadataRepository> repositories = new ArrayList<IMetadataRepository>(); for (URI uri : locations) { - IMetadataRepository repository = session.getMetadataRepositoryManager().loadRepository(uri, - monitor.newChild(1)); + IMetadataRepository repository = session.getMetadataRepositoryManager().loadRepository(uri, monitor.newChild(1)); repositories.add(repository); } return repositories; diff --git a/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/util/ControlListItem.java b/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/util/ControlListItem.java index 86874122c..c01f19b74 100644 --- a/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/util/ControlListItem.java +++ b/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/util/ControlListItem.java @@ -76,8 +76,7 @@ public abstract class ControlListItem<T> extends Composite { Color lightColor = PlatformUI.getWorkbench().getDisplay().getSystemColor(SWT.COLOR_LIST_BACKGROUND); // Determine a dark color by shifting the list color - RGB darkRGB = new RGB(Math.max(0, lightColor.getRed() + shift), Math.max(0, lightColor.getGreen() + shift), - Math.max(0, lightColor.getBlue() + shift)); + RGB darkRGB = new RGB(Math.max(0, lightColor.getRed() + shift), Math.max(0, lightColor.getGreen() + shift), Math.max(0, lightColor.getBlue() + shift)); JFaceResources.getColorRegistry().put(DARK_COLOR_KEY, darkRGB); } @@ -96,10 +95,10 @@ public abstract class ControlListItem<T> extends Composite { mouseListener = doCreateMouseListener(); mouseTrackListener = doCreateMouseTrackListener(); registerChild(this); -// Control[] children = getChildren(); -// for (Control child : children) { -// registerChild(child); -// } + // Control[] children = getChildren(); + // for (Control child : children) { + // registerChild(child); + // } setHot(false); } diff --git a/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/util/ControlListViewer.java b/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/util/ControlListViewer.java index 025740cbb..477a69ebb 100644 --- a/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/util/ControlListViewer.java +++ b/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/util/ControlListViewer.java @@ -122,62 +122,62 @@ public abstract class ControlListViewer extends StructuredViewer { return; } switch (event.detail) { - case SWT.TRAVERSE_ARROW_PREVIOUS: { - Control[] children = control.getChildren(); - if (children.length > 0) { - boolean selected = false; - for (int i = 0; i < children.length; i++) { - ControlListItem item = (ControlListItem) children[i]; - if (item.isSelected()) { - selected = true; - if (i > 0) { - setSelection(new StructuredSelection(children[i - 1].getData()), true); + case SWT.TRAVERSE_ARROW_PREVIOUS : { + Control[] children = control.getChildren(); + if (children.length > 0) { + boolean selected = false; + for (int i = 0; i < children.length; i++) { + ControlListItem item = (ControlListItem) children[i]; + if (item.isSelected()) { + selected = true; + if (i > 0) { + setSelection(new StructuredSelection(children[i - 1].getData()), true); + } + break; } - break; + } + if (!selected) { + setSelection(new StructuredSelection(children[children.length - 1].getData()), true); } } - if (!selected) { - setSelection(new StructuredSelection(children[children.length - 1].getData()), true); - } + break; } - break; - } - case SWT.TRAVERSE_ARROW_NEXT: { - Control[] children = control.getChildren(); - if (children.length > 0) { - boolean selected = false; - for (int i = 0; i < children.length; i++) { - ControlListItem item = (ControlListItem) children[i]; - if (item.isSelected()) { - selected = true; - if (i < children.length - 1) { - setSelection(new StructuredSelection(children[i + 1].getData()), true); + case SWT.TRAVERSE_ARROW_NEXT : { + Control[] children = control.getChildren(); + if (children.length > 0) { + boolean selected = false; + for (int i = 0; i < children.length; i++) { + ControlListItem item = (ControlListItem) children[i]; + if (item.isSelected()) { + selected = true; + if (i < children.length - 1) { + setSelection(new StructuredSelection(children[i + 1].getData()), true); + } + break; } - break; + } + if (!selected) { + setSelection(new StructuredSelection(children[0].getData()), true); } } - if (!selected) { - setSelection(new StructuredSelection(children[0].getData()), true); - } + break; } - break; - } - default: - handleEvent = false; - event.doit = true; - Control control = ControlListViewer.this.control; - Shell shell = control.getShell(); - while (control != null) { - if (control.traverse(event.detail)) { - break; - } - if (!event.doit || control == shell) { - break; + default : + handleEvent = false; + event.doit = true; + Control control = ControlListViewer.this.control; + Shell shell = control.getShell(); + while (control != null) { + if (control.traverse(event.detail)) { + break; + } + if (!event.doit || control == shell) { + break; + } + control = control.getParent(); } - control = control.getParent(); - } - handleEvent = true; - break; + handleEvent = true; + break; } } }); @@ -254,16 +254,16 @@ public abstract class ControlListViewer extends StructuredViewer { */ private ControlListItem createNewItem(Object element) { final ControlListItem item = doCreateItem(control, element); -// item.getChildren()[0].addPaintListener(new PaintListener() { -// public void paintControl(PaintEvent e) { -// if (hasFocus && item.isSelected()) { -// Point size = item.getSize(); -// e.gc.setForeground(e.gc.getDevice().getSystemColor(SWT.COLOR_DARK_GRAY)); -// e.gc.setLineDash(new int[] { 1, 2 }); -// e.gc.drawRoundRectangle(0, 0, size.x - 1, size.y - 1, 5, 5); -// } -// } -// }); + // item.getChildren()[0].addPaintListener(new PaintListener() { + // public void paintControl(PaintEvent e) { + // if (hasFocus && item.isSelected()) { + // Point size = item.getSize(); + // e.gc.setForeground(e.gc.getDevice().getSystemColor(SWT.COLOR_DARK_GRAY)); + // e.gc.setLineDash(new int[] { 1, 2 }); + // e.gc.drawRoundRectangle(0, 0, size.x - 1, size.y - 1, 5, 5); + // } + // } + // }); item.setIndexListener(new ControlListItem.IndexListener() { public void selectNext() { Control[] children = control.getChildren(); @@ -331,7 +331,7 @@ public abstract class ControlListViewer extends StructuredViewer { unmapElement(item); } item.dispose(); - add(new Object[] { element }); + add(new Object[] {element}); } @Override @@ -379,7 +379,7 @@ public abstract class ControlListViewer extends StructuredViewer { } Widget widget = findItem(element); if (widget == null) { - add(new Object[] { element }); + add(new Object[] {element}); return; } ((ControlListItem) widget).refresh(); @@ -426,7 +426,7 @@ public abstract class ControlListViewer extends StructuredViewer { } } - @SuppressWarnings({ "unchecked", "rawtypes" }) + @SuppressWarnings({"unchecked", "rawtypes"}) @Override protected void setSelectionToWidget(List list, boolean reveal) { if (list != null) { @@ -461,14 +461,14 @@ public abstract class ControlListViewer extends StructuredViewer { Control[] children = control.getChildren(); if (children.length > 0) { // causes the item's tool bar to get focus when clicked which is undesirable -// for (Control element : children) { -// ControlListItem item = (ControlListItem) element; -// if (item.isSelected()) { -// if (item.setFocus()) { -// return; -// } -// } -// } + // for (Control element : children) { + // ControlListItem item = (ControlListItem) element; + // if (item.isSelected()) { + // if (item.setFocus()) { + // return; + // } + // } + // } control.forceFocus(); } else { noEntryArea.setFocus(); diff --git a/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/util/FilteredViewer.java b/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/util/FilteredViewer.java index 46494727d..3cd3e4f62 100644 --- a/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/util/FilteredViewer.java +++ b/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/util/FilteredViewer.java @@ -13,9 +13,7 @@ *******************************************************************************/ package org.eclipse.equinox.internal.p2.ui.discovery.util; -import org.eclipse.core.runtime.IProgressMonitor; -import org.eclipse.core.runtime.IStatus; -import org.eclipse.core.runtime.Status; +import org.eclipse.core.runtime.*; import org.eclipse.equinox.internal.p2.ui.discovery.DiscoveryImages; import org.eclipse.equinox.internal.p2.ui.discovery.wizards.Messages; import org.eclipse.jface.layout.GridDataFactory; @@ -23,28 +21,12 @@ import org.eclipse.jface.layout.GridLayoutFactory; import org.eclipse.jface.viewers.StructuredViewer; import org.eclipse.jface.viewers.Viewer; import org.eclipse.swt.SWT; -import org.eclipse.swt.accessibility.ACC; -import org.eclipse.swt.accessibility.AccessibleAdapter; -import org.eclipse.swt.accessibility.AccessibleControlAdapter; -import org.eclipse.swt.accessibility.AccessibleControlEvent; -import org.eclipse.swt.accessibility.AccessibleEvent; -import org.eclipse.swt.events.DisposeEvent; -import org.eclipse.swt.events.DisposeListener; -import org.eclipse.swt.events.ModifyEvent; -import org.eclipse.swt.events.ModifyListener; -import org.eclipse.swt.events.MouseAdapter; -import org.eclipse.swt.events.MouseEvent; -import org.eclipse.swt.events.MouseMoveListener; -import org.eclipse.swt.events.MouseTrackListener; -import org.eclipse.swt.events.SelectionAdapter; -import org.eclipse.swt.events.SelectionEvent; +import org.eclipse.swt.accessibility.*; +import org.eclipse.swt.events.*; import org.eclipse.swt.graphics.Image; import org.eclipse.swt.graphics.Point; import org.eclipse.swt.layout.GridData; -import org.eclipse.swt.widgets.Composite; -import org.eclipse.swt.widgets.Control; -import org.eclipse.swt.widgets.Label; -import org.eclipse.swt.widgets.Text; +import org.eclipse.swt.widgets.*; import org.eclipse.ui.progress.WorkbenchJob; /** @@ -92,6 +74,7 @@ public abstract class FilteredViewer { protected StructuredViewer viewer; public FilteredViewer() { + // constructor } private void clearFilterText() { @@ -326,8 +309,7 @@ public abstract class FilteredViewer { public void setMinimumHeight(int minimumHeight) { this.minimumHeight = minimumHeight; if (viewer != null) { - GridDataFactory.fillDefaults().grab(true, true).hint(SWT.DEFAULT, minimumHeight).applyTo( - viewer.getControl()); + GridDataFactory.fillDefaults().grab(true, true).hint(SWT.DEFAULT, minimumHeight).applyTo(viewer.getControl()); } } diff --git a/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/util/GradientCanvas.java b/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/util/GradientCanvas.java index 2f06f33e7..a7fe65646 100644 --- a/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/util/GradientCanvas.java +++ b/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/util/GradientCanvas.java @@ -13,16 +13,9 @@ package org.eclipse.equinox.internal.p2.ui.discovery.util; import java.util.Hashtable; import java.util.Map; - import org.eclipse.swt.SWT; -import org.eclipse.swt.graphics.Color; -import org.eclipse.swt.graphics.GC; -import org.eclipse.swt.graphics.Image; -import org.eclipse.swt.graphics.Rectangle; -import org.eclipse.swt.widgets.Canvas; -import org.eclipse.swt.widgets.Composite; -import org.eclipse.swt.widgets.Event; -import org.eclipse.swt.widgets.Listener; +import org.eclipse.swt.graphics.*; +import org.eclipse.swt.widgets.*; /** * Based on {@link org.eclipse.ui.internal.forms.widgets.FormHeading}. @@ -232,8 +225,8 @@ public class GradientCanvas extends Canvas { } if (gradientInfo != null) { // TODO e3.4 use FormImages -// gradientImage = FormImages.getInstance().getGradient(gradientInfo.gradientColors, gradientInfo.percents, -// gradientInfo.vertical ? rect.height : rect.width, gradientInfo.vertical, getColor(COLOR_BASE_BG)); + // gradientImage = FormImages.getInstance().getGradient(gradientInfo.gradientColors, gradientInfo.percents, + // gradientInfo.vertical ? rect.height : rect.width, gradientInfo.vertical, getColor(COLOR_BASE_BG)); boolean vertical = gradientInfo.vertical; int width = vertical ? 1 : rect.width; int height = vertical ? rect.height : 1; diff --git a/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/util/GradientToolTip.java b/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/util/GradientToolTip.java index db3025698..be5b105c5 100644 --- a/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/util/GradientToolTip.java +++ b/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/util/GradientToolTip.java @@ -54,8 +54,7 @@ public abstract class GradientToolTip extends ToolTip { headLayout.numColumns = 1; gradient.setLayout(headLayout); - gradient.setBackgroundGradient(new Color[] { colors.getGradientBegin(), colors.getGradientEnd() }, - new int[] { 100 }, true); + gradient.setBackgroundGradient(new Color[] {colors.getGradientBegin(), colors.getGradientEnd()}, new int[] {100}, true); createToolTipArea(event, gradient); diff --git a/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/util/WorkbenchUtil.java b/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/util/WorkbenchUtil.java index 120241f69..2f426b28f 100644 --- a/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/util/WorkbenchUtil.java +++ b/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/util/WorkbenchUtil.java @@ -127,8 +127,7 @@ public class WorkbenchUtil { IPluginContribution contribution = (IPluginContribution) object; if (contribution.getPluginId() != null) { IWorkbenchActivitySupport workbenchActivitySupport = PlatformUI.getWorkbench().getActivitySupport(); - IIdentifier identifier = workbenchActivitySupport.getActivityManager().getIdentifier( - createUnifiedId(contribution)); + IIdentifier identifier = workbenchActivitySupport.getActivityManager().getIdentifier(createUnifiedId(contribution)); return identifier.isEnabled(); } } @@ -173,25 +172,21 @@ public class WorkbenchUtil { if (location != null) { url = new URL(location); } - if (WebBrowserPreference.getBrowserChoice() == WebBrowserPreference.EXTERNAL - || (customFlags & IWorkbenchBrowserSupport.AS_EXTERNAL) != 0) { + if (WebBrowserPreference.getBrowserChoice() == WebBrowserPreference.EXTERNAL || (customFlags & IWorkbenchBrowserSupport.AS_EXTERNAL) != 0) { try { IWorkbenchBrowserSupport support = PlatformUI.getWorkbench().getBrowserSupport(); support.getExternalBrowser().openURL(url); } catch (PartInitException e) { - Status status = new Status(IStatus.ERROR, DiscoveryUi.ID_PLUGIN, - Messages.WorkbenchUtil_Browser_Initialization_Failed); + Status status = new Status(IStatus.ERROR, DiscoveryUi.ID_PLUGIN, Messages.WorkbenchUtil_Browser_Initialization_Failed); MessageDialog.openError(getShell(), Messages.WorkbenchUtil_Open_Location_Title, status.getMessage()); } } else { IWebBrowser browser = null; int flags = customFlags; if (WorkbenchBrowserSupport.getInstance().isInternalWebBrowserAvailable()) { - flags |= IWorkbenchBrowserSupport.AS_EDITOR | IWorkbenchBrowserSupport.LOCATION_BAR - | IWorkbenchBrowserSupport.NAVIGATION_BAR; + flags |= IWorkbenchBrowserSupport.AS_EDITOR | IWorkbenchBrowserSupport.LOCATION_BAR | IWorkbenchBrowserSupport.NAVIGATION_BAR; } else { - flags |= IWorkbenchBrowserSupport.AS_EXTERNAL | IWorkbenchBrowserSupport.LOCATION_BAR - | IWorkbenchBrowserSupport.NAVIGATION_BAR; + flags |= IWorkbenchBrowserSupport.AS_EXTERNAL | IWorkbenchBrowserSupport.LOCATION_BAR | IWorkbenchBrowserSupport.NAVIGATION_BAR; } String generatedId = "org.eclipse.mylyn.web.browser-" + Calendar.getInstance().getTimeInMillis(); //$NON-NLS-1$ @@ -199,18 +194,15 @@ public class WorkbenchUtil { browser.openURL(url); } } catch (PartInitException e) { - Status status = new Status(IStatus.ERROR, DiscoveryUi.ID_PLUGIN, - Messages.WorkbenchUtil_Browser_Initialization_Failed, e); + Status status = new Status(IStatus.ERROR, DiscoveryUi.ID_PLUGIN, Messages.WorkbenchUtil_Browser_Initialization_Failed, e); LogHelper.log(status); MessageDialog.openError(getShell(), Messages.WorkbenchUtil_Open_Location_Title, status.getMessage()); } catch (MalformedURLException e) { if (location != null && location.trim().equals("")) { //$NON-NLS-1$ - Status status = new Status(IStatus.WARNING, DiscoveryUi.ID_PLUGIN, Messages.WorkbenchUtil_No_URL_Error, - e); + Status status = new Status(IStatus.WARNING, DiscoveryUi.ID_PLUGIN, Messages.WorkbenchUtil_No_URL_Error, e); MessageDialog.openWarning(getShell(), Messages.WorkbenchUtil_Open_Location_Title, status.getMessage()); } else { - Status status = new Status(IStatus.ERROR, DiscoveryUi.ID_PLUGIN, NLS.bind( - Messages.WorkbenchUtil_Invalid_URL_Error, location), e); + Status status = new Status(IStatus.ERROR, DiscoveryUi.ID_PLUGIN, NLS.bind(Messages.WorkbenchUtil_Invalid_URL_Error, location), e); MessageDialog.openError(getShell(), Messages.WorkbenchUtil_Open_Location_Title, status.getMessage()); } } diff --git a/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/wizards/AbstractDiscoveryItem.java b/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/wizards/AbstractDiscoveryItem.java index aa1cf7c76..8ca1c4433 100644 --- a/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/wizards/AbstractDiscoveryItem.java +++ b/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/wizards/AbstractDiscoveryItem.java @@ -51,19 +51,18 @@ public abstract class AbstractDiscoveryItem<T> extends ControlListItem<T> { } } - protected void hookTooltip(final Control parent, final Widget tipActivator, final Control exitControl, - final Control titleControl, AbstractCatalogSource source, Overview overview, Image image) { + protected void hookTooltip(final Control parent, final Widget tipActivator, final Control exitControl, final Control titleControl, AbstractCatalogSource source, Overview overview, Image image) { final OverviewToolTip toolTip = new OverviewToolTip(parent, source, overview, image); Listener listener = new Listener() { public void handleEvent(Event event) { switch (event.type) { - case SWT.MouseHover: - toolTip.show(titleControl); - break; - case SWT.Dispose: - case SWT.MouseWheel: - toolTip.hide(); - break; + case SWT.MouseHover : + toolTip.show(titleControl); + break; + case SWT.Dispose : + case SWT.MouseWheel : + toolTip.hide(); + break; } } @@ -82,23 +81,23 @@ public abstract class AbstractDiscoveryItem<T> extends ControlListItem<T> { Listener exitListener = new Listener() { public void handleEvent(Event event) { switch (event.type) { - case SWT.MouseWheel: - toolTip.hide(); - break; - case SWT.MouseExit: - /* - * Check if the mouse exit happened because we move over the - * tooltip - */ - Rectangle containerBounds = exitControl.getBounds(); - Point displayLocation = exitControl.getParent().toDisplay(containerBounds.x, containerBounds.y); - containerBounds.x = displayLocation.x; - containerBounds.y = displayLocation.y; - if (containerBounds.contains(Display.getCurrent().getCursorLocation())) { + case SWT.MouseWheel : + toolTip.hide(); + break; + case SWT.MouseExit : + /* + * Check if the mouse exit happened because we move over the + * tooltip + */ + Rectangle containerBounds = exitControl.getBounds(); + Point displayLocation = exitControl.getParent().toDisplay(containerBounds.x, containerBounds.y); + containerBounds.x = displayLocation.x; + containerBounds.y = displayLocation.y; + if (containerBounds.contains(Display.getCurrent().getCursorLocation())) { + break; + } + toolTip.hide(); break; - } - toolTip.hide(); - break; } } }; diff --git a/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/wizards/CatalogViewer.java b/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/wizards/CatalogViewer.java index 6197c3c79..2f70224b1 100644 --- a/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/wizards/CatalogViewer.java +++ b/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/wizards/CatalogViewer.java @@ -108,8 +108,7 @@ public class CatalogViewer extends FilteredViewer { protected boolean isLeafMatch(Viewer viewer, Object element) { if (element instanceof CatalogItem) { CatalogItem descriptor = (CatalogItem) element; - if (!(filterMatches(descriptor.getName()) || filterMatches(descriptor.getDescription()) - || filterMatches(descriptor.getProvider()) || filterMatches(descriptor.getLicense()))) { + if (!(filterMatches(descriptor.getName()) || filterMatches(descriptor.getDescription()) || filterMatches(descriptor.getProvider()) || filterMatches(descriptor.getLicense()))) { return false; } return true; @@ -119,15 +118,15 @@ public class CatalogViewer extends FilteredViewer { } -// private class ConnectorBorderPaintListener implements PaintListener { -// public void paintControl(PaintEvent e) { -// Composite composite = (Composite) e.widget; -// Rectangle bounds = composite.getBounds(); -// GC gc = e.gc; -// gc.setLineStyle(SWT.LINE_DOT); -// gc.drawLine(bounds.x, bounds.y, bounds.x + bounds.width, bounds.y); -// } -// } + // private class ConnectorBorderPaintListener implements PaintListener { + // public void paintControl(PaintEvent e) { + // Composite composite = (Composite) e.widget; + // Rectangle bounds = composite.getBounds(); + // GC gc = e.gc; + // gc.setLineStyle(SWT.LINE_DOT); + // gc.drawLine(bounds.x, bounds.y, bounds.x + bounds.width, bounds.y); + // } + // } private static final int DEFAULT_HEIGHT = 250; @@ -157,8 +156,7 @@ public class CatalogViewer extends FilteredViewer { private Set<Tag> visibleTags; - public CatalogViewer(Catalog catalog, IShellProvider shellProvider, IRunnableContext context, - CatalogConfiguration configuration) { + public CatalogViewer(Catalog catalog, IShellProvider shellProvider, IRunnableContext context, CatalogConfiguration configuration) { Assert.isNotNull(catalog); Assert.isNotNull(shellProvider); Assert.isNotNull(context); @@ -190,8 +188,7 @@ public class CatalogViewer extends FilteredViewer { } if (categoryWithConnectorCount == 0) { // nothing was discovered: notify the user - MessageDialog.openWarning(getShell(), Messages.ConnectorDiscoveryWizardMainPage_noConnectorsFound, - Messages.ConnectorDiscoveryWizardMainPage_noConnectorsFound_description); + MessageDialog.openWarning(getShell(), Messages.ConnectorDiscoveryWizardMainPage_noConnectorsFound, Messages.ConnectorDiscoveryWizardMainPage_noConnectorsFound_description); } } viewer.setInput(catalog); @@ -207,10 +204,9 @@ public class CatalogViewer extends FilteredViewer { statusCause = new Status(IStatus.ERROR, DiscoveryUi.ID_PLUGIN, cause.getMessage(), cause); } if (statusCause.getMessage() != null) { - message = NLS.bind(Messages.ConnectorDiscoveryWizardMainPage_message_with_cause, message, - statusCause.getMessage()); + message = NLS.bind(Messages.ConnectorDiscoveryWizardMainPage_message_with_cause, message, statusCause.getMessage()); } - IStatus status = new MultiStatus(DiscoveryUi.ID_PLUGIN, 0, new IStatus[] { statusCause }, message, cause); + IStatus status = new MultiStatus(DiscoveryUi.ID_PLUGIN, 0, new IStatus[] {statusCause}, message, cause); return status; } @@ -368,14 +364,14 @@ public class CatalogViewer extends FilteredViewer { return i; } -// private int compare(Comparator<Object> comparator, Object key1, Object key2) { -// if (key1 == null) { -// return (key2 != null) ? 1 : 0; -// } else if (key2 == null) { -// return -1; -// } -// return comparator.compare(key1, key2); -// } + // private int compare(Comparator<Object> comparator, Object key1, Object key2) { + // if (key1 == null) { + // return (key2 != null) ? 1 : 0; + // } else if (key2 == null) { + // return -1; + // } + // return comparator.compare(key1, key2); + // } private CatalogCategory getCategory(Object o) { if (o instanceof CatalogCategory) { return (CatalogCategory) o; @@ -397,7 +393,7 @@ public class CatalogViewer extends FilteredViewer { return viewer; } - @SuppressWarnings({ "rawtypes", "unchecked" }) + @SuppressWarnings({"rawtypes", "unchecked"}) protected ControlListItem<?> doCreateViewerItem(Composite parent, Object element) { if (element instanceof CatalogItem) { return new DiscoveryItem(parent, SWT.NONE, resources, shellProvider, (CatalogItem) element, this); @@ -498,8 +494,7 @@ public class CatalogViewer extends FilteredViewer { protected void postDiscovery() { for (CatalogItem connector : catalog.getItems()) { - connector.setInstalled(installedFeatures != null - && installedFeatures.containsAll(connector.getInstallableUnits())); + connector.setInstalled(installedFeatures != null && installedFeatures.containsAll(connector.getInstallableUnits())); } } @@ -556,8 +551,7 @@ public class CatalogViewer extends FilteredViewer { }); if (result[0] != null && !result[0].isOK()) { - StatusManager.getManager().handle(result[0], - StatusManager.SHOW | StatusManager.BLOCK | StatusManager.LOG); + StatusManager.getManager().handle(result[0], StatusManager.SHOW | StatusManager.BLOCK | StatusManager.LOG); } } catch (InvocationTargetException e) { IStatus status = computeStatus(e, Messages.ConnectorDiscoveryWizardMainPage_unexpectedException); @@ -571,15 +565,13 @@ public class CatalogViewer extends FilteredViewer { if (configuration.isVerifyUpdateSiteAvailability() && !catalog.getItems().isEmpty()) { try { context.run(true, true, new IRunnableWithProgress() { - public void run(IProgressMonitor monitor) throws InvocationTargetException, - InterruptedException { + public void run(IProgressMonitor monitor) throws InvocationTargetException, InterruptedException { //discovery.verifySiteAvailability(monitor); } }); } catch (InvocationTargetException e) { IStatus status = computeStatus(e, Messages.ConnectorDiscoveryWizardMainPage_unexpectedException); - StatusManager.getManager().handle(status, - StatusManager.SHOW | StatusManager.BLOCK | StatusManager.LOG); + StatusManager.getManager().handle(status, StatusManager.SHOW | StatusManager.BLOCK | StatusManager.LOG); } catch (InterruptedException e) { // cancelled by user so nothing to do here. wasCancelled = true; diff --git a/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/wizards/CategoryItem.java b/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/wizards/CategoryItem.java index 27ca14cf7..2c73aaabc 100644 --- a/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/wizards/CategoryItem.java +++ b/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/wizards/CategoryItem.java @@ -34,8 +34,7 @@ public class CategoryItem<T extends CatalogCategory> extends AbstractDiscoveryIt } protected boolean hasTooltip(final CatalogCategory category) { - return category.getOverview() != null && category.getOverview().getSummary() != null - && category.getOverview().getSummary().length() > 0; + return category.getOverview() != null && category.getOverview().getSummary() != null && category.getOverview().getSummary().length() > 0; } private void createContent(T category) { @@ -44,8 +43,7 @@ public class CategoryItem<T extends CatalogCategory> extends AbstractDiscoveryIt final GradientCanvas categoryHeaderContainer = new GradientCanvas(this, SWT.NONE); categoryHeaderContainer.setSeparatorVisible(true); categoryHeaderContainer.setSeparatorAlignment(SWT.TOP); - categoryHeaderContainer.setBackgroundGradient(new Color[] { resources.getGradientBeginColor(), - resources.getGradientEndColor() }, new int[] { 100 }, true); + categoryHeaderContainer.setBackgroundGradient(new Color[] {resources.getGradientBeginColor(), resources.getGradientEndColor()}, new int[] {100}, true); categoryHeaderContainer.putColor(GradientCanvas.H_BOTTOM_KEYLINE1, resources.getGradientBeginColor()); categoryHeaderContainer.putColor(GradientCanvas.H_BOTTOM_KEYLINE2, resources.getGradientEndColor()); @@ -71,8 +69,7 @@ public class CategoryItem<T extends CatalogCategory> extends AbstractDiscoveryIt ToolItem infoButton = new ToolItem(toolBar, SWT.PUSH); infoButton.setImage(resources.getInfoImage()); infoButton.setToolTipText(Messages.ConnectorDiscoveryWizardMainPage_tooltip_showOverview); - hookTooltip(toolBar, infoButton, categoryHeaderContainer, nameLabel, category.getSource(), - category.getOverview(), null); + hookTooltip(toolBar, infoButton, categoryHeaderContainer, nameLabel, category.getSource(), category.getOverview(), null); GridDataFactory.fillDefaults().align(SWT.END, SWT.CENTER).applyTo(toolBar); } else { new Label(categoryHeaderContainer, SWT.NULL).setText(" "); //$NON-NLS-1$ diff --git a/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/wizards/DiscoveryItem.java b/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/wizards/DiscoveryItem.java index d27fb3658..4ffeb7b3d 100644 --- a/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/wizards/DiscoveryItem.java +++ b/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/wizards/DiscoveryItem.java @@ -69,8 +69,7 @@ public class DiscoveryItem<T extends CatalogItem> extends AbstractDiscoveryItem< private final CatalogViewer viewer; - public DiscoveryItem(Composite parent, int style, DiscoveryResources resources, IShellProvider shellProvider, - final T connector, CatalogViewer viewer) { + public DiscoveryItem(Composite parent, int style, DiscoveryResources resources, IShellProvider shellProvider, final T connector, CatalogViewer viewer) { super(parent, style, resources, connector); this.shellProvider = shellProvider; this.connector = connector; @@ -102,12 +101,12 @@ public class DiscoveryItem<T extends CatalogItem> extends AbstractDiscoveryItem< // help UI tests checkbox.setData("connectorId", connector.getId()); //$NON-NLS-1$ // FIXME -// checkbox.addFocusListener(new FocusAdapter() { -// @Override -// public void focusGained(FocusEvent e) { -// bodyScrolledComposite.showControl(this); -// } -// }); + // checkbox.addFocusListener(new FocusAdapter() { + // @Override + // public void focusGained(FocusEvent e) { + // bodyScrolledComposite.showControl(this); + // } + // }); GridDataFactory.swtDefaults().align(SWT.CENTER, SWT.CENTER).applyTo(checkbox); iconLabel = new Label(checkboxContainer, SWT.NONE); @@ -126,26 +125,22 @@ public class DiscoveryItem<T extends CatalogItem> extends AbstractDiscoveryItem< // always disabled color to make it less prominent providerLabel.setForeground(resources.getColorDisabled()); if (connector.getCertification() != null) { - providerLabel.setText(NLS.bind(Messages.DiscoveryViewer_Certification_Label0, new String[] { - connector.getProvider(), connector.getLicense(), connector.getCertification().getName() })); + providerLabel.setText(NLS.bind(Messages.DiscoveryViewer_Certification_Label0, new String[] {connector.getProvider(), connector.getLicense(), connector.getCertification().getName()})); if (connector.getCertification().getUrl() != null) { providerLabel.addSelectionListener(new SelectionAdapter() { @Override public void widgetSelected(SelectionEvent e) { - WorkbenchUtil.openUrl(connector.getCertification().getUrl(), - IWorkbenchBrowserSupport.AS_EXTERNAL); + WorkbenchUtil.openUrl(connector.getCertification().getUrl(), IWorkbenchBrowserSupport.AS_EXTERNAL); } }); } Overview overview = new Overview(); overview.setSummary(connector.getCertification().getDescription()); overview.setUrl(connector.getCertification().getUrl()); - Image image = resources.getIconImage(connector.getSource(), connector.getCertification().getIcon(), 48, - true); + Image image = resources.getIconImage(connector.getSource(), connector.getCertification().getIcon(), 48, true); hookTooltip(providerLabel, providerLabel, this, providerLabel, connector.getSource(), overview, image); } else { - providerLabel.setText(NLS.bind(Messages.ConnectorDiscoveryWizardMainPage_provider_and_license, - connector.getProvider(), connector.getLicense())); + providerLabel.setText(NLS.bind(Messages.ConnectorDiscoveryWizardMainPage_provider_and_license, connector.getProvider(), connector.getLicense())); } if (hasTooltip(connector) || connector.isInstalled()) { @@ -183,8 +178,7 @@ public class DiscoveryItem<T extends CatalogItem> extends AbstractDiscoveryItem< } protected boolean hasTooltip(final CatalogItem connector) { - return connector.getOverview() != null && connector.getOverview().getSummary() != null - && connector.getOverview().getSummary().length() > 0; + return connector.getOverview() != null && connector.getOverview().getSummary() != null && connector.getOverview().getSummary().length() > 0; } public void initializeListeners() { @@ -219,15 +213,11 @@ public class DiscoveryItem<T extends CatalogItem> extends AbstractDiscoveryItem< protected boolean maybeModifySelection(boolean selected) { if (selected) { if (connector.isInstalled()) { - MessageDialog.openWarning(shellProvider.getShell(), "Install Connector", NLS.bind( - "{0} is already installed.", connector.getName())); + MessageDialog.openWarning(shellProvider.getShell(), "Install Connector", NLS.bind("{0} is already installed.", connector.getName())); return false; } if (connector.getAvailable() != null && !connector.getAvailable()) { - MessageDialog.openWarning(shellProvider.getShell(), - Messages.ConnectorDiscoveryWizardMainPage_warningTitleConnectorUnavailable, NLS.bind( - Messages.ConnectorDiscoveryWizardMainPage_warningMessageConnectorUnavailable, - connector.getName())); + MessageDialog.openWarning(shellProvider.getShell(), Messages.ConnectorDiscoveryWizardMainPage_warningTitleConnectorUnavailable, NLS.bind(Messages.ConnectorDiscoveryWizardMainPage_warningMessageConnectorUnavailable, connector.getName())); return false; } } diff --git a/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/wizards/DiscoveryResources.java b/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/wizards/DiscoveryResources.java index 2811f4524..91e8d7916 100644 --- a/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/wizards/DiscoveryResources.java +++ b/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/wizards/DiscoveryResources.java @@ -87,21 +87,21 @@ public class DiscoveryResources { public Image getIconImage(AbstractCatalogSource discoverySource, Icon icon, int dimension, boolean fallback) { String imagePath; switch (dimension) { - case 64: - imagePath = icon.getImage64(); - if (imagePath != null || !fallback) { + case 64 : + imagePath = icon.getImage64(); + if (imagePath != null || !fallback) { + break; + } + case 48 : + imagePath = icon.getImage48(); + if (imagePath != null || !fallback) { + break; + } + case 32 : + imagePath = icon.getImage32(); break; - } - case 48: - imagePath = icon.getImage48(); - if (imagePath != null || !fallback) { - break; - } - case 32: - imagePath = icon.getImage32(); - break; - default: - throw new IllegalArgumentException(); + default : + throw new IllegalArgumentException(); } if (imagePath != null && imagePath.length() > 0) { URL resource = discoverySource.getResource(imagePath); diff --git a/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/wizards/OverviewToolTip.java b/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/wizards/OverviewToolTip.java index c6efd7324..950bfd28c 100644 --- a/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/wizards/OverviewToolTip.java +++ b/bundles/org.eclipse.equinox.p2.ui.discovery/src/org/eclipse/equinox/internal/p2/ui/discovery/wizards/OverviewToolTip.java @@ -95,21 +95,15 @@ class OverviewToolTip extends GradientToolTip { final int containerWidthHintWithImage = 650; final int containerWidthHintWithoutImage = 500; - GridDataFactory.fillDefaults().grab(true, true).hint( - image == null ? containerWidthHintWithoutImage : containerWidthHintWithImage, SWT.DEFAULT).applyTo( - container); + GridDataFactory.fillDefaults().grab(true, true).hint(image == null ? containerWidthHintWithoutImage : containerWidthHintWithImage, SWT.DEFAULT).applyTo(container); - GridLayoutFactory.fillDefaults().numColumns((leftImage != null) ? 3 : 2).margins(5, 5).spacing(3, 0).applyTo( - container); + GridLayoutFactory.fillDefaults().numColumns((leftImage != null) ? 3 : 2).margins(5, 5).spacing(3, 0).applyTo(container); if (leftImage != null) { Label imageLabel = new Label(container, SWT.NONE); imageLabel.setImage(leftImage); int imageWidthHint = leftImage.getBounds().width + 5; - GridDataFactory.fillDefaults() - .align(SWT.BEGINNING, SWT.BEGINNING) - .hint(imageWidthHint, SWT.DEFAULT) - .applyTo(imageLabel); + GridDataFactory.fillDefaults().align(SWT.BEGINNING, SWT.BEGINNING).hint(imageWidthHint, SWT.DEFAULT).applyTo(imageLabel); } String summary = overview.getSummary(); @@ -118,9 +112,7 @@ class OverviewToolTip extends GradientToolTip { summaryContainer.setBackground(null); GridLayoutFactory.fillDefaults().applyTo(summaryContainer); - GridDataFactory gridDataFactory = GridDataFactory.fillDefaults() - .grab(true, true) - .span(image == null ? 2 : 1, 1); + GridDataFactory gridDataFactory = GridDataFactory.fillDefaults().grab(true, true).span(image == null ? 2 : 1, 1); if (image != null) { gridDataFactory.hint(widthHint, heightHint); } @@ -142,12 +134,10 @@ class OverviewToolTip extends GradientToolTip { final Composite imageContainer = new Composite(container, SWT.BORDER); GridLayoutFactory.fillDefaults().applyTo(imageContainer); - GridDataFactory.fillDefaults().grab(false, false).align(SWT.CENTER, SWT.BEGINNING).hint( - widthHint + (borderWidth * 2), heightHint).applyTo(imageContainer); + GridDataFactory.fillDefaults().grab(false, false).align(SWT.CENTER, SWT.BEGINNING).hint(widthHint + (borderWidth * 2), heightHint).applyTo(imageContainer); Label imageLabel = new Label(imageContainer, SWT.NULL); - GridDataFactory.fillDefaults().hint(widthHint, fixedImageHeight).indent(borderWidth, borderWidth).applyTo( - imageLabel); + GridDataFactory.fillDefaults().hint(widthHint, fixedImageHeight).indent(borderWidth, borderWidth).applyTo(imageLabel); imageLabel.setImage(image); imageLabel.setBackground(null); imageLabel.setSize(widthHint, fixedImageHeight); |