bugGNU Astronomy Utilities - Bugs: bug #48692, Replace TABs with spaces

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #48692: Replace TABs with spaces

Submitted by:  Vladimir Markelov <vvm>
Submitted on:  Wed 03 Aug 2016 07:57:09 PM UTC  
 
Category: All GnuastroSeverity: 3 - Normal
Item Group: Source codeStatus: Fixed
Privacy: PublicAssigned to: Mohammad Akhlaghi <makhlaghi>
Open/Closed: Closed

Thu 04 Aug 2016 11:07:09 AM UTC, comment #1:

Thank you very much for highlighting this issue! Since the no TABs convention wasn't adopted from the exact start of Gnuastro, some of the older files and functions (and even code pieces that are repeated a lot) actually used TABs for indentation. For those who use Emacs, the '.dir-locals.el' file will be loaded and will not allow the use of TABs for indentation on new code.

This problem was haunting me for a very long time (ever since the no-TABs convention was adopted). I always knew that there is probably code that still uses TABs for indentation, but I would always have more important things to do. So now, I thought of putting the necessary hour or two to go through all the '.c' and '.h' files in 'lib/' and 'src/' and making the correction. It was finally completed and all the changes have been pushed to master.

Thanks again Vladimir for bringing up this issue, and also thanks to Mosè for writing the '.dir-locals.el' file so Emacs users don't have to worry about this issue any more.

Mohammad Akhlaghi <makhlaghi>
Project AdministratorIn charge of this item.
Wed 03 Aug 2016 07:57:09 PM UTC, original submission:

Imgcrop is an old application and its source contains mix of spaces and TAB characters. Need to beautify the code and eliminate all TABs

Vladimir Markelov <vvm>
Project Member

 

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by makhlaghi (Posted a comment)
  • -unavailable- added by vvm (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 4 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Thu 04 Aug 2016 11:07:09 AM UTCmakhlaghiCategoryCrop=>All Gnuastro
      StatusNone=>Fixed
      Assigned toNone=>makhlaghi
      Open/ClosedOpen=>Closed

    Back to the top


    Powered by Savane 3.1-cleanup1