I've been doing some work on ctools, improving function docblocks and tests, and got 'sidetracked' into some views code doing the same thing.

There are two parts to the patch:
- comments and source code style (line too long etc) and inclusion of 'public' access setting;
- some small improvements, which I have pulled out for ease of checking.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

rivimey created an issue. See original summary.

rivimey’s picture

Both of these patch files apply independently to current 7.x-3.x-dev git.

There is a small conflict between them which is easily resolved; I won't confuse matters further with a combined patch (yet).

DamienMcKenna’s picture

Assigned: rivimey » Unassigned
Status: Active » Needs review