Item9031: COSMETIC -- core files at least should have coherent structure

pencil
Priority: Urgent
Current State: Closed
Released In: 1.1.0
Target Release: minor
Applies To: Engine
Component:
Branches:
Reported By: OlivierRaginel
Waiting For: Main.OlivierRaginel
Last Change By: KennethLavrsen
Mainy core files have headers at the top, some at the end, some inside a DATA container...

Also, all core should use strict and use warnings, unless explicitly disabled or noted.

And while I'm at it, I'll perltidy them all once more.

Marking it as urgent as I do not want the next release to be a mess, but I'm hoping to close this one today.

-- OlivierRaginel - 17 May 2010

Ok, all the .pm files in core/ are done for the copyrights, all after an END marker, and all with a line at the top saying to check the bottom. Will see if use warnings doesn't make a huge mess in the unit tests before pushing.

-- OlivierRaginel - 17 May 2010

Added use strict and use warnings in all core .pm files including all core plugins. Copyright is also coherent in everything shipped now, unless I missed something.

Yes, need to do bin/ and tools/, but... Tomorrow? smile

-- OlivierRaginel - 05 Jun 2010

Ok, now only cleanup is needed, but I can close this urgent task as, if I'm not mistaken, everything that will be shipped is now OK.

-- OlivierRaginel - 13 Jun 2010

 
Topic revision: r18 - 04 Oct 2010, KennethLavrsen
The copyright of the content on this website is held by the contributing authors, except where stated elsewhere. See Copyright Statement. Creative Commons License    Legal Imprint    Privacy Policy