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 Gnuastro Severity:  3 - Normal
Item Group:  Source code Status:  Fixed
Privacy:  Public Assigned 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
  • -email is unavailable- added by makhlaghi (Posted a comment)
  • -email is unavailable- added by vvm (Submitted the item)

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

    Only logged-in users can vote.




    Follow 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2016-08-04 makhlaghi CategoryCrop => All Gnuastro
        StatusNone => Fixed
        Assigned toNone => makhlaghi
        Open/ClosedOpen => Closed

    Back to the top

    Powered by Savane 3.5