00:10:09 Got this comment to change from CamelCase (FeePriority) to snakeCase (feePriority) for an enum. I don;'t see a standard naming convention. Some code uses `this_convention` too. 00:10:10 What's the correct approach? vtnerd 00:10:12 https://github.com/monero-project/monero/pull/9838#discussion_r2085036133 00:12:29 I might be wrong but feePriority isn't snake case, fee_priority is 00:12:34 i'm pretty sure snakeCase is actually snake_case 00:12:43 yeah exactly 00:12:47 Jinx 00:13:10 and CamelCase is actually camelCase 00:13:32 oh wait no im wrong on this one nvm 00:13:50 ah ok depends there is lower camel case and upper camel case 00:15:17 Some people call thisStyle "Pascal case" 00:15:42 interesting 00:16:02 i suppose this is a psyop/plot from the Pascal community, these guys are more fanatics than rustaceans 00:16:25 No wait I screwed that up . Uppercase camel case = Pascal case 00:16:56 don't want some good kebab-case 00:17:09 🥙 00:28:24 Oh derp 00:28:35 Ok, will change to that. I confused pascal and snake. 00:29:29 naming convention would be cool 00:29:50 almost forgot with Rust that C++ don't particularly enforce one. 00:33:53 Snake case classes and enumerations just feels dirty to me. 00:35:27 agree 15:58:09 This pr was updated 16:00:16 9838 (for irc). Precursor to 9847 (increase auto fee to lvl 3 if over 12hrs backlog at greater than lvl 2) 20:52:27 ofrnxmr: when are you learning C++ to help? 20:52:49 apologies i thought i was in #monero 20:55:07 Yesterday 20:55:49 great 20:55:53 first PR for monero when? 20:57:18 Already merged 20:59:33 2nd* already, first coauthored 9494 20:59:33 Another closed in favor of 9847 20:59:33 I stay in my lane 22:16:19 Hello! I am looking for a list of IPs of spy nodes. Does this list exist?. I would like to dinamically update this list using an nftables set. Will this work? Thanks! 22:18:31 sure, there is https://github.com/Boog900/monero-ban-list/blob/main/ban_list.txt and https://github.com/rblaine95/monero-banlist/blob/master/block.txt. 22:18:48 and i think the later included the first but im not sure. boog900 ? 22:19:24 also didn't know you could establish a dynamic set of addresses with nftables, sounds awesome. 22:19:27 mostly I think there are a couple not 22:20:00 well then only the first 22:21:18 you can still use the later, its only like 2 IPs. The later includes IPs of more malicious nodes not just the current spys. 22:21:54 Excellent. No need to use a firewall, the daemon supports it directly. Even better. Good job guys 🙌 22:24:03 I will give a try tomorrow. It might be more efficient than adding it to the daemon. 22:24:34 I'm certain it will much more INefficient 22:25:00 You mean with nftables more inneficient? 22:25:03 adding the ban list will just prohibit these nodes from connecting to you or you trying to connect to them. if you set nftable it will do this check at every ip connections 22:25:06 yes 22:25:49 Makes sense