Daniel Grunwald
17c4315974
Rename IsExtensionMethodGroupConversion -> DelegateCapturesFirstArgument.
12 years ago
Daniel Grunwald
a2580063dd
Fix ExpressionOfCompatibleTypeCastIssue.
12 years ago
Daniel Grunwald
d09dc0e10a
Fix compiler warnings
12 years ago
Daniel Grunwald
e85128df57
Fix unit test failure due to mix of tabs and spaces.
12 years ago
Ciprian Khlud
82c272545a
Fixed spacing
12 years ago
Ciprian Khlud
8666d6ffae
Fix of bug #11855
...
https://bugzilla.xamarin.com/show_bug.cgi?id=11855
This will notify user to use after ifs, whiles and fors not colons, but blocks.
12 years ago
Mike Krüger
ba13e28750
Fixed bug in redundant assignment issue.
12 years ago
Mike Krüger
1ad8c17390
Fixed completion bug.
12 years ago
Mike Krüger
05c5931a05
Disabled convert to static method issue.
...
It's not clear to me how the body analyzation is working - it seems to
be missing at all. Therefore I disable this issue until it's
implemented.
12 years ago
Mike Krüger
f40346db06
Fixed some issues in convert to static method issue.
12 years ago
Mike Krüger
fa671aacff
Fixed duplicate if chain issue tests.
12 years ago
Mike Krüger
5f867c5321
Ignore duplicate body method issue.
...
I take that one out for now - tests are failing and that's a very
unlikely issue to happen.
12 years ago
Mike Krüger
78442bb33a
Fixed static constructor access modifier issue tests.
12 years ago
Ciprian Khlud
c43a3ee9c3
Remove duplicate ifs, in if/else chains in case that the expression is duplicated. It can happen in copy/paste cases.
...
Fixes formatting of the previous commit to match K&R style.
12 years ago
Ciprian Khlud
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
12 years ago
Luís Reis
e287bdb020
Fixed bug in TestRefactoringContext that attempted to put fields inside properties
12 years ago
Luís Reis
94cfe777b4
Improved ExtractFieldAction for static getters and setters.
12 years ago
Luís Reis
8602033a04
Improved ExtractField. Tests currently fail for variables in getters.
12 years ago
Luís Reis
3ea78f887a
Added (currently failing) tests for static fields. Minor formatting fixes.
12 years ago
Luís Reis
2710510610
Fixed ExtractFieldAction to handle type inference better and added unit tests that failed with the previous version.
12 years ago
Luís Reis
d57537f061
Added additional tests. One of new ones is failing.
12 years ago
Ciprian Khlud
3dcad69810
Remove "Remove Field Refactory" as it is dangerous.
12 years ago
Ciprian Khlud
bd486b50b0
Code updates
12 years ago
Ciprian Khlud
9db3f46cbc
Support non void arguments
12 years ago
Ciprian Khlud
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)
12 years ago
Mike Krüger
1c86d72523
Revert "Simplified the scrip API a bit (insert cursor/links are no longer task"
...
I do it different.
This reverts commit 4fa3c33d6b
.
12 years ago
Ciprian Khlud
eca73de6e4
ExpressionOfCompatibleTypeCodeIssue will not convert class types any longer.
12 years ago
Ciprian Khlud
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)
12 years ago
Ciprian Khlud
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
Ciprian Khlud
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
Ciprian Khlud
c965df75ab
Adds a refactor that removes fairly accurate a field in all places (it is based on FindReferences).
13 years ago
Mike Krüger
4fa3c33d6b
Simplified the scrip API a bit (insert cursor/links are no longer task
...
based).
13 years ago
Mike Krüger
b632cece34
Fixed formatting bug in create changed event.
13 years ago
Erik Källén
02cc0a38c6
Added property Conversion.IsExtensionMethodGroupConversion to determine whether a method group conversion is being performed on an extension method using extension method invocation syntax (eg. Func<int> f = myEnumerable.Single).
13 years ago
Mike Krüger
62aaf41af6
Fixed failing built.
13 years ago
leoowen19
73f679e6c3
[CodeIssue] Modified PublicConstructorInAbstractClass issue to support Resharper disable comment
13 years ago
ciplogic
d1196a8ffb
Fixes bug #10728 (Auto-Fix regarding control flow: "Convert to correct type")
13 years ago
ciplogic
4fe454fd59
Fixes bug 11710 based on review of Daniel Grunwald
13 years ago
Daniel Grunwald
d1be453e2a
Fix return value conversion within async lambda.
13 years ago
leoowen19
579d2aca48
[CodeIssue]PublicConstructorInAbstractClass
...
Convert modifiers of constructors in Abstract class from "public" to
"protected".
13 years ago
leoowen19
42921dca18
[CodeIssue] Redundant Constructor issue
...
Do not need an empty constructor
13 years ago
Mike Krüger
0882827229
Fixed failing unit test.
13 years ago
Daniel Grunwald
4a11a13672
Documentation fixes.
13 years ago
Mike Krüger
da89feb228
Added missing formatting option.
13 years ago
leoowen19
0d99a1c634
Two new code issues
...
1. Static contructor cannot take parameters.
2. Static constructor cannot take access modifier.
13 years ago
ciplogic
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
Mike Krüger
1550fa1195
Added test for the delegate/constructor bug.
13 years ago
ciplogic
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
ciplogic
f806f635be
Auto-Fix for remove redundant assignment have issues in some cases (Part II) fixes
13 years ago
Mike Krüger
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