comparison base/ResourceFinder.cpp @ 705:66de0ad10bb3

Remove out-of-context comment
author Chris Cannam
date Fri, 07 Oct 2011 17:03:04 +0100
parents 1424aa29ae95
children 9a0272c2d596
comparison
equal deleted inserted replaced
704:32773072e4af 705:66de0ad10bb3
284 if (!file.copy(target)) { 284 if (!file.copy(target)) {
285 std::cerr << "ResourceFinder::unbundleResource: ERROR: Failed to un-bundle resource file \"" << fileName << "\" to user location \"" << target << "\"" << std::endl; 285 std::cerr << "ResourceFinder::unbundleResource: ERROR: Failed to un-bundle resource file \"" << fileName << "\" to user location \"" << target << "\"" << std::endl;
286 return false; 286 return false;
287 } 287 }
288 288
289 // Now since the file is in the user's editable space, the user should get
290 // to edit it. The chords.xml file I unbundled came out 444 instead of 644
291 // which won't do. Rather than put the chmod code there, I decided to put
292 // it here, because I think it will always be appropriate to make unbundled
293 // files editable. That's rather the point in many cases, and for the rest,
294 // nobody will likely notice they could have edited their font files or what
295 // have you that were unbundled to improve performance. (Dissenting
296 // opinions welcome. We can always shuffle this somewhere else if
297 // necessary. There are many possibilities.)
298 QFile chmod(target); 289 QFile chmod(target);
299 chmod.setPermissions(QFile::ReadOwner | 290 chmod.setPermissions(QFile::ReadOwner |
300 QFile::ReadUser | /* for potential platform-independence */ 291 QFile::ReadUser | /* for potential platform-independence */
301 QFile::ReadGroup | 292 QFile::ReadGroup |
302 QFile::ReadOther | 293 QFile::ReadOther |