1:length constructs

[bpbond]

I'm looking at the code for layout_as_dynamic and see many loops similar to this:

  for(i in 1:length(gList)){

This is risky in R, because if gList is an empty list, the loop will execute twice!

> for(i in 1:length(list())) print(i)
[1] 1
[1] 0

In general it's safer to use R's seq_along.

Noted as part of openjournals/joss-reviews#5238

[schochastics]

This has been on my last far too long, thanks for calling it out.