Refactor Event Processing Logic into Separate Function

[stav-bentov]

Encapsulated loop logic into a dedicated function processSingleFileEvent to isolate responsibilities and simplify the main code flow.

[codecov]

Codecov Report

Attention: Patch coverage is 80.95238% with 4 lines in your changes missing coverage. Please review.

Project coverage is 71.01%. Comparing base (9071a5c) to head (8f5655b).
Report is 9 commits behind head on unstable.

Files with missing lines	Patch %	Lines
src/ae.c	80.95%	4 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##           unstable    #1635      +/-   ##
============================================
+ Coverage     70.85%   71.01%   +0.16%     
============================================
  Files           121      121              
  Lines         65169    65177       +8     
============================================
+ Hits          46176    46287     +111     
+ Misses        18993    18890     -103     

Files with missing lines	Coverage Δ
src/ae.c	78.27% <80.95%> (+0.62%)	⬆️

... and 18 files with indirect coverage changes

[madolson]

Suggested change

	static void processSingleFileEvent(
	aeEventLoop *eventLoop,
	aeFiredEvent *firedEvent) {
	static void processSingleFileEvent(aeEventLoop *eventLoop, aeFiredEvent *firedEvent) {

[zuiderkwast]

I don't think this change is necessary. I vote to not merge it.

[zuiderkwast]

Btw, hi Stav and thanks for contributing. :)

Sometimes not all maintainers agree. It can be a good idea to wait some time for more opinions and to discuss it, especially if you want to convince us about something important.