[task #15567] Converting hh:mm:ss and dd:mm:ss notations to degrees and vice vesa

[Mohammad Akhlaghi]

Follow-up Comment #55, task #15567 (project gnuastro):

Thanks Kartik. I pulled the changes but still see that the commit message
title is two lines! Please correct it to be a single and complete line that is
shorter than 75 characters (which can be readable with `git log --oneline').

In the two `gal_units_decimal_to_dec' and `gal_units_decimal_to_ra', I noticed
that you have defined the variables in the middle of the functions. This isn't
a technical problem in modern C compilers, but it is a bad style ;-)! It makes
it so so hard to read the code! 

It is much better if all the variables are defined at the top of the function,
before any operation is done. When reading a code, someone can simply pull to
the top of the function and see all the various types used and variables
defined (which is itself valuable information before going into the details).


In the same two functions, the last element (`seconds' and `arc_seconds') is
floating point: in many cases we have sub-arcsecond accuracy ;-).

    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/task/?15567>

_______________________________________________
  Message sent via Savannah
  https://savannah.gnu.org/