05:55:22 Is there a place except for github and here, like a newsgroup server, where one can ask questions and discuss solutions? Big fan of threaded discussions and easy access from CLI... 06:23:45 boa, such threaded discussions are typically done ... via PRs. 06:24:26 That's why I myself don't over invest time in them, because there are usually many different opinions. 06:51:00 Ack, but PRs are for code changes, right? What I had in mind were developer-to-developer questions and discussions. For example, I see that parts of the code could need some love, like lmdb and other externals. Also, clang-tidy produces lots of output (via the health script you wrote and I like). Another example, the code allocates memory to report "out of memory" situations, which may be less than ideal 06:51:06 ;-) Where can we discuss questions like this? 07:06:52 Thanks boa . Yes, this is a good point you're raising. But still for what I know, PRs are kinds of best ways here to prove that a concept works, and that the tests pass. Github offers DRAFT PR, which suits this purpose perfectly. 07:07:19 I'll give you an example, why you should never over invest time, especially when it comes to Linting: 07:07:35 https://github.com/monero-project/monero/pull/7480/files 07:07:42 https://github.com/monero-project/monero/pull/7480 07:09:58 You may also open an Issue on Github, if you only have an idea, or request. 07:10:08 Maybe this would be a better starting point. 07:13:00 I can do that instead. Thanks, I appreciate you helping out a noob here :) 07:13:34 n00b helps n00b 07:18:13 selsta, could Adaptive Mining be next? 7618 07:18:32 So that we finally don't have to ignore the test results. 07:18:57 and thanks luigi1111 for the quick merge. 07:20:19 boa, once the Issue is created and discussed, and you think it's time for a PR, it's best to start the description with "Fixes [link-to-issue]". This will link the Issue with the PR. 07:22:05 And if you like my health script, I'm just generating a range of UT coverage reports. It will take a day I think. 08:06:31 mj-xmr I've added a few comments on 7480 08:07:49 Thanks. 09:01:49 Why does the Saviour of NASA take a group achievement award and present it as a proof of individual glory? twitter.com/hyc_symas/status/1203709575226183683 09:38:35 mj-xmr: The "Fixes [link-to-issue]" is nice, but what if there is no issue except for one I've found in the source code? Should I add the issue to github first, and then add the PR? I'm asking because I fear it may be a lot of "placeholder" issues, e.g. if one starts to clean up the output from the clang-tidy script. 09:45:05 Only actual problems, not "change code because clang whines" please. 09:45:33 I mean, it whines on preposterous shit at times. 09:49:17 True, like gcc's -Wconversion. Nlks 09:49:43 (meh, pressed ENTER too soon.) 09:51:08 What I tried to write, was that -Wconversion is a noisy little fellow, but that occationally it warns about something important. It's easier to see the important stuff if it isn't hidden in lots of noise, IME 09:55:57 (figured out the rebase thing, btw. ) 09:56:47 What is the rebase thing ? 09:59:43 Just a question I asked earlier. Should've googled it first. My bad. 11:11:13 boa, regarding what moneromooo said: Monero has a serious bottleneck in reviewing power, because there are much worse problems to solve than the potential ones. So Clang-Tidy warnings have a low priority. 11:11:39 That's why I will refrain from doing random checks from now. 11:16:43 ack. /me traces uses of malloc(), strdup(), getenv() and friends, just to get to know the code base and see if there are issues. There are issues... Not sure if it's just me being a code nazi or if the issues are real. It's fun, though, been missing nit-picking :) 11:18:06 :) 11:18:23 Leaks are definitely problems. As are buffer overflows. 11:18:36 Though some leaks may be ignored if they're one shot. 11:20:44 It's more like failing to test for null returns, and failure to sanitize external data (getenv()). Also quite a few instances of failure to check for error returns from functions, even invoking UB. Mostly in the external directory. 11:22:51 These could be problems. malloc will usually not return NULL for OOM, but blow up later when trying to map. 11:22:51 Checking error returns is usually a good idea, so feel free. 11:23:05 External stuff is probably best sent upstream (though easylogging got abandoned). 11:42:46 boa, http://enjo.hopto.org/pub/monero/data/5b8d714a4/5b8d714a4-unit-tests-lcov.tar.xz 11:42:57 UTest coverage for master. 11:43:24 Full report with some history will be here about tomorrow: http://enjo.hopto.org/pub/monero/ 11:43:38 I hope it helps. 12:02:56 boa, to answer your "issue" question. They are only needed if you needed a "threaded" discussion/. 12:04:13 * boa is a big fan of threaded discussions. Old enough to remember usenet, unfortunately... 13:15:38 ok, boomer :) 13:19:10 :) 14:05:21 Hmm, I see that the submodules aren't forked automatically. Oh, dear me, what have I done? 14:06:20 git submodule init/sync/update 14:08:49 yes, did that, but the remote is still the original monero-project repo instead of my forked repo. No biggie though, just wanted others to know that this may be a problem 14:24:15 no need to fork the submodules usually 14:33:35 ack. I'm just messing around in the unbound project. The code needs some love... 14:37:01 the unbound submodule would ideally get bumped to the latest upstream release but that would require some cmake changes 14:41:20 08:18 selsta, could Adaptive Mining be next? 7618 <-- yes if no one else leaves any blocking comments 15:15:06 Howard, you know why all the 'Titanic intelligence Saviour of NASA' posts stopped last year? They know you are an embarrassment. Your arse didn't learn anything new in 25 years, and that includes C++. They just can't say it to your face. 15:50:29 boa, since we're talking about memory, what I'd prefer to see, is make_unique() instead of malloc. Firstly it will deallocate the memory on exception and will self throw on misallocation. Also it will hold nullptr initially, which is better than some random location. 15:50:46 if this is not C code, that is. 16:17:21 does anyone know how to get the size of a specific transaction in bytes/kbytes from Daemon RPC? I don't see an output for that anywhere. I'm trying the get_transactions method https://web.getmonero.org/resources/developer-guides/daemon-rpc.html#get_transactions 16:20:12 Doesn't seem to be in the RPC response. Easy to add if needed. 16:20:39 Oh. You can get the dump of it. Divide length by 2. 16:34:28 I see, thx 18:48:09 HI. https://havovesu.page.link/PocUxdgkJRdpz2cB6 18:48:44 why even attempt with such suspicious looking links lol 18:57:50 To check if your Noscript still works :) 18:58:03 http://enjo.hopto.org/pub/monero/ <--- finally ready with LCov reports. 21:00:12 -xmr-pr- hyc opened pull request #7622: ITS#9500 fix regression from ITS#8662 21:00:12 -xmr-pr- > https://github.com/monero-project/monero/pull/7622 21:05:43 ^ please also open against release branch if relevant 21:07:12 ok 21:09:10 done 21:15:12 -xmr-pr- hyc opened pull request #7623: ITS#9500 fix regression from ITS#8662 [release-v0.17] 21:15:12 -xmr-pr- > https://github.com/monero-project/monero/pull/7623 22:26:31 I think everyone here knows that most Monero exchange withdrawals and desposits are traceable (Breaking Monero - poisoned outputs). How does it feel to go out and lie to people that they are private and then earn less than holding BTC on your bag?