www.digitalmars.com         C & C++   DMDScript  

digitalmars.D.bugs - [Issue 12185] New: AA get() is getting in the way of diagnostics

reply d-bugmail puremagic.com writes:
https://d.puremagic.com/issues/show_bug.cgi?id=12185

           Summary: AA get() is getting in the way of diagnostics
           Product: D
           Version: D2
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: normal
          Priority: P2
         Component: druntime
        AssignedTo: nobody puremagic.com
        ReportedBy: andrej.mitrovich gmail.com



14:41:33 PST ---
void main()
{
    auto x = not;
}

test.d(5): Error: undefined identifier not, did you mean template get(T :
Value[Key], Value, Key, K, V)(T aa, K key, lazy V defaultValue) if
(is(typeof(true ? aa[key] : defaultValue)))?

Unfortunately due to the recent AA changes the new UFCS functions are getting
picked up by the spellchecker. I'm getting the above diagnostic more and more
frequently. I wonder if we can do something about this without adding special
cases to the compiler.

-- 
Configure issuemail: https://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Feb 16 2014
next sibling parent d-bugmail puremagic.com writes:
https://d.puremagic.com/issues/show_bug.cgi?id=12185


yebblies <yebblies gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |yebblies gmail.com



We could tweak the spell-checker so with len=3 it requires diff < 2

-- 
Configure issuemail: https://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Feb 16 2014
prev sibling next sibling parent d-bugmail puremagic.com writes:
https://d.puremagic.com/issues/show_bug.cgi?id=12185




00:33:07 PST ---
Maybe I should rename this to "spellchecker recommends symbols which look
nothing like each other". not => get is an awful recommendation here.

-- 
Configure issuemail: https://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Feb 17 2014
prev sibling next sibling parent d-bugmail puremagic.com writes:
https://d.puremagic.com/issues/show_bug.cgi?id=12185






 Maybe I should rename this to "spellchecker recommends symbols which look
 nothing like each other". not => get is an awful recommendation here.


But they only differ by two characters!!!

-- 
Configure issuemail: https://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Feb 17 2014
prev sibling next sibling parent d-bugmail puremagic.com writes:
https://d.puremagic.com/issues/show_bug.cgi?id=12185




-    size_t maxdist = seedlen < 3 ? seedlen - 1 : 2;
+    size_t maxdist = seedlen < 3 ? seedlen - 1 : seedlen == 3 ? 1 : 2;

-- 
Configure issuemail: https://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Feb 17 2014
prev sibling next sibling parent d-bugmail puremagic.com writes:
https://d.puremagic.com/issues/show_bug.cgi?id=12185




05:58:51 PST ---


 -    size_t maxdist = seedlen < 3 ? seedlen - 1 : 2;
 +    size_t maxdist = seedlen < 3 ? seedlen - 1 : seedlen == 3 ? 1 : 2;


Don't we have a natural language processing / AI expert among us? Cough cough
Andrei. :)

-- 
Configure issuemail: https://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Feb 17 2014
prev sibling next sibling parent d-bugmail puremagic.com writes:
https://d.puremagic.com/issues/show_bug.cgi?id=12185






 
 Don't we have a natural language processing / AI expert among us? Cough cough
 Andrei. :)


"undefined identifier 'not', but all your facebook friends liked this page
about cats! <link>"

-- 
Configure issuemail: https://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Feb 17 2014
prev sibling next sibling parent d-bugmail puremagic.com writes:
https://d.puremagic.com/issues/show_bug.cgi?id=12185




06:03:51 PST ---




 
 Don't we have a natural language processing / AI expert among us? Cough cough
 Andrei. :)


 
 "undefined identifier 'not', but all your facebook friends liked this page
 about cats! <link>"


LOL. So you planning on making this pull or too busy now?

-- 
Configure issuemail: https://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Feb 17 2014
prev sibling next sibling parent d-bugmail puremagic.com writes:
https://d.puremagic.com/issues/show_bug.cgi?id=12185










 
 Don't we have a natural language processing / AI expert among us? Cough cough
 Andrei. :)


 
 "undefined identifier 'not', but all your facebook friends liked this page
 about cats! <link>"


 
 LOL. So you planning on making this pull or too busy now?


Haha I'll go do it.

-- 
Configure issuemail: https://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Feb 17 2014
prev sibling next sibling parent d-bugmail puremagic.com writes:
https://d.puremagic.com/issues/show_bug.cgi?id=12185


yebblies <yebblies gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |pull
          Component|druntime                    |DMD
            Summary|AA get() is getting in the  |Spellchecker matches symbol
                   |way of diagnostics          |that are more than 50%
                   |                            |different



https://github.com/D-Programming-Language/dmd/pull/3276

-- 
Configure issuemail: https://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Feb 17 2014
prev sibling next sibling parent d-bugmail puremagic.com writes:
https://d.puremagic.com/issues/show_bug.cgi?id=12185




Commits pushed to master at https://github.com/D-Programming-Language/dmd

https://github.com/D-Programming-Language/dmd/commit/032a6292e9c9fa354540e5afab079574d27b7f71
Fix Issue 12185 - Spellchecker matches symbols that are more than 50% different

https://github.com/D-Programming-Language/dmd/commit/a98247dc37c27b1fb39b60b7d3be434975ee3984


Issue 12185 - Spellchecker matches symbols that are more than 50% different

-- 
Configure issuemail: https://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Feb 17 2014
prev sibling parent d-bugmail puremagic.com writes:
https://d.puremagic.com/issues/show_bug.cgi?id=12185


Andrej Mitrovic <andrej.mitrovich gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED


-- 
Configure issuemail: https://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
Feb 17 2014