[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Quilt-dev] [PATCH 23/39] backup-files: Optimize noop
From: |
Jean Delvare |
Subject: |
[Quilt-dev] [PATCH 23/39] backup-files: Optimize noop |
Date: |
Sat, 19 Mar 2011 10:52:53 +0100 |
User-agent: |
quilt/0.48-17.1 |
Optimize the noop operation. The only use case in quilt is to ensure
that files have a link count of 1, so we can check that OPT_NOLINKS
is set once and for all, instead of checking again with every file.
Signed-off-by: Jean Delvare <address@hidden>
Reviewed-by: Raphael Hertzog <address@hidden>
---
quilt/scripts/backup-files.in | 13 ++++++++++---
1 file changed, 10 insertions(+), 3 deletions(-)
--- a/quilt/scripts/backup-files.in
+++ b/quilt/scripts/backup-files.in
@@ -151,18 +151,17 @@ remove()
rmdir -p "${backup%/*}" 2> /dev/null || true
}
-noop()
+noop_nolinks()
{
local file=$1
- if [ -e "$file" ] && [ -n "$OPT_NOLINKS" ]; then
+ if [ -e "$file" ]; then
ensure_nolinks "$file"
fi
}
ECHO=echo
-OPT_WHAT=noop
while [ $# -gt 0 ]; do
case $1 in
-b) OPT_WHAT=backup
@@ -210,6 +209,14 @@ if [ "$1" = - ] && [ ! -d "$OPT_PREFIX"
exit 1
fi
+if [ -z "$OPT_WHAT" ]; then
+ if [ -n "$OPT_NOLINKS" ]; then
+ OPT_WHAT=noop_nolinks
+ else
+ exit 0
+ fi
+fi
+
if [ -n "$OPT_FILE" ]; then
cat "$OPT_FILE" \
| while read nextfile; do
- [Quilt-dev] [PATCH 25/39] backup-files: Separate function restore_all, (continued)
- [Quilt-dev] [PATCH 25/39] backup-files: Separate function restore_all, Jean Delvare, 2011/03/19
- [Quilt-dev] [PATCH 26/39] backup-files: Quick exit when unlinking is not needed, Jean Delvare, 2011/03/19
- [Quilt-dev] [PATCH 13/39] backup-files: Split code to sparate functions, Jean Delvare, 2011/03/19
- [Quilt-dev] [PATCH 24/39] backup-files: Avoid negations, Jean Delvare, 2011/03/19
- [Quilt-dev] [PATCH 14/39] backup-files: Mass reindent, Jean Delvare, 2011/03/19
- [Quilt-dev] [PATCH 01/39] Make backup-files a shell script, Jean Delvare, 2011/03/19
- [Quilt-dev] [PATCH 22/39] backup-files: Check if backup exists, Jean Delvare, 2011/03/19
- [Quilt-dev] [PATCH 12/39] backup-files: Simplify find loop, Jean Delvare, 2011/03/19
- [Quilt-dev] [PATCH 15/39] backup-files: Optimize restore, Jean Delvare, 2011/03/19
- [Quilt-dev] [PATCH 23/39] backup-files: Optimize noop,
Jean Delvare <=
- [Quilt-dev] [PATCH 27/39] backup-files: Remember the list of files, Jean Delvare, 2011/03/19
- [Quilt-dev] [PATCH 17/39] backup-files: Drop variable filelist, Jean Delvare, 2011/03/19
- [Quilt-dev] [PATCH 09/39] backup-files: Inline copy_file and link_or_copy_file, Jean Delvare, 2011/03/19
- [Quilt-dev] [PATCH 19/39] backup-files: Use internal implementation of dirname, Jean Delvare, 2011/03/19
- [Quilt-dev] [PATCH 33/39] backup-files: Try mass copy first on copy, Jean Delvare, 2011/03/19
- [Quilt-dev] [PATCH 05/39] Clean up legacy C code, Jean Delvare, 2011/03/19
- [Quilt-dev] [PATCH 11/39] backup-files: Prefix must be a directory, Jean Delvare, 2011/03/19
- [Quilt-dev] [PATCH 30/39] backup-files: Skip first mkdir on restore, Jean Delvare, 2011/03/19
- [Quilt-dev] [PATCH 29/39] backup-files: Try mass link/copy first on restore, Jean Delvare, 2011/03/19
- [Quilt-dev] [PATCH 36/39] backup-files: Drop support for -L on restore, Jean Delvare, 2011/03/19