Skip to content

Commit

Permalink
Don't store file names separately
Browse files Browse the repository at this point in the history
  • Loading branch information
lalinsky committed Feb 7, 2011
1 parent cffef28 commit 099353b
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 7 deletions.
2 changes: 1 addition & 1 deletion run-tests.sh
@@ -1,4 +1,4 @@
#!/bin/sh

make && ./tests --gtest_filter="*$1*"
make && ./tests --gtest_print_time --gtest_filter="*$1*"

6 changes: 1 addition & 5 deletions src/store/ram_directory.cpp
Expand Up @@ -32,22 +32,19 @@ void RAMDirectory::close()

QStringList RAMDirectory::listFiles()
{
return m_names;
return m_data.keys();
}

void RAMDirectory::deleteFile(const QString &name)
{
if (!m_data.contains(name)) {
return;
}
m_names.removeAll(name);
delete m_data.take(name);
}

void RAMDirectory::renameFile(const QString &oldName, const QString &newName)
{
m_names.removeAll(oldName);
m_names.append(newName);
m_data.insert(newName, m_data.take(oldName));
}

Expand All @@ -64,7 +61,6 @@ OutputStream *RAMDirectory::createFile(const QString &name)
{
QByteArray *data = new QByteArray();
m_data.insert(name, data);
m_names.append(name);
return new RAMOutputStream(data);
}

Expand Down
1 change: 0 additions & 1 deletion src/store/ram_directory.h
Expand Up @@ -42,7 +42,6 @@ class RAMDirectory : public Directory
QStringList listFiles();

private:
QStringList m_names;
QHash<QString, QByteArray*> m_data;
};

Expand Down

0 comments on commit 099353b

Please sign in to comment.