From 17d5667e86828fbfbe142542e83dffce08b43121 Mon Sep 17 00:00:00 2001
From: Pino Toscano <pino@kde.org>
Date: Mon, 27 Aug 2007 15:44:00 +0000
Subject: [PATCH] give CamelCase names to the values of the document type enum

svn path=/trunk/KDE/kdeedu/libkdeedu/; revision=705258
---
 keduvocdocument/keduvocdocument.cpp | 40 ++++++++++++++---------------
 keduvocdocument/keduvocdocument.h   | 21 ++++++++-------
 keduvocdocument/tests/converter.cpp |  2 +-
 3 files changed, 33 insertions(+), 30 deletions(-)

diff --git a/keduvocdocument/keduvocdocument.cpp b/keduvocdocument/keduvocdocument.cpp
index fe92dd0..d27d731 100644
--- a/keduvocdocument/keduvocdocument.cpp
+++ b/keduvocdocument/keduvocdocument.cpp
@@ -174,7 +174,7 @@ KEduVocDocument::FileType KEduVocDocument::detectFileType(const QString &fileNam
 {
   QIODevice * f = KFilterDev::deviceForFile(fileName);
   if (!f->open(QIODevice::ReadOnly))
-    return csv;
+    return Csv;
 
   QDataStream is(f);
 
@@ -199,25 +199,25 @@ KEduVocDocument::FileType KEduVocDocument::detectFileType(const QString &fileNam
   line.prepend(c1);
 
   if (!is.device()->isOpen())
-    return kvd_none;
+    return KvdNone;
 
   f->close();
   if (c1 == '<' && c2 == '?' && c3 == 'x' && c4 == 'm' && c5 == 'l') {
     if (line2.indexOf("pauker", 0) >  0)
-      return pauker;
+      return Pauker;
     else if (line2.indexOf("xdxf", 0) >  0)
-      return xdxf;
+      return Xdxf;
     else
-      return kvtml;
+      return Kvtml;
   }
 
   if (line == WQL_IDENT)
-    return wql;
+    return Wql;
 
   if (c1 == '"' && (line.contains('"') == 1 || line.contains(QRegExp("\",[0-9]"))))
-    return vokabeln;
+    return Vokabeln;
 
-  return csv;
+  return Csv;
 }
 
 
@@ -243,7 +243,7 @@ bool KEduVocDocument::open(const KUrl& url)
     FileType ft = detectFileType(url.path());
 
     switch (ft) {
-      case kvtml:
+      case Kvtml:
       {
         KEduVocKvtml2Reader kvtmlReader(f);
         read = kvtmlReader.readDoc(this);
@@ -252,7 +252,7 @@ bool KEduVocDocument::open(const KUrl& url)
       }
       break;
 
-      case wql:
+      case Wql:
       {
         KEduVocWqlReader wqlReader(f);
         d->m_url.setFileName(i18n("Untitled"));
@@ -262,7 +262,7 @@ bool KEduVocDocument::open(const KUrl& url)
       }
       break;
 
-      case pauker:
+      case Pauker:
       {
         KEduVocPaukerReader paukerReader(this);
         d->m_url.setFileName(i18n("Untitled"));
@@ -272,7 +272,7 @@ bool KEduVocDocument::open(const KUrl& url)
       }
       break;
 
-      case vokabeln:
+      case Vokabeln:
       {
         KEduVocVokabelnReader vokabelnReader(f);
         d->m_url.setFileName(i18n("Untitled"));
@@ -282,7 +282,7 @@ bool KEduVocDocument::open(const KUrl& url)
       }
       break;
 
-      case csv:
+      case Csv:
       {
         KEduVocCsvReader csvReader(f);
         read = csvReader.readDoc(this);
@@ -291,7 +291,7 @@ bool KEduVocDocument::open(const KUrl& url)
       }
       break;
 
-      case xdxf:
+      case Xdxf:
       {
         KEduVocXdxfReader xdxfReader(this);
         d->m_url.setFileName(i18n("Untitled"));
@@ -326,12 +326,12 @@ bool KEduVocDocument::saveAs(const KUrl & url, FileType ft, const QString & gene
 {
   KUrl tmp(url);
 
-  if (ft == automatic)
+  if (ft == Automatic)
   {
     if (tmp.path().right(strlen("." KVTML_EXT)) == "." KVTML_EXT)
-      ft = kvtml;
+      ft = Kvtml;
     else if (tmp.path().right(strlen("." CSV_EXT)) == "." CSV_EXT)
-      ft = csv;
+      ft = Csv;
     else
     {
       return false;
@@ -351,20 +351,20 @@ bool KEduVocDocument::saveAs(const KUrl & url, FileType ft, const QString & gene
     }
 
     switch (ft) {
-      case kvtml: {
+      case Kvtml: {
 		// write version 2 file
         KEduVocKvtml2Writer kvtmlWriter(&f);
         saved = kvtmlWriter.writeDoc(this, generator);
       }
       break;
-	  case kvtml1: {
+	  case Kvtml1: {
 		// write old version 1 file
 		KEduVocKvtmlWriter kvtmlWriter(&f);
 		saved = kvtmlWriter.writeDoc(this, generator);
 	    }
 		break;
 /** @todo include the csv write again, as soon as it's ported to the new classes
-      case csv: {
+      case Csv: {
         KEduVocCsvWriter csvWriter(&f);
         saved = csvWriter.writeDoc(this, generator);
       }
diff --git a/keduvocdocument/keduvocdocument.h b/keduvocdocument/keduvocdocument.h
index 1873eed..0c53ed6 100644
--- a/keduvocdocument/keduvocdocument.h
+++ b/keduvocdocument/keduvocdocument.h
@@ -44,15 +44,18 @@ class KEDUVOCDOCUMENT_EXPORT KEduVocDocument : public QObject
   Q_OBJECT
 public:
 
-  enum FileType { kvd_none,
-                  automatic,
-                  kvtml,
-                  wql,
-                  pauker,
-                  vokabeln,
-                  xdxf,
-                  csv,
-				  kvtml1 };
+  enum FileType
+  {
+    KvdNone,
+    Automatic,
+    Kvtml,
+    Wql,
+    Pauker,
+    Vokabeln,
+    Xdxf,
+    Csv,
+    Kvtml1
+  };
 
   /**
    * Constructor for a KdeEdu vocabulary document
diff --git a/keduvocdocument/tests/converter.cpp b/keduvocdocument/tests/converter.cpp
index 364d1c0..44386fe 100644
--- a/keduvocdocument/tests/converter.cpp
+++ b/keduvocdocument/tests/converter.cpp
@@ -57,7 +57,7 @@ int main (int argc, char ** argv)
       kDebug() << "Reading " << infile;
       document.open(infile);
       kDebug() << "Writing to " << outfile;
-      document.saveAs(outfile, KEduVocDocument::kvtml, "converter");
+      document.saveAs(outfile, KEduVocDocument::Kvtml, "converter");
     }
   }
   
-- 
2.47.3