Should the comparison be changed from <= to <, since there's no need to swap an element with itself, unless
the swap isn't important (i.e., it doesn't do any harm) at that point and the need is just to adjust both i and j variables?

// If the values are in the wrong order, swap them.
if( i <= j ){

Compared to
// ...
if( i < j ) {

Thanks,

Exactly: when i == j the swap does nothing, but adjusting the values of i and j is essential. You could modify the code to eliminate this wasted swap, but I'm not sure the increased complexity would be worth it.

John