Skip to content

Typo in keywords.txt for comparison reference [imported] #966

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
cmaglie opened this issue Nov 15, 2012 · 2 comments
Closed

Typo in keywords.txt for comparison reference [imported] #966

cmaglie opened this issue Nov 15, 2012 · 2 comments
Assignees
Labels
Component: IDE The Arduino IDE
Milestone

Comments

@cmaglie
Copy link
Member

cmaglie commented Nov 15, 2012

This is Issue 966 moved from a Google Code project.
Added by 2012-06-25T03:32:13.000Z by roncraig...@gmail.com.
Please review that bug for more context and additional comments, but update this bug.

Original labels: Type-Defect, Priority-Medium, Component-Build

Original description

**What steps will reproduce the problem?**
1. Enter if (x <= 3 || x >= 8 || y == 7)
2. Double-click each comparison operator and press Ctrl-Shift-F in turn, noting the message in the status line. Also notice whether context menu enables checking the reference materials for each operator.

What is the expected output? 

All three comparison operators should enable the context menu option. All three should display a message saying no reference is available.

What do you see instead?

You'll see message reporting 'No reference available for ">="' or "==". That's correct for "==" according to keyword.txt, but wrong for ">=". Also, the context menu won't enable the option to look up the ">=" operator in the reference materials.

The reason is that keywords.txt has these lines in it:
<       greaterthan
<=      greaterthanorequalto
which is incorrect, of course, and also
<       lessthan
<=      lessthanorequalto

I think we should add If to the ==, <, >, >=, and <= operators like so:
==      If
<               If
<=              If
>               If
>=              If
since the documentation for If (conditional) also covers the comparison operators.


**What version of the Arduino software are you using? On what operating**
**system?  Which Arduino board are you using?**
Using version 1.0.1. OS and board don't matter.


**Please provide any additional information below.**

@ffissore ffissore added the New label Feb 27, 2014
@ffissore ffissore self-assigned this Jul 1, 2015
@ffissore ffissore added the Component: IDE The Arduino IDE label Jul 9, 2015
@ffissore ffissore added this to the Release 1.6.6 milestone Jul 9, 2015
@ffissore
Copy link
Contributor

ffissore commented Jul 9, 2015

Fixed. Fix will be available with next hourly build http://www.arduino.cc/en/Main/Software#hourly

@MarkusLange
Copy link

cool!
INPUT_PULLDOWN LITERAL2 Constants RESERVED_WORD_2

is also missing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: IDE The Arduino IDE
Projects
None yet
Development

No branches or pull requests

4 participants