Skip to content

Commit 247261b

Browse files
committed
Minor cleanup to avoid extraneous permutations
1 parent 97b4369 commit 247261b

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

src/boundary_conditions.jl

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,6 @@ end
3333

3434
function get_interparticle_distance(ri, rj, pbc::PeriodicBoundaryConditions)
3535
rij = ri - rj
36-
r, r2 = zero(eltype(ri)), zero(eltype(ri))
3736
x, y, z = rij
3837
while x < pbc[1] x += pbc[2]-pbc[1] end
3938
while x >= pbc[2] x -= pbc[2]-pbc[1] end
@@ -42,7 +41,7 @@ function get_interparticle_distance(ri, rj, pbc::PeriodicBoundaryConditions)
4241
while z < pbc[5] z += pbc[6]-pbc[5] end
4342
while z >= pbc[6] z -= pbc[6]-pbc[5] end
4443
rij = @SVector [x, y, z]
45-
r2 = dot(rij, rij)
44+
r2 = rij[1]^2 + rij[2]^2 + rij[3]^2
4645
r = sqrt(r2)
4746
return (rij, r, r2)
4847
end
@@ -59,14 +58,14 @@ function get_interparticle_distance(ri, rj, bc::CubicPeriodicBoundaryConditions)
5958
while z >= radius z -= size end
6059
while z < -radius z += size end
6160
rij = @SVector [x, y, z]
62-
r2 = dot(rij, rij)
61+
r2 = rij[1]^2 + rij[2]^2 + rij[3]^2
6362
r = sqrt(r2)
6463
return (rij, r, r2)
6564
end
6665

6766
function get_interparticle_distance(ri, rj, ::BoundaryConditions)
6867
rij = ri - rj
69-
r2 = dot(rij, rij)
68+
r2 = rij[1]^2 + rij[2]^2 + rij[3]^2
7069
r = sqrt(r2)
7170
(rij, r, r2)
7271
end

0 commit comments

Comments
 (0)