Opened 13 years ago

Closed 13 years ago

#3820 closed defect (fixed)

mapkml*.* code review - coding style fixes required

Reported by: dmorissette Owned by: assefa
Priority: normal Milestone: 6.0 release
Component: Output-KML Version: unspecified
Severity: normal Keywords:
Cc:

Description

On a quick code review, I noticed the following coding style issues in the mapkml*.cpp/.h files. It would be ideal to fix them ASAP before the 6.0 release so that eventual patches will be easier to port in the future between 6.0 and 6.x releases.

1- The files use a mix of TABs and spaces for indent. The convention in MapServer source is that all tabs should be converted to the right amount of spaces so that indent is right in any editor, no matter what the tab size setting is in the editor. (Most editors have a setting to automatically convert tabs to spaces to help avoiding this problem in the future)

2- The files are stored in DOS CR/LF format in the SVN repository. This is not mandatory, but while we're at it we could run dos2unix on them and commit new versions without the CR/LF

Note that emacs has macros to facilitate the fix of those issues, so most of the work after applying them should be reviewing the results for correctness. If you are not setup to do this easily then please let me know and I'll try to do it before next beta.

Change History (3)

comment:1 by dmorissette, 13 years ago

Did a svn propset that seems to have solved the DOS CR/LF issue. (r11500, r11501).

svn propset svn:eol-style native *.h *.c *.cpp

comment:2 by assefa, 13 years ago

I will review this before the next beta. I am setup to do the indentation/tab properly that with emacs. I will check all functions.

comment:3 by assefa, 13 years ago

Resolution: fixed
Status: newclosed

passed through the files and fixed tabs/spaces. Committed in r11505. Id tag committed r11506

Note: See TracTickets for help on using tickets.