On Fri, Oct 31, 2014 at 11:17 AM, David Blaikie <[email protected]> wrote:

> Again reminds me that maybe an iterator idiom for typo corrections might
> be nice
>
> (then peaking would be "std::next(i)->...")
>

But that's up to you when/how you want to address that - given the current
design, this looks plausible.

If there's a nice way to implement this function without having to change
the object's internal state and then change it back, that'd be great (can
the guts of getNextCorrection be factored in such a way as you could call
"nextCorrectionIndex" as a non-mutating operation, then just use that to
lookup the ValidatedCorrections array? Then the original getNextCorrection
function could be something like CurrentTCIndex = nextCorrectionIndex(... ))


>
> On Wed, Oct 29, 2014 at 12:49 PM, Kaelyn Takata <[email protected]> wrote:
>
>> ---
>>  include/clang/Sema/SemaInternal.h | 15 +++++++++++++++
>>  1 file changed, 15 insertions(+)
>>
>>
>> _______________________________________________
>> cfe-commits mailing list
>> [email protected]
>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>>
>>
>
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to