Opened 10 years ago

Last modified 10 years ago

#13899 closed defect

Don't use TAB characters for indentation — at Version 2

Reported by: Jeroen Demeyer Owned by: Jason Grout
Priority: trivial Milestone: sage-5.6
Component: misc Keywords:
Cc: Merged in:
Authors: Reviewers:
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by Jeroen Demeyer)

  • TABs in Makefiles are needed, so keep them.
  • Some TABs not used for indentation are also kept.
  • TABs in the obsolete directory devel/sage/sage/server/notebook (#11409) are not fixed.
  • The directory sage/graphs/planarity_c uses a very inconsistent indentation scheme (mixing spaces and TABs and having varying amounts of indentation) is hopeless to easily fix automatically.
  • Also, make indentation consistent and/or remove trailing spaces in some places.

Change History (5)

comment:1 Changed 10 years ago by Jeroen Demeyer

Description: modified (diff)

comment:2 Changed 10 years ago by Jeroen Demeyer

Description: modified (diff)

Changed 10 years ago by Jeroen Demeyer

Attachment: 13899_TAB_sage_root.patch added

Changed 10 years ago by Jeroen Demeyer

Attachment: 13899_TAB_sagelib.patch added

Changed 10 years ago by Jeroen Demeyer

Attachment: 13899_TAB_scripts.patch added
Note: See TracTickets for help on using tickets.