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

#define pre processor #194

Open
Tracked by #175
samthakur587 opened this issue Oct 1, 2024 · 6 comments · May be fixed by #262
Open
Tracked by #175

#define pre processor #194

samthakur587 opened this issue Oct 1, 2024 · 6 comments · May be fixed by #262
Assignees
@himaanisrivatsava
Labels
Directx backend issue at directx backend enhancement New feature or request hacktoberfest

Comments

@samthakur587
Copy link
Contributor

samthakur587 commented Oct 1, 2024 •
edited by NripeshN
Loading

No description provided.

@samthakur587 samthakur587 mentioned this issue Oct 1, 2024
Open
20 tasks
@samthakur587 samthakur587 added enhancement New feature or request Directx backend issue at directx backend hacktoberfest labels Oct 1, 2024
@amarcoder01
Copy link

I’d like to work on this issue. Please assign it to me

@NripeshN
Copy link
Member

I’d like to work on this issue. Please assign it to me

To assign issues to yourself please read this documentation: https://github.com/CrossGL/crosstl/blob/main/CONTRIBUTING.md#-assigning-or-creating-issues

@CrossGL CrossGL deleted a comment from NripeshN Dec 26, 2024
@CrossGL CrossGL deleted a comment from NripeshN Dec 26, 2024
@CrossGL CrossGL deleted a comment from NripeshN Dec 26, 2024
@himaanisrivatsava
Copy link

@CrossGL-issue-bot assign me

@CrossGL-issue-bot
Copy link

Thank you for taking on this issue! If you have any questions or need assistance, feel free to ask.

himaanisrivatsava added a commit to himaanisrivatsava/crosstl that referenced this issue Jan 5, 2025
@himaanisrivatsava
CrossGL#194 define preprocessor
e0e714c 
feat: Add DirectX preprocessor for handling #define, #ifdef, #include, and related directives

refactor: Integrate preprocessor into DirectX lexer

refactor: Update DirectX backend init to include preprocessing pipeline

test: Add tests for DirectX preprocessor functionality

fix: Improve error handling for invalid directives in preprocessor
himaanisrivatsava added a commit to himaanisrivatsava/crosstl that referenced this issue Jan 5, 2025
@himaanisrivatsava
CrossGL#194 define preprocessor
f4c8ec1 
Fix: importing HLSLLexer from crossGL backend
himaanisrivatsava added a commit to himaanisrivatsava/crosstl that referenced this issue Jan 5, 2025
@himaanisrivatsava
CrossGL#194 define preprocessor .... #'commit30?
2808ec6 
Mocking os.path.exists: I added the @patch("os.path.exists", return_value=True) decorator to mock os.path.exists for all the test methods that deal with file inclusion. This ensures that file existence checks return True during tests.

Raising ValueError for Undefined Macros: I updated the expand_macros method to raise a ValueError when undefined macros are encountered.
@github-actions github-actions bot added the stale label Jan 15, 2025
@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Jan 22, 2025
@himaanisrivatsava
Copy link

@CrossGL-issue-bot assign me

@CrossGL-issue-bot
Copy link

The issue you are trying to assign to yourself is already assigned.

@NripeshN NripeshN linked a pull request Jan 24, 2025 that will close this issue
Open
4 tasks
@NripeshN NripeshN reopened this Jan 24, 2025
@github-actions github-actions bot removed the stale label Jan 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@himaanisrivatsava himaanisrivatsava

Labels
Directx backend issue at directx backend enhancement New feature or request hacktoberfest
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

#194 define Preprocessor himaanisrivatsava/crosstl
5 participants
@samthakur587 @NripeshN @amarcoder01 @himaanisrivatsava @CrossGL-issue-bot