-
homeslice
hi!
-
moneromooo
hi
-
homeslice
I'm a comp sci student and have also been trading monero for nearly 1 yr
-
homeslice
I would like to help out on any projects happening!
-
moneromooo
Welcome. What's your area of expertise, and what's your area of preference ?
-
homeslice
I would like to start with GUI stuff, and as far as expertise i'm somewhat experienced in C++/Objected oriented programming/Algorithm and data structures/web development but not any skills directly related to GUI. I would like to volunteer and to gain experience
-
moneromooo
Try asking in #monero-gui then.
-
homeslice
Thanks
-
sethsimmons
homeslice: this could be a good starting point for you as well:
sethsimmons.me/posts/contributing-to-monero/#for-developers
-
homeslice
sethsimmons I'll take a look into it, cheers
-
selsta
.merge+ 7371
-
xmr-pr
Added
-
Arkoo
I was looking through the database and saw that the string comparator for the properties table does not take size into account
-
Arkoo
when we do MDB_val_str(k, "pruning_seed");
-
Arkoo
the right hand argument is cast to a void *,that is (void * ) "pruning_seed"
-
Arkoo
In c++ is pruning seed automatically converted to a C-style string when we write something like this?
-
moneromooo
IIRC the zero is stored.
-
moneromooo
(for this particular table)
-
Arkoo
It has to be. But where does it happen do you know?
-
moneromooo
I'm not sure I understand what you're asking here.
-
Arkoo
where is the null terminator set
-
Arkoo
I wouldn't have thought it was done during the (void *) cast
-
Arkoo
ffs
-
Arkoo
void star
-
Arkoo
haha
-
moneromooo
I don't understnad what you're asking here.
-
moneromooo
The NUL terminator is set at the end of the string. Obviously. So that's probably not what you're asking.
-
moneromooo
The cast doesn't change this.
-
moneromooo
The cast is because the MDB_val data arg is a void* since it can point to anything.
-
Arkoo
c++ style strings don't have a null terminator. std::string temp = "hello whats up"; will not have a null termiator
-
Arkoo
if we simply write "some text" without writing std::string, is it interpreted as a C-style string with a null terminator?
-
moneromooo
It shuld have one since c_str needs one.
-
moneromooo
It's a C string.
-
Arkoo
right
-
Arkoo
thanks
-
moneromooo
You could make a custom compare function that strncmp, but lmdb will crash happily if you craft the db anyway, so it really doesn't matter I think.
-
moneromooo
Why is it people want to file a bug about expcetions all the bloody time if it's working...
-
moneromooo
It doens't show on the console for a reason...
-
mj-xmr
Arkoo: a C++ string does contain a NULL terminator. It's just not counted in with size().
-
mj-xmr
Comparing const char * with string's c_str() will work for strncmp
-
omega
is there anything 'wrong' with setting p2p-bind-port=80 on my home daemon? Thank you
-
ErCiccione
moneromooo: could you unlock xmronadaily on gitlab. It's the problem of the login with github.
-
moneromooo
Sure. I totally forgot about doing those rounds for ages.
-
ErCiccione
nobody had complained about it so far :)
-
moneromooo
Funny that. It asks for for 2FA, and I did not set that up.
-
moneromooo
ErCiccione: it will wait, I have login issues to the admin interface.
-
ErCiccione
Alright, thanks. I think serhack has access has well.
-
sgp_
If all goes well, this channel will be bridged to the Discord tonight. It was bridged before, but if you have concerns lmk
-
hyc
eh? there are actual developer on discord that aren't on irc?
-
binaryFate
Seems all large enough monero channels will get bridged. Should allow everyone to get persistence without bouncer or paying for irccloud. Easier for some people
-
sethsimmons
Matrix is much better and more private and should be the recommendation.
-
sethsimmons
But discord is good for normies
-
sethsimmons
Should only be RO to this channel though.
-
hyc
yeah RO here sounds like a good idea