Skip to content

Make delete calls not look like function calls. #2298

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

Merged

Conversation

hzeller
Copy link
Contributor

@hzeller hzeller commented Apr 26, 2023

Minor purely textual cleanup.

@hzeller hzeller requested a review from jmah76 April 26, 2023 20:49
@github-actions github-actions bot added the VPR VPR FPGA Placement & Routing Tool label Apr 26, 2023
@vaughnbetz
Copy link
Contributor

Needs a make format. It's an odd formatting request, but best to do it anyway:

  •    delete[] *pb_graph_pins;
    
  •    delete[] * pb_graph_pins;
    

If the argument to the delete keyword is parenthesized,
it looks like a function call, which is confusing while reading.

Signed-off-by: Henner Zeller <[email protected]>
@hzeller hzeller force-pushed the 20230426-delete-not-function branch from 8f6c026 to 187308d Compare April 27, 2023 17:02
@hzeller
Copy link
Contributor Author

hzeller commented Apr 27, 2023

Done.

@vaughnbetz vaughnbetz merged commit 5557dce into verilog-to-routing:master Apr 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
VPR VPR FPGA Placement & Routing Tool
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants