<div dir="ltr">Hello,<br>I noticed that there's this new commit (b8fb6d896a234eaa06) which removes<br>explicit initialization from check_xroutes() in xroute.c.<br><br>Personally I don't think it's a great idea because the initial values of the<br>variables involved are going to be undefined now, which makes the code depend on<br>whether someone correctly initializes each variable - especially given that<br>`struct filter_result` has a char pointer in it. An uninitialized `struct<br>filter_result` would likely bypass all null checks on that field. Finally, the<br>performance improvement from eliminating what probably will become a single<br>`mov` instruction is negligible.<br><br>Thanks,<br>Stanislaw<br><br></div>