search

heldentodd / xray-diffraction

This repository contains the code for one simulation, but eventually three are planned: bragg-law, Single Crystal Diffraction, and Powder Diffraction.
GNU General Public License v3.0
2 stars 1 forks source link

whitespace between the `//` and the first letter of the line comment #23

Closed pixelzoom closed 4 years ago

pixelzoom commented 4 years ago

Related to #1 (code review).

  • [ ] Line comments should have whitespace between the // and the first letter of the line comment. See the preceding example.

For example, in Lattice.js:

          //Find top, center atom and place it in sites[0]. sites[1,2,3] are all still the origin.
          //We only need to do this for max x and max y if we need to optimize. Could be separated out
          if((Math.abs(xCos - ySin) <= topRowSpacingHalf) && ((Math.abs(yCos - xSin) > this.sites[0].y))) {
heldentodd commented 4 years ago

Found several cases of these and added the space.