Skip to content

Conversation

@FabrizioSandri
Copy link
Owner

In this pull request I add some additional improvements to the actual commenting system.

@FabrizioSandri
Copy link
Owner Author

FabrizioSandri commented Jul 29, 2022

The last few commits fixes the problem that the output of capture.output(dput(some.data)) can be a vector. Additionally I removed the inputs column since the inputs for the test are located in the R_code column.

Fixes tdhock/binsegRcpp#13 (comment)

@github-actions
Copy link

github-actions bot commented Jul 29, 2022

RcppDeepState Report

function_name message file_line address_trace R_code
rcpp_read_out_of_bound Invalid read of size 4 read_out_of_bound.cpp:7 No Address Trace found
Test code
testlist <- list(rbound = -1707129401L)
result <- do.call(testSAN::rcpp_read_out_of_bound, testlist)
rcpp_use_after_deallocate Invalid read of size 1 use_after_deallocate.cpp:8 use_after_deallocate.cpp:6
Test code
testlist <- list(array_size = -1332783554L)
result <- do.call(testSAN::rcpp_use_after_deallocate, testlist)
rcpp_use_after_free Argument 'size' of function malloc has a fishy (possibly negative) value: -8580860900 use_after_free.cpp:6 No Address Trace found
Test code
testlist <- list(alloc_size = -994750403L)
result <- do.call(testSAN::rcpp_use_after_free, testlist)
rcpp_write_index_outofbound Invalid write of size 4 write_index_outofbound.cpp:8 No Address Trace found
Test code
testlist <- list(wbound = -1541987514L)
result <- do.call(testSAN::rcpp_write_index_outofbound, testlist)
rcpp_zero_sized_array Invalid write of size 4 zero_sized_array.cpp:8 zero_sized_array.cpp:7
Test code
testlist <- list(value = -1910784704L)
result <- do.call(testSAN::rcpp_zero_sized_array, testlist)

@FabrizioSandri
Copy link
Owner Author

FabrizioSandri commented Jul 29, 2022

In addition, I fixed the issue with links that contained two concatenations of the github.com string.

I discovered that this problem was due to the gsub command that is used to remove the occurrences of two ore more / symbols in hyperlinks, ending up removing them also from https://.

Fixes tdhock/binsegRcpp#13 (comment)

@FabrizioSandri FabrizioSandri merged commit 1aaca1b into main Jul 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants