Update the i/o naming convention to follow the update on the SWIFT side

pelahi / VELOCIraptor-STF

Galaxy/(sub)Halo finder for N-body simulations

MIT License

19 stars 26 forks source link

Update the i/o naming convention to follow the update on the SWIFT side #38

Closed MatthieuSchaller closed 5 years ago

MatthieuSchaller commented 5 years ago

Does what it says on the tin. :)

Just updated the constructor HDF_Part_Info() in hdfitems.h to use the new SWIFT names that have been in our master as of last week.

I have also taken the liberty to add comments in-between the different variable names to keep things a bit clearer.

Note that there is something wrong in the logic between the two if-statements on lines 929 and 940. As I don't know which should be kept, I have left things as they are.

rtobar commented 5 years ago

@MatthieuSchaller I added a couple of small changes related to whitespacing and logic compression; let me know if you agree, and if so I'll merge.

MatthieuSchaller commented 5 years ago

Thanks. All good for me.