aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | Merge "Use lint error mechanism for config issues like no classes found"Tor Norbye2012-02-216-13/+66
|\ \ \ \ \ \
| * | | | | | Use lint error mechanism for config issues like no classes foundTor Norbye2012-02-216-13/+66
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If Lint cannot find the .class files for a project, it cannot run any of the class-file based checks (such as the NewApi check). This changeset adds a new category and issue id, "Lint Error", for these types of issues. In HTML reports, these errors are listed at the top. The issue explanation states that these errors don't represent bugs in the user's code, but that lint was not able to check certain things for the reasons given. In the case of no .class files found, it asks whether the project needs to be built first. It also uses these lint errors to emit errors in processing lint.xml configuration files. (Note that if you don't want to see these types of errors, you can suppress it via --disable LintError.) Change-Id: Ifc2f55566f3a0cd20189d43e4205201bc21ee280
* | | | | | Merge "Fix lint library project configuration handling"Tor Norbye2012-02-215-15/+17
|\ \ \ \ \ \
| * | | | | | Fix lint library project configuration handlingTor Norbye2012-02-215-15/+17
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A lint.xml file in a library project should be able to suppress lint errors in that project. This changeset fixes a couple of bugs which prevented this from working: In Eclipse, the configuration was cached, which meant a single lint run through multiple projects would just use the configuration of the first encountered project. And from the command line driver, the code to handle --check handling would override configuration severity in one scenario. Change-Id: Ib08e2b14582ab53e5ab89244738d1bd77839610e
* | | | | | Merge "SDK Stats: report OS vs JVM architecture."Raphael2012-02-211-32/+180
|\ \ \ \ \ \ | |_|/ / / / |/| | | | |
| * | | | | SDK Stats: report OS vs JVM architecture.Raphael2012-02-171-32/+180
| | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I1fef83d30fc9e7fa91c136fa9c459bf18a1e1d9c
* | | | | | ADT editors Hyperlinks potentially not using the right GLE.Raphael2012-02-215-7/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes a wrong GLE reference in Hyperlinks.getCongigutation(). Also suppresses a few annoying warnings (mostly obsolete imports.) Change-Id: Ibff38ecf3048d96c2eb3090b2dfd2f4066fe8bc6
* | | | | | ADT fix save on layout configuration change.Raphael2012-02-2112-79/+80
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When a layout is opened and its configuration is changed, the new delegate layout wasn't properly saving the layout before changing files. This is due to another instance of the delegate code calling a method on the delegate whereas it should have called a method with the same exact name on the editor. To reduce this confusion, this CL adds a 'delegate' prefix to methods which are common in the CommonXmlEditor and the delegates. At the expense of being a bit more verbose it makes it clearer which method is being invoked. Change-Id: I34e620a9d25272fe75dab54279161b04a961a598
* | | | | | ADT fix GLE configuration change.Raphael2012-02-212-3/+4
| |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | SDK Bug: 6036506 Change-Id: I92f86940bf158ca8da8597a7f387d1f2595c0131
* | | | | Merge "Lint rule checking that activities and services are registered"Tor Norbye2012-02-2119-2/+501
|\ \ \ \ \ | |_|/ / / |/| | | |
| * | | | Lint rule checking that activities and services are registeredTor Norbye2012-02-1619-2/+501
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This changeset adds a lint rule checking that any named, non-abstract classes which extend Activity, Service or ContentProvider have a corresponding registration in the manifest file. It also ensures that the registration is under the right kind of tag (<activity>, <service>, <provider> or <receiver>). Change-Id: Ifbb587ae7b3eb6529d5099b33245d0478d48c41f
* | | | | Merge "gltrace: Show wall time and thread time."Siva Velusamy2012-02-176-15/+83
|\ \ \ \ \
| * | | | | gltrace: Show wall time and thread time.Siva Velusamy2012-02-176-15/+83
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Extend the protobuf to include wall time and thread time. Both durations are displayed in the function view. Change-Id: I407255464469195f7e6cf1a4b526637cb7c02823
* | | | | | Merge "Fix lombok jar location name"Tor Norbye2012-02-171-1/+1
|\ \ \ \ \ \ | |/ / / / / |/| | | | |
| * | | | | Fix lombok jar location nameTor Norbye2012-02-171-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I4881ffe726e72549b5a782ac648ae8dcbec19435
* | | | | | Fix build: generate addon.xml in v3, not new v4 xsdRaphael2012-02-171-0/+0
| |_|/ / / |/| | | | | | | | | | | | | | Change-Id: I7a416dfacfa871755ce86660feb73b699ff97dc1
* | | | | Merge "Remove warning that minsdkVersion is lower than the compile target."Xavier Ducrohet2012-02-163-21/+14
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | Remove warning that minsdkVersion is lower than the compile target.Xavier Ducrohet2012-02-163-21/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is not needed anymore now that Lint can check if this is actually a problem or not. Change-Id: I9f43f46033e07c44b78728e8582364b355ff1ccf
* | | | | Add @android resource version checking to the lint API detectorTor Norbye2012-02-168-35/+159
| |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | This changeset makes the API lint detector look at XML attribute values and XML value text nodes and check any references to @android resources to ensure that they are available in all supported versions. Change-Id: Iab0d23423c30381e06b32f54aa902a31cc1f9a1c
* | | | Add suppress-scope handling for more XML detectorsTor Norbye2012-02-1621-85/+458
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This changeset adds is-suppressed checking to most of the remaining XML detectors. These are detectors which do post processing on data in order to compute the warnings, at which point they no longer have access to the original DOM node context. This changeset adds various checks to handle this, such as storing the DOM node in the location handle client data and checking explicitly before using the handles. It adds various unit tests for the suppress scenarios too. There are now two remaining detectors which don't properly handle suppression: the TranslateDetector, and the DuplicateIdDetector (in cross layout scenarios). These will be addressed in a separate CL. Change-Id: I53e792dd2f2802c7dce16beb73127f20d9b2dba8
* | | | Build fix: fix import packageTor Norbye2012-02-162-2/+4
| | | | | | | | | | | | | | | | Change-Id: Id8f467c5a088101e6b50c33fa0703520e31bdede
* | | | Merge "Add onClick check for lint"Tor Norbye2012-02-169-28/+521
|\ \ \ \
| * | | | Add onClick check for lintTor Norbye2012-02-169-28/+521
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This lint check makes sure that if a layout references android:onClick="foo" then one of the Java classes provide a public void foo(View) method. It also looks for simple typos, and warns if the access modifiers are not right. (It also moves some ASM-related utility code out of specific detectors and into the ClassContext class) Change-Id: Ifb0820221175a0760b83992f54e99f761b4d7097
* | | | | Merge "hierarchyviewer: Add DumpDisplayList action."Siva Velusamy2012-02-161-0/+3
|\ \ \ \ \ | |_|/ / / |/| | | |
| * | | | hierarchyviewer: Add DumpDisplayList action.Siva Velusamy2012-02-161-0/+3
| | | | | | | | | | | | | | | | | | | | Change-Id: Ic9df226e765db323881cbd53f7f3220029e21fae
* | | | | Merge "Delayed builds from the builder."Xavier Ducrohet2012-02-162-2/+40
|\ \ \ \ \ | |_|/ / / |/| | | |
| * | | | Delayed builds from the builder.Xavier Ducrohet2012-01-202-2/+40
| | |/ / | |/| | | | | | | | | | Change-Id: I70adc2f7ad75aa631e4d90139ab7b08a6aa0b5a0
* | | | Merge "Update changes.txt for ADT 17"Tor Norbye2012-02-152-0/+25
|\ \ \ \
| * | | | Update changes.txt for ADT 17Tor Norbye2012-02-152-0/+25
| | | | | | | | | | | | | | | | | | | | Change-Id: I394036a7025a2cbd050aa6916204a09bcce83bae
* | | | | Merge "Fix exception reported on android-developers forum: r can be null"Tor Norbye2012-02-151-1/+1
|\ \ \ \ \
| * | | | | Fix exception reported on android-developers forum: r can be nullTor Norbye2012-02-151-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Iff529a37901b62bcbc53b23189d1aaa2215dbfea
* | | | | | Merge "Add suppress-this to Lint View, and preserve tree expansion state"Tor Norbye2012-02-159-38/+204
|\ \ \ \ \ \
| * | | | | | Add suppress-this to Lint View, and preserve tree expansion stateTor Norbye2012-02-159-38/+204
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This changeset adds a new toolbar button to the Lint View: "Suppress this error with an annotation/attribute". This is enabled when the selected warning is for a Java or XML file, and it adds the annotation or suppress attribute in the relevant source file for the warning. It also adds some handling to preserve the expanded tree state in the Lint View when the view is refreshed, which happens automatically whenever a fix is applied or a file is saved and the marker set changes. Change-Id: I393ee5f6e062a0e08a7e9457a03f37d8899a4df8
* | | | | | Merge "SDK Repository: name/vendor id/display for addons."Raphael2012-02-1513-158/+340
|\ \ \ \ \ \
| * | | | | | SDK Repository: name/vendor id/display for addons.Raphael2012-02-1513-158/+340
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | SDK Bug: 21942, part 2 of 3. (part 3 is replacing manifest.ini by source.props) Change-Id: I96be0d8210120927744224d909f7ee7184ebfecf
* | | | | | | Merge "SDK Manager: support forward evolution of repository-N.xml"Raphael2012-02-152-15/+46
|\ \ \ \ \ \ \ | |/ / / / / /
| * | | | | | SDK Manager: support forward evolution of repository-N.xmlRaphael2012-02-152-15/+46
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I1ec10f677d542b02d1588c43d5b40ed10d94f874
* | | | | | | Merge "SDK Repository: vendor id/display name for extras."Raphael2012-02-1522-243/+1273
|\ \ \ \ \ \ \ | |/ / / / / /
| * | | | | | SDK Repository: vendor id/display name for extras.Raphael2012-02-1522-243/+1273
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | SDK Bug: 21942, part 1 of 2. Change-Id: Id9c026965b365e57302c56620f4ec8f88573d633
* | | | | | | Merge "SDK Repository: remove <extra> from repository.xml"Raphael2012-02-156-5/+1039
|\ \ \ \ \ \ \ | |/ / / / / / | | | / / / / | |_|/ / / / |/| | | | |
| * | | | | SDK Repository: remove <extra> from repository.xmlRaphael2012-02-156-5/+1039
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | To simplify and avoid XSD duplication, remove extra packages from the repository.xsd. Instead they should be defined via an addon.xml. Note that -repository-6.xsd is prefixed with a dash, meaning the SDK Manager 17 knows about it and can enforce it but the build server will continue creating repos in v5 for now. Once r17 is deployed, we can make the xsd v6 mandatory for r18. Change-Id: I00d4644378bcc95bfd9ceb7eede0c8c720926f74
* | | | | | Merge "gltrace: Provide details view for uniform variables."Siva Velusamy2012-02-152-0/+174
|\ \ \ \ \ \ | |/ / / / / |/| | | / / | | |_|/ / | |/| | |
| * | | | gltrace: Provide details view for uniform variables.Siva Velusamy2012-02-152-0/+174
| |/ / / | | | | | | | | | | | | | | | | | | | | Display uniforms rendered as a vector or a matrix. Change-Id: Ifbdaad571f71e2f4d477027548dd6951eddba8e5
* | | | Merge "SDK Manager: use find_lock.exe when install dir is locked."Raphael2012-02-153-41/+172
|\ \ \ \
| * | | | SDK Manager: use find_lock.exe when install dir is locked.Raphael2012-02-093-41/+172
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | For the SDK Manager on Windows, this uses find_lock.exe to try to find the processes that are locking an existing SDK folder that cannot be moved out of the way. The result is displayed in a simple yes/no dialog or on the command-line if invoked without UI. Change-Id: I39fa7529e207e870376a30c8311440f9c240d4b3
* | | | | Merge "Tweak category order"Tor Norbye2012-02-151-1/+1
|\ \ \ \ \ | |_|/ / / |/| | | |
| * | | | Tweak category orderTor Norbye2012-02-141-1/+1
| | | | | | | | | | | | | | | | | | | | Change-Id: If83a942f99da90d061529092feaab3be94a8e0ad
* | | | | gltrace: Add support for shader uniforms.Siva Velusamy2012-02-147-7/+357
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch adds the necessary state and transformations to support uniforms used in shaders. Change-Id: Ibf93098977d8f8a733d6572b1524ff50eda28b6f
* | | | | Merge "Fix HTML export category handling"Tor Norbye2012-02-144-11/+14
|\ \ \ \ \ | |/ / / /
| * | | | Fix HTML export category handlingTor Norbye2012-02-144-11/+14
| | | | | | | | | | | | | | | | | | | | Change-Id: Ia8f0c1aebb4e797352a0513b8113b5d6b75ef0dc