GCC 4.3 warning for mitab

[rouault]

mitab_utils.cpp:685: warning: deprecated conversion from string constant to ‘char*’ mitab_utils.cpp:685: warning: deprecated conversion from string constant to ‘char*’ mitab_utils.cpp:685: warning: deprecated conversion from string constant to ‘char*’ mitab_utils.cpp:685: warning: deprecated conversion from string constant to ‘char*’ mitab_utils.cpp:685: warning: deprecated conversion from string constant to ‘char*’ mitab_utils.cpp:685: warning: deprecated conversion from string constant to ‘char*’ mitab_utils.cpp:685: warning: deprecated conversion from string constant to ‘char*’ mitab_utils.cpp:685: warning: deprecated conversion from string constant to ‘char*’ mitab_utils.cpp:685: warning: deprecated conversion from string constant to ‘char*’ mitab_utils.cpp:685: warning: deprecated conversion from string constant to ‘char*’ mitab_utils.cpp:685: warning: deprecated conversion from string constant to ‘char*’ mitab_utils.cpp:685: warning: deprecated conversion from string constant to ‘char*’ mitab_utils.cpp:685: warning: deprecated conversion from string constant to ‘char*’

[mloskot]

Patch fixing compilation warning about string literal to non-const char* conversion

[dmorissette]

mloskot, what is the difference between what you used in the patch

  char const* pszAbbrev;

and the more familiar form below?

  const char * pszAbbrev;

Could we not use the second one instead to avoid confusion or is there really a difference between the two?

[mloskot]

These two have the same effect. I just use the following

T const* p;

as more readable convention than

const T* p;

[dmorissette]

I've used the 'const char * pszAbbrev;' form since that's what is used everywhere else in MITAB.

Committed to MITAB CVS upstream, and sync'd GDAL SVN trunk with current dev version of MITAB in r14983.