Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Improve idetools --def logic to find more keywords #415

Closed
gradha opened this issue May 2, 2013 · 0 comments
Closed

Improve idetools --def logic to find more keywords #415

gradha opened this issue May 2, 2013 · 0 comments
Labels
Tools

Comments

@gradha
Copy link
Contributor

gradha commented May 2, 2013

In the following example only the first and last execProcesses are located by idetools. It would be nice if idetools also worked in the other two cases:

import osproc
execProcesses
execProcesses()
execProcesses(
execProcesses(@[])
@Araq Araq added the Idetools label Mar 12, 2015
@reactormonk reactormonk mentioned this issue Jun 14, 2015
Open
Clyybber pushed a commit to Clyybber/Nim that referenced this issue Nov 7, 2022
@bors @saem
Merge nim-lang#415
fa6495b 
415: internal: rewrite semTemplateDef in case style r=saem a=saem

## Summary
- refactoring to clean-up style and use nkError in more places

## Details
- refactoring of semTemplateDef in a case style
- treats `result` as a production, instead of mutating `n`
-`semPattern` now produces nkErrors
- `semPattern` nkError change required reporting in `semProcAux`

Co-authored-by: Saem Ghani <saemghani+github@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Tools
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@reactormonk @gradha @Araq