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.

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

Pre-processor comment slash error with include #276

Closed
jonpas opened this issue Feb 9, 2020 · 2 comments
Closed

Pre-processor comment slash error with include #276

jonpas opened this issue Feb 9, 2020 · 2 comments
Labels
bug Something isn't working

Comments

@jonpas
Copy link
Collaborator

jonpas commented Feb 9, 2020

HEMTT Version: 0.7.3-897d451-debug
Project: CBATeam/CBA_A3@b445594

Description:

Pre-processor fails when slash / is used in a comment which is in the same line as #include pre-processor instruction. If comment is on it's own line all works ok.

Steps to reproduce:

  • Add this line in some pre-processed file: #include "script_component.hpp" // /.
  • Observe error.

Additional information:

Works fine with #define and other pre-processor instructions. Also works fine with some other text. Pre-processor might possibly be trying to calculate a math expression?

HEMTT Output:

[jonpas@loki] $ ./hemttdev build jam                                                                                                          (hemtt) ~/Data/Work/Arma 3/Mods/CBA_A3
HEMTT 0.7.3-897d451-debug
Environment: dev

warning: .gitignore is missing recommended value `keys/*`
warning: .gitignore is missing recommended value `.hemtt`
warning: deprecated value `sig_name` in `hemtt.toml` - use `authority`
warning: Unable to locate binarize.exe
    Files will be packed as is
♻️ Clean          [00:00:00] [##############################] [1|1]
🔍 Checks         [00:00:00] [##############################] [1|1]
🚧 Prebuild       [00:00:00] [##############################] [1|1]
error: Unable to preprocess `CfgMagazineWells.hpp`
    Preprocessor Parse: class CfgMagazineWells {
    
<snip>

    #include "magwells_357SIG.hpp"                  
    #include "magwells_380ACP.hpp"                  
    #include "magwells_38Spec.hpp" 
    #include "magwells_38_200.hpp"                  // .38/200 | 9x20mmR
    #include "magwells_40SW.hpp"    
                
<snip>

error: Unable to build `addons/jam`

Comment with slash is left, others are correctly removed.

@jonpas jonpas added the bug Something isn't working label Feb 9, 2020
@jonpas
Copy link
Collaborator Author

jonpas commented Feb 9, 2020

Last blocker for current CBA use, ref.: CBATeam/CBA_A3#1145 (also test branch)

@BrettMayson
Copy link
Owner

closed with 477e44c

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants