Page 1 of 1

Spell Checker issues

Posted: Tue Jul 08, 2014 3:22 am
by Gibbon1
Umm... checking out the new 'spell check' which is interesting to me since I'm a horrid spellmeister

Running it on some of my code I noted that it could be improved it it would recognize
hex numbers like 0xFF23.
function, variable and file names.

Note really complaining about those as they are feature requests really.

But have been having this problem. Original code with the word Hopfully which the spell checker caught but flubbed replacing it so it ended up as HHopefullythe. Might be that fixing multiple spelling errors (in my code? never!) gets the spell checker out of sync between where the word now is in the file and where is used to be.

/**
* @brief Set Master Ping Mode
* @param prm user supplied channel
* @param set_var unused
* @return cli response
*
* Sets up the master hop mode. Where the unit transmits test packets to
* the gold unit (in slave idle mode). Hopfully the slave responds with
* response test packets.
*
*/

/**
* @brief Set Master Ping Mode
* @param prm user supplied channel
* @param set_var unused
* @return cli response
*
* Sets up the master hop mode. Where the unit transmits test packets to
* the gold unit (in slave idle mode). HHopefullythe slave responds with
* response test packets.
*/

Re: Spell Checker issues

Posted: Tue Jul 08, 2014 6:29 pm
by frank_frl
Hi Gibbon1,

I'm the developer of the spell checker and I'm going to take a look at it. Guess you right and it's getting out of sync for some reason.

What do mean with recognizing 'function, variable and file names', do mean when they are in a comment.
I did had this feature implemented a while ago, but to do this I have to parse Eran's code completion and he changed a lot there meanwhile.
I'm also afraid that this takes to much time for the real time checker (the old version had only a on demand checker)

Regards

Frank

Re: Spell Checker issues

Posted: Wed Jul 09, 2014 2:53 am
by Gibbon1
>What do mean with recognizing 'function, variable and file names', do mean when they are in a comment.

Yeah, would be a nice feature, but it's 'polishing'

Also I noted that the spell checker doesn't handle contractions like doesn't and can't

Otherwise having a spell checker is utterly great!