- 
                Notifications
    You must be signed in to change notification settings 
- Fork 641
Prolog: new parser #4249
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
Prolog: new parser #4249
Conversation
6a40e2e    to
    c2cc60d      
    Compare
  
    | Codecov Report❌ Patch coverage is  
 Additional details and impacted files@@            Coverage Diff             @@
##           master    #4249      +/-   ##
==========================================
+ Coverage   85.92%   85.98%   +0.06%     
==========================================
  Files         248      249       +1     
  Lines       63733    64102     +369     
==========================================
+ Hits        54765    55121     +356     
- Misses       8968     8981      +13     ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
 | 
da928d4    to
    69adaf2      
    Compare
  
    9a60ebd    to
    b928802      
    Compare
  
    6bbab30    to
    81a473d      
    Compare
  
    There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Adds a new Prolog parser to Universal Ctags, implementing comprehensive tag extraction for predicates, grammars, and modules in Prolog source files. The parser supports various Prolog syntax features including module-qualified atoms, DCG rules, string literals, and numeric literals.
- New Prolog parser with support for predicates, grammars, and modules
- Language selection mechanism to differentiate between Perl and Prolog files
- Integration with build system and test infrastructure
Reviewed Changes
Copilot reviewed 32 out of 32 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description | 
|---|---|
| parsers/prolog.c | Main parser implementation with tokenizer and tag extraction | 
| main/tokeninfo.c | Helper functions for token-based tag creation | 
| main/tokeninfo.h | Header declarations for new token utilities | 
| main/selectors.c | Language selection logic for Perl vs Prolog disambiguation | 
| main/selectors.h | Header for language selector function | 
| parsers/perl.c | Updated to use new language selector | 
| main/parsers_p.h | Registration of Prolog parser | 
| source.mak | Build system integration | 
| win32/* | Visual Studio project files | 
| docs/news/HEAD.rst | Documentation update | 
| Units/parser-prolog.r/* | Test cases and expected outputs | 
| misc/validators/validator-swipl | Validation script for SWI-Prolog | 
| circle.yml | CI configuration update | 
| Tmain/* | Updated test expectations | 
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 32 out of 32 changed files in this pull request and generated 3 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 36 out of 36 changed files in this pull request and generated 3 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Signed-off-by: Masatake YAMATO <[email protected]>
Signed-off-by: Masatake YAMATO <[email protected]>
Signed-off-by: Masatake YAMATO <[email protected]>
Signed-off-by: Masatake YAMATO <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 36 out of 36 changed files in this pull request and generated 1 comment.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 36 out of 36 changed files in this pull request and generated 1 comment.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Close universal-ctags#2628. Signed-off-by: Masatake YAMATO <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 36 out of 36 changed files in this pull request and generated no new comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Signed-off-by: Masatake YAMATO <[email protected]>
Close #2628.