6e4f648b48
Added code issue to address bug #11856
...
https://bugzilla.xamarin.com/show_bug.cgi?id=11856
This works for expressions by doing the following:
- it looks if the expression is the first in a sequence of conditional expressions
- if is first, it extracts all expressions in the chained expressions
- after that it compares all expressions and if two are found as duplicates, the first duplicate is shown
The code fix will remove the duplicate expression.
This bug will also look to bit operations ( & and | ) as they have similar semantics
13 years ago
e287bdb020
Fixed bug in TestRefactoringContext that attempted to put fields inside properties
13 years ago
94cfe777b4
Improved ExtractFieldAction for static getters and setters.
13 years ago
8602033a04
Improved ExtractField. Tests currently fail for variables in getters.
13 years ago
3ea78f887a
Added (currently failing) tests for static fields. Minor formatting fixes.
13 years ago
2710510610
Fixed ExtractFieldAction to handle type inference better and added unit tests that failed with the previous version.
13 years ago
d57537f061
Added additional tests. One of new ones is failing.
13 years ago
3dcad69810
Remove "Remove Field Refactory" as it is dangerous.
13 years ago
bd486b50b0
Code updates
13 years ago
fe53dec615
untracked files on master: 9a482c2 Remove unused field
13 years ago
9db3f46cbc
Support non void arguments
13 years ago
ad0c12d1da
Sample implementation for bug #11866
...
https://bugzilla.xamarin.com/show_bug.cgi?id=11866
The test fails because EOLN issues for now, and it works just for methods that return void for now. But it does look for methods inside the classes that have same: arguments, return type and modifiers (so are duplicates)
13 years ago
1c86d72523
Revert "Simplified the scrip API a bit (insert cursor/links are no longer task"
...
I do it different.
This reverts commit 4fa3c33d6b
13 years ago
eca73de6e4
ExpressionOfCompatibleTypeCodeIssue will not convert class types any longer.
13 years ago
17b26e4ce0
Added more unit tests to:
...
https://bugzilla.xamarin.com/show_bug.cgi?id=10728
There is a bug in conversion between double and float (in Convert part of the base classes)
13 years ago
0d391896fb
Fixes bug #10731
...
https://bugzilla.xamarin.com/show_bug.cgi?id=10731
It will look as a separate issue checker for basic interfaces and it will give the first implementation that works as base (so it will not give 5 possible conversions, even all are possible). In practice works really well. It is losely based on the ParameterCanbeDemotedIssue so I exposed the code to not duplicate it. Yet the code is much simpler this action.
13 years ago
dd8caea759
Fixes bug 11709 part of Xamarin's bugzilla: https://bugzilla.xamarin.com/show_bug.cgi?id=11709
...
It makes ConvertToStaticMethod as an issue.
13 years ago
c965df75ab
Adds a refactor that removes fairly accurate a field in all places (it is based on FindReferences).
13 years ago
4fa3c33d6b
Simplified the scrip API a bit (insert cursor/links are no longer task
...
based).
13 years ago
b632cece34
Fixed formatting bug in create changed event.
13 years ago
9397fc1987
Fixed some issues in wrapping formatting.
13 years ago
a796caa828
Fixed bug in wrapping options.
13 years ago
c2ca95ac63
Fixed some formatting bugs.
13 years ago
02bf0ba580
Added alignment options for declarations.
13 years ago
985ebe36a6
Added alignment tests / fixed failing wrapping unit tests.
13 years ago
62aaf41af6
Fixed failing built.
13 years ago
dd449ac3ec
Fixed project.
13 years ago
73f679e6c3
[CodeIssue] Modified PublicConstructorInAbstractClass issue to support Resharper disable comment
13 years ago
d1196a8ffb
Fixes bug #10728 (Auto-Fix regarding control flow: "Convert to correct type")
13 years ago
4fe454fd59
Fixes bug 11710 based on review of Daniel Grunwald
13 years ago
d1be453e2a
Fix return value conversion within async lambda.
13 years ago
579d2aca48
[CodeIssue]PublicConstructorInAbstractClass
...
Convert modifiers of constructors in Abstract class from "public" to
"protected".
13 years ago
42921dca18
[CodeIssue] Redundant Constructor issue
...
Do not need an empty constructor
13 years ago
0882827229
Fixed failing unit test.
13 years ago
4a11a13672
Documentation fixes.
13 years ago
da89feb228
Added missing formatting option.
13 years ago
0d99a1c634
Two new code issues
...
1. Static contructor cannot take parameters.
2. Static constructor cannot take access modifier.
13 years ago
e5f2c0bfd6
Add the capability to remove base call if is not needed. It exposes some classes so maybe I'm not using the proper APIs. But it works good for practical the test case I wrote.
13 years ago
1550fa1195
Added test for the delegate/constructor bug.
13 years ago
d22b96f12c
Fixed formatting bug.
13 years ago
ac9732776c
Makes not static methods
...
A better than nothing implementation of Bug #10730 .
https://bugzilla.xamarin.com/show_bug.cgi?id=10730
When multi-file tracking of methods is implemented, the change should change the usages of variables.Call(...) to ClassName.Call(...)
13 years ago
f806f635be
Auto-Fix for remove redundant assignment have issues in some cases (Part II) fixes
13 years ago
74ea8505b8
Added fix from Ciprian Khlud<ciprian.mustiata@yahoo.com>: Fix in
...
hoping to address the issues regarding some edge cases of remove
redundant variables in relation with functions.
13 years ago
ebe7045d05
Added some tests to the redundant null check.
13 years ago
01d4361350
Reformatted redundant null check.
13 years ago
ee3c6e89ed
[CodeIssue] Redundant Null Check
...
No need to check null where is keyword exits.
13 years ago
d8314d7e49
Implemented array initializer formatting.
13 years ago
28b623840b
Add unit test demonstrating that the DepthFirstVisitor optimization was invalid.
13 years ago
1d5c85f3f1
Implemented resharper disableing for redundant using directive.
13 years ago
020128ba58
CanBeDemotedIssue now respects the
...
CA1011:ConsiderPassingBaseTypesAsParameters suppression.
13 years ago
Ciprian Khlud
Luís Reis
Mike Krüger
leoowen19
Daniel Grunwald