Problem/Motivation

Code cleanup.

  • Depreciated methods
  • Missing doc for the function headers.
  • Unused code

Proposed resolution

  • Remove depreciated methods
  • Add doc for the function headers.
  • Remove unused code

Remaining tasks

patch, review and fix.

CommentFileSizeAuthor
#4 code_clean_up-2746139-1.patch1.04 KBfornigam

Comments

neerajskydiver created an issue. See original summary.

neerajskydiver’s picture

neerajskydiver commented
Assigned: Unassigned » neerajskydiver
Status: Active » Needs work
neerajskydiver’s picture

neerajskydiver commented
Assigned: neerajskydiver » Unassigned
Status: Needs work » Needs review
StatusFileSize
new 2746139-3.patch5.66 KB
fornigam’s picture

fornigam commented
Title: Code clean up. » Code clean up. README.txt
StatusFileSize
new code_clean_up-2746139-1.patch1.04 KB

Hi

I have update the README.txt for module installation steps. Please review and commit it.

-Nigam

anish.a’s picture

anish.a commented
Title: Code clean up. README.txt » Code clean up.
Status: Needs review » Fixed

neerajskydiver, your patch is merged.

@fornigam, Your patch doesn't make sense. Module enabling steps don't need elaborations. Please don't hide other's patches, unless your's superseeds the other's.

anish_zyxware’s picture

anish_zyxware commented
fornigam’s picture

fornigam commented

Agree Anish, Readme.txt is not mandate but it would one kind of help to beginner. Thanks.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.