(very) Minor const correctness change in dcdict.cxx

All other questions regarding DCMTK

Moderator: Moderator Team

Post Reply
Message
Author
okigan
Posts: 8
Joined: Mon, 2004-11-08, 22:31

(very) Minor const correctness change in dcdict.cxx

#1 Post by okigan » Tue, 2005-07-19, 00:06

Seems like

splitFields(const char* line, char* fields[], int maxFields, char splitChar)
{
char* p;

should be:
splitFields(const char* line, char* fields[], int maxFields, char splitChar)
{
const char* p;

VS2005 is being somewhat vocal on this.

Thanks for great work and regards,
Igor
Last edited by okigan on Wed, 2011-02-16, 15:55, edited 3 times in total.

Jörg Riesmeier
ICSMED DICOM Services
ICSMED DICOM Services
Posts: 2217
Joined: Fri, 2004-10-29, 21:38
Location: Oldenburg, Germany

#2 Post by Jörg Riesmeier » Tue, 2005-07-19, 10:03

Thank you for the report. This change has already been done on the CVS version:

Code: Select all

** $Log: dcdict.cc,v $
** Revision 1.33  2004/08/03 16:45:58  meichel
** Minor changes for platforms on which strchr/strrchr return a const pointer.

okigan
Posts: 8
Joined: Mon, 2004-11-08, 22:31

#3 Post by okigan » Tue, 2005-07-19, 19:30

Sorry for beating the dead horse.

I did check the that I am using the 3.5.3 release. Is anon-read-only CVS access available ?

Thanks,
Igor
Last edited by okigan on Wed, 2011-02-16, 15:55, edited 3 times in total.

Jörg Riesmeier
ICSMED DICOM Services
ICSMED DICOM Services
Posts: 2217
Joined: Fri, 2004-10-29, 21:38
Location: Oldenburg, Germany

#4 Post by Jörg Riesmeier » Wed, 2005-07-20, 10:37

Don't worry about that. I just wanted to inform you about the fact that we already solved the "problem".

In case you're interested to get a current CVS snapshot just send an email to dicom/at/offis/dot/de.

Post Reply

Who is online

Users browsing this forum: Bing [Bot] and 1 guest