diff options
| author | jvoisin | 2012-12-27 16:31:29 +0100 |
|---|---|---|
| committer | jvoisin | 2012-12-27 16:31:29 +0100 |
| commit | f11ba1f694bca8626bce837c388fedb535cbe97d (patch) | |
| tree | 67ff0e25db16fe28e5a98815751333980a44a43c /MAT/mat.py | |
| parent | 4fdf80d7649fe168f16641c48d749819c9c56701 (diff) | |
Improve mat.py (mostly exceptions handling)
Diffstat (limited to 'MAT/mat.py')
| -rw-r--r-- | MAT/mat.py | 9 |
1 files changed, 3 insertions, 6 deletions
| @@ -92,13 +92,13 @@ def secure_remove(filename): | |||
| 92 | try: | 92 | try: |
| 93 | subprocess.call(['shred', '--remove', filename]) | 93 | subprocess.call(['shred', '--remove', filename]) |
| 94 | removed = True | 94 | removed = True |
| 95 | except: | 95 | except OSError: |
| 96 | logging.error('Unable to securely remove %s' % filename) | 96 | logging.error('Unable to securely remove %s' % filename) |
| 97 | 97 | ||
| 98 | if removed is False: | 98 | if not removed: |
| 99 | try: | 99 | try: |
| 100 | os.remove(filename) | 100 | os.remove(filename) |
| 101 | except: | 101 | except OSError: |
| 102 | logging.error('Unable to remove %s' % filename) | 102 | logging.error('Unable to remove %s' % filename) |
| 103 | 103 | ||
| 104 | 104 | ||
| @@ -149,7 +149,4 @@ def create_class_file(name, backup, **kwargs): | |||
| 149 | logging.info('Don\'t have stripper for %s format' % mime) | 149 | logging.info('Don\'t have stripper for %s format' % mime) |
| 150 | return None | 150 | return None |
| 151 | 151 | ||
| 152 | if mime.endswith('pdf') and mime.startswith('application/'): | ||
| 153 | return stripper_class(filename, parser, mime, backup, **kwargs) | ||
| 154 | |||
| 155 | return stripper_class(filename, parser, mime, backup, **kwargs) | 152 | return stripper_class(filename, parser, mime, backup, **kwargs) |
