Skip to content

Remove duplicate function prototypes in ae.h#41

Open
tomerqodo wants to merge 1 commit into
unstablefrom
tomerqodo-patch-1
Open

Remove duplicate function prototypes in ae.h#41
tomerqodo wants to merge 1 commit into
unstablefrom
tomerqodo-patch-1

Conversation

@tomerqodo

Copy link
Copy Markdown

No description provided.

@qodo-code-review

Copy link
Copy Markdown

Review Summary by Qodo

Consolidate duplicate function prototypes in ae.h

🐞 Bug fix

Grey Divider

Walkthroughs

Description
• Removed duplicate function prototypes from ae.h
• Relocated aeDeleteEventLoop and aeStop declarations
• Consolidated prototypes to single definition location
Diagram
flowchart LR
  A["Original: Duplicate prototypes<br/>at lines 97-98"] -- "Remove duplicates" --> B["Relocated to<br/>lines 114-115"]
  B -- "Result" --> C["Single definition<br/>per prototype"]

Loading

Grey Divider

File Changes

1. src/ae.h 🐞 Bug fix +2/-2

Consolidate duplicate function prototypes

• Removed duplicate aeDeleteEventLoop and aeStop function prototypes from their original
 location
• Relocated these two prototypes to the end of the prototype section after aeSetDontWait
• Eliminated code duplication while maintaining all function declarations

src/ae.h


Grey Divider

Qodo Logo

@qodo-code-review

qodo-code-review Bot commented Jun 3, 2026

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

Qodo Logo

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant