-
br-m
<jpk68:matrix.org> selsta: I might be missing something, but is there a reason I should include <string_view> if it compiles without it?
-
br-m
<jpk68:matrix.org> (responding to your review comment)
-
selsta
If you use it and it's not included in the .cpp or .h I'd include it
-
br-m
<vtnerd> It's useful for compatibility purposes. Omitting it is relying on particular libraries to indirectly include the feature
-
selsta
in the past we had builds failing suddenly because some compiler suddenly got more strict
-
br-m
<jpk68:matrix.org> Thanks, good to know
-
selsta
jpk68: there are open PRs regarding man pages, just pointing it out so that you don't duplicate work
-
br-m
<jpk68:matrix.org> I know, mine is for monero-wallet-cli specifically, not the daemon, as the others are for
-
br-m
<jpk68:matrix.org> Thanks for the heads-up, though :)
-
br-m
<jpk68:matrix.org> Not counting the one PR that had them converted with Pandoc, I mean
-
br-m
<jpk68:matrix.org> Alright, I pushed fixes and added a test case for incorrect version byte. The mirror might take a bit to get updated but that's fine
-
br-m
<jpk68:matrix.org> Somehow also just realized that every (valid) onion address ends with "d" :P
-
br-m
<jpk68:matrix.org> selsta: Would you mind uploading a pastebin of the build error for 10612?
-
br-m
<jpk68:matrix.org> It works on my machine
-
selsta
it can't find the boost header
-
br-m
-
br-m
<ofrnxmr:xmr.mx> Have you benched the improvement? Is it even worth the micro optinmiization?
-
br-m
<jpk68:matrix.org> No, but what would be the harm in changing it? It would clearly be much slower if you included the entirety of Asio everywhere
-
br-m
<ofrnxmr:xmr.mx> its not clear if it doesnt build and hasnt been compared
-
br-m
<jpk68:matrix.org> Well, that's strange... if you look on my fork, it's a different header
-
br-m
-
br-m
-
br-m
<ofrnxmr:xmr.mx> Maybe sent sneed a different patch?
-
br-m
<ofrnxmr:xmr.mx> Youre right that the prs dont match.
-
br-m
<jpk68:matrix.org> Pretty sure I didn't, I just gave him the link to my fork
-
br-m
<jpk68:matrix.org> Anyways, I pushed a fix for the other one and both should be updated soon
-
br-m
<sneedlewoods_xmr:matrix.org> yeah sorry, that was my fault
-
br-m
<ofrnxmr:xmr.mx> You're fired
-
br-m
<ofrnxmr:xmr.mx> what i do.. is i just add jpks remote, then checkout the branch, then push it as-is to my repo
-
br-m
<ofrnxmr:xmr.mx> To my remote*
-
br-m
<jpk68:matrix.org> Don't worry about it
-
br-m
<jpk68:matrix.org> I just wanted to let people know what was going on
-
br-m
<that-guy123:fedora.im> Jpk, i had rights to the merkle roots verification scheme before this group. How can we resolve this?
-
br-m
<that-guy123:fedora.im> It seems this is an underlying schema to the functionality of the coin cryptographic protocall
-
br-m
<jpk68:matrix.org> Apologies if this is rude, but I have no idea what you're talking about, I am not the right person to answer this question, it sounds like you're trolling, and this is the wrong channel anyways
-
br-m
<that-guy123:fedora.im> Ok maybe u are the wrong individual to inquire. Who made the game? Do u know or are aware?
-
br-m
<jpk68:matrix.org> #monero-community:monero.social
-
br-m
<that-guy123:fedora.im> Eh
-
br-m
<jpk68:matrix.org> Please move this convo to the channel I linked above. This is the wrong channel ;)
-
br-m
<that-guy123:fedora.im> What do u think of the situation?
-
br-m
<jpk68:matrix.org> Wrong channel
-
br-m
<that-guy123:fedora.im> Ok
-
br-m
<that-guy123:fedora.im> K
-
br-m
<jpk68:matrix.org> selsta: What do you mean by your comment?
-
br-m
<jpk68:matrix.org> Nothing seems off on my end
-
selsta
sneedle fixed it after my comment
-
br-m
<sneedlewoods_xmr:matrix.org> sorry for the inconvenience, today's not my day, to my excuse I need to sleep
-
br-m
<jpk68:matrix.org> DW, not a big problem
-
br-m
<jpk68:matrix.org> Alright, looks like all three are good to go now
-
br-m
<jpk68:matrix.org> vtnerd: WIN32 is apparently defined in <windows.h>, or by MinGW. In any case, I'm fairly certain it's not by us
-
br-m
-
br-m
<vtnerd> Then why the switch to _WIN32 ?
-
br-m
<jpk68:matrix.org> Because it can be defined falsely (by the user or otherwise), and requires headers
-
br-m
<jpk68:matrix.org> This is explained in the post I linked to above
-
br-m
<jpk68:matrix.org> On the other hand, _WIN32 is strictly compiler-defined and can't be messed with