Skip to content

Bython parser improvement#31

Open
filipefjn wants to merge 36 commits into
mathialo:masterfrom
filipefjn:master
Open

Bython parser improvement#31
filipefjn wants to merge 36 commits into
mathialo:masterfrom
filipefjn:master

Conversation

@filipefjn

@filipefjn filipefjn commented May 30, 2019 •
edited
Loading

Copy link
Copy Markdown

Reimplemented the parser function to use recursion. This implementation allowed it to be more consistent. Also, the following features were implemented:

  • Detection of C style comments (/* */)
  • Fixed python dictionaries (= { })
  • Added a CLI option "--debug" that outputs the scope detection
  • Other minor fixes

I've also changed the version from 0.8 to 0.9.

Notes:

  • Detection of C++ style comments (//) was removed due to a conflict with the python '//' operator

filipefjn added 30 commits May 15, 2019 20:12
@filipefjn
c++ style single line comments implemented
179ce3a
@filipefjn
started the implementation of a recursive parser
305e8ac
@filipefjn
added '#' scope on recursive parser
f658382
@filipefjn
recommited all the changes with my name set on git
d64fe83
@filipefjn
Merge branch 'master' of https://github.com/filipefjn/bython
091f466
@filipefjn
implemented tests for: comments (c, c++ and python styled), strings (…
1fab9b0 
…single and double quoted) and dicts
@filipefjn
unified the global and brace scopes conditional
6af238d
@filipefjn
outputting the .py file for the first time
4a12556
@filipefjn
implemented a function for removing indentation
a4d6e9c
@filipefjn
implemented functions for preparing braces and removing empty lines. …
070f19a 
…also changed the remove_identation function
@filipefjn
implemented indentation
a5ca359
@filipefjn
fixed python dicts
2713ce1
@filipefjn
fixed error when there's a comment after a call and before the openin…
a349d6c 
…g brace
@filipefjn
implemented the removal of semicolons
daa7e3e
@filipefjn
fixed double indentation and wrong indentation after dict
83ed356
@filipefjn
implemented multi-line comments
6825876
@filipefjn
fixed doubled character after non-dicts
5041fa2
@filipefjn
disabled the remove_semicolons for now
d870bd1
@filipefjn
added some comments and minor fixes
5e9cd48
@filipefjn
fixed a few cases where comments would be removed
ca0ce33
@filipefjn
renamed a function to parse_file_recursively
258af8a
@filipefjn
added a debug mode
69bb5bf
@filipefjn
preparing for first release
e86c6a7
@filipefjn
disabled one regex substitution and fixed an infinite loop
cf452c7
@filipefjn
implemented a debug option for the cli
d1aeeae
@filipefjn
fixed empty lines bug. filtered file now only appears when debug mode…
611c1c2 
… is active
@filipefjn
fixed '} else {'
5b69929
@filipefjn
implemented parenthesis scope
ffc4a9c
@filipefjn
added function descriptions and more comments
1ed72b6
@filipefjn
put my name in the authors
65d15d2
sairam4123
sairam4123 approved these changes Jan 4, 2021
View reviewed changes

@sairam4123 sairam4123 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

2 more reviewers

@sairam4123 sairam4123 sairam4123 approved these changes

@Willie169 Willie169 Willie169 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@filipefjn @sairam4123 @Willie169