Attached are 3 patches that add more support for adding */& mismatch fixits. In particular, C function parameters would get handled as well as assignments in general.

1) In the first patch, I refactored most of the fixit generation functionality out of SemaOverload and provided a simple conversion checking function. This patch also includes a bug fix - the range for FixItHint::CreateRemoval was not specified correctly.

Attachment: deref_addof_refactor.diff
Description: Binary data


2) This one adds fixit generation to the  Sema::DiagnoseAssignmentResult().

Attachment: deref_addof_assign.diff
Description: Binary data


3) Test the fixits generated as a result of  patch #2.

Attachment: deref_addof_assign_tests.diff
Description: Binary data


Not done:
ObjectiveC requires more testing. 
We might be able to use the factored out code to insert the fixit generation in places other then overloading and DiagnoseAssignmentResult().
ConversionFixItGenerator::CompareTypesSimple() could be improved.

_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to