solenv/bin/modules/installer/remover.pm        |    8 -
 solenv/bin/modules/par2script/converter.pm     |   64 ---------
 solenv/bin/modules/par2script/parameter.pm     |    1 
 solenv/bin/modules/par2script/remover.pm       |   24 ---
 solenv/bin/modules/par2script/systemactions.pm |  170 -------------------------
 solenv/bin/modules/pre2par/files.pm            |   10 -
 solenv/bin/modules/pre2par/parameter.pm        |   14 +-
 solenv/bin/modules/pre2par/remover.pm          |   24 ---
 solenv/bin/modules/pre2par/systemactions.pm    |  111 ----------------
 solenv/bin/modules/pre2par/work.pm             |   39 -----
 solenv/bin/pre2par.pl                          |    6 
 11 files changed, 11 insertions(+), 460 deletions(-)

New commits:
commit a04bb13513ea64fef867365d6eb0264970d58182
Author:     Christian Lohmaier <lohmaier+libreoff...@googlemail.com>
AuthorDate: Thu May 27 15:40:41 2021 +0200
Commit:     Christian Lohmaier <lohmaier+libreoff...@googlemail.com>
CommitDate: Thu May 27 19:10:19 2021 +0200

    installer-code: remove some unused or pointless subs
    
    of course just the very tip of the iceberg, but you gotta start
    somewhere…
    
    Change-Id: I24ef6a51ec70c70418368da54595637283055d84
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/116255
    Tested-by: Jenkins
    Reviewed-by: Christian Lohmaier <lohmaier+libreoff...@googlemail.com>

diff --git a/solenv/bin/modules/installer/remover.pm 
b/solenv/bin/modules/installer/remover.pm
index 1e088a63e0a1..426056eef37f 100644
--- a/solenv/bin/modules/installer/remover.pm
+++ b/solenv/bin/modules/installer/remover.pm
@@ -40,14 +40,6 @@ sub remove_leading_and_ending_quotationmarks
     $$stringref =~ s/\"\s*$//;
 }
 
-sub remove_leading_and_ending_slashes
-{
-    my ( $stringref ) = @_;
-
-    $$stringref =~ s/^\s*\///g;
-    $$stringref =~ s/\/\s*$//g;
-}
-
 sub remove_ending_pathseparator
 {
     my ( $stringref ) = @_;
diff --git a/solenv/bin/modules/par2script/converter.pm 
b/solenv/bin/modules/par2script/converter.pm
index d8323f40dc37..43c3a751f14a 100644
--- a/solenv/bin/modules/par2script/converter.pm
+++ b/solenv/bin/modules/par2script/converter.pm
@@ -25,29 +25,6 @@ use par2script::remover;
 # Converter
 #############################
 
-sub convert_array_to_hash
-{
-    my ($arrayref) = @_;
-
-    my ($line, $key, $value);
-
-    my %newhash = ();
-
-    for ( my $i = 0; $i <= $#{$arrayref}; $i++ )
-    {
-        $line = ${$arrayref}[$i];
-
-        if ( $line =~ /^\s*(\w+?)\s+(.*?)\s*$/ )
-        {
-            $key = $1;
-            $value = $2;
-            $newhash{$key} = $value;
-        }
-    }
-
-    return \%newhash;
-}
-
 sub convert_stringlist_into_array_2
 {
     my ( $input, $separator ) = @_;
@@ -72,45 +49,4 @@ sub convert_stringlist_into_array_2
     return \@newarray;
 }
 
-sub convert_stringlist_into_array
-{
-    my ( $includestringref, $separator ) = @_;
-
-    my @newarray = ();
-    my ($first, $last);
-
-    $last = ${$includestringref};
-
-    while ( $last =~ /^\s*(.+?)\s*\Q$separator\E\s*(.+)\s*$/)   # "$" for 
minimal matching
-    {
-        $first = $1;
-        $last = $2;
-        par2script::remover::remove_leading_and_ending_whitespaces(\$first);
-        push(@newarray, $first);
-    }
-
-    par2script::remover::remove_leading_and_ending_whitespaces(\$last);
-    push(@newarray, $last);
-
-    return \@newarray;
-}
-
-#############################################################################
-# The file name contains for some files "/". If this programs runs on
-# a windows platform, this has to be converted to "\".
-#############################################################################
-
-sub convert_slash_to_backslash
-{
-    my ($filesarrayref) = @_;
-
-    my ($onefile, $filename);
-
-    for ( my $i = 0; $i <= $#{$filesarrayref}; $i++ )
-    {
-        $onefile = ${$filesarrayref}[$i];
-        $onefile->{'Name'} =~ s/\//\\/g;
-    }
-}
-
 1;
diff --git a/solenv/bin/modules/par2script/parameter.pm 
b/solenv/bin/modules/par2script/parameter.pm
index 645b28255686..86416cd9e265 100644
--- a/solenv/bin/modules/par2script/parameter.pm
+++ b/solenv/bin/modules/par2script/parameter.pm
@@ -22,7 +22,6 @@ package par2script::parameter;
 use Cwd;
 use par2script::files;
 use par2script::globals;
-use par2script::systemactions;
 
 ############################################
 # Parameter Operations
diff --git a/solenv/bin/modules/par2script/remover.pm 
b/solenv/bin/modules/par2script/remover.pm
index 4625bf0b0405..8b40a8ff46f7 100644
--- a/solenv/bin/modules/par2script/remover.pm
+++ b/solenv/bin/modules/par2script/remover.pm
@@ -39,28 +39,4 @@ sub remove_leading_and_ending_comma
     $$stringref =~ s/\,\s*$//g;
 }
 
-sub remove_leading_and_ending_quotationmarks
-{
-    my ( $stringref ) = @_;
-
-    $$stringref =~ s/^\s*\"//g;
-    $$stringref =~ s/\"\s*$//g;
-}
-
-sub remove_leading_and_ending_slashes
-{
-    my ( $stringref ) = @_;
-
-    $$stringref =~ s/^\s*\///g;
-    $$stringref =~ s/\/\s*$//g;
-}
-
-sub remove_leading_and_ending_backslashes
-{
-    my ( $stringref ) = @_;
-
-    $$stringref =~ s/^\s*\\//g;
-    $$stringref =~ s/\\\s*$//g;
-}
-
 1;
diff --git a/solenv/bin/modules/par2script/systemactions.pm 
b/solenv/bin/modules/par2script/systemactions.pm
deleted file mode 100644
index b6e6b546b8c5..000000000000
--- a/solenv/bin/modules/par2script/systemactions.pm
+++ /dev/null
@@ -1,170 +0,0 @@
-#
-# This file is part of the LibreOffice project.
-#
-# This Source Code Form is subject to the terms of the Mozilla Public
-# License, v. 2.0. If a copy of the MPL was not distributed with this
-# file, You can obtain one at http://mozilla.org/MPL/2.0/.
-#
-# This file incorporates work covered by the following license notice:
-#
-#   Licensed to the Apache Software Foundation (ASF) under one or more
-#   contributor license agreements. See the NOTICE file distributed
-#   with this work for additional information regarding copyright
-#   ownership. The ASF licenses this file to you under the Apache
-#   License, Version 2.0 (the "License"); you may not use this file
-#   except in compliance with the License. You may obtain a copy of
-#   the License at http://www.apache.org/licenses/LICENSE-2.0 .
-#
-
-
-package par2script::systemactions;
-
-use File::Copy;
-use par2script::exiter;
-use par2script::globals;
-
-######################################################
-# Creating a new directory
-######################################################
-
-sub create_directory
-{
-    my ($directory) = @_;
-
-    my $returnvalue = 1;
-
-    if (!(-d $directory))
-    {
-        $returnvalue = mkdir($directory, 0775);
-
-        if ($returnvalue)
-        {
-            $infoline = "Created directory: $directory\n";
-            push(@par2script::globals::logfileinfo, $infoline);
-
-            if ($par2script::globals::isunix)
-            {
-                my $localcall = "chmod 775 $directory \>\/dev\/null 2\>\&1";
-                system($localcall);
-            }
-        }
-        else
-        {
-            par2script::exiter::exit_program("Error: Could not create 
directory: $directory", "create_directory");
-        }
-    }
-}
-
-#######################################################################
-# Creating the directories, in which files are generated or unzipped
-#######################################################################
-
-sub create_directories
-{
-    my ($directory, $languagesref) =@_;
-
-    $par2script::globals::unpackpath =~ 
s/\Q$par2script::globals::separator\E\s*$//;    # removing ending slashes and 
backslashes
-
-    my $path = $par2script::globals::unpackpath;     # this path already exists
-
-    $path = $path . $par2script::globals::separator . 
$par2script::globals::build . $par2script::globals::separator;
-    create_directory($path);
-
-    $path = $path . $par2script::globals::minor . 
$par2script::globals::separator;
-    create_directory($path);
-
-    if ($directory eq "unzip" )
-    {
-    }
-    else
-    {
-        $path = $path . $par2script::globals::platformid . 
$par2script::globals::separator;
-        create_directory($path);
-
-        $path = $path . $par2script::globals::product . 
$par2script::globals::separator;
-        create_directory($path);
-
-        $path = $path . $directory . $par2script::globals::separator;
-        create_directory($path);
-
-        if (!($$languagesref eq "" ))   # this will be a path like "01_49", 
for Profiles and ConfigurationFiles, idt-Files
-        {
-            $path = $path . $$languagesref . $par2script::globals::separator;
-            create_directory($path);
-        }
-    }
-
-    $path =~ s/\Q$par2script::globals::separator\E\s*$//;
-
-    return $path;
-}
-
-########################
-# Copying one file
-########################
-
-sub copy_one_file
-{
-    my ($source, $dest) = @_;
-
-    my ($copyreturn, $returnvalue);
-    my $infoline;
-
-    $copyreturn = copy($source, $dest);
-
-    if ($copyreturn)
-    {
-        $infoline = "Copy: $source to $dest\n";
-        $returnvalue = 1;
-    }
-    else
-    {
-        $infoline = "Error: Could not copy $source to $dest $!\n";
-        $returnvalue = 0;
-    }
-
-    push(@par2script::globals::logfileinfo, $infoline);
-
-    return $returnvalue;
-}
-
-##########################################
-# Copying all files from one directory
-# to another directory
-##########################################
-
-sub copy_directory
-{
-    my ($sourcedir, $destdir) = @_;
-
-    my ($onefile, $sourcefile, $destfile);
-    my @sourcefiles = ();
-
-    $sourcedir =~ s/\Q$par2script::globals::separator\E\s*$//;
-    $destdir =~ s/\Q$par2script::globals::separator\E\s*$//;
-
-    $infoline = "\n";
-    push(@par2script::globals::logfileinfo, $infoline);
-    $infoline = "Copying files from directory $sourcedir to directory 
$destdir\n";
-    push(@par2script::globals::logfileinfo, $infoline);
-
-    opendir(DIR, $sourcedir);
-    @sourcefiles = readdir(DIR);
-    closedir(DIR);
-
-    foreach $onefile (@sourcefiles)
-    {
-        if ((!($onefile eq ".")) && (!($onefile eq "..")))
-        {
-            $sourcefile = $sourcedir . $par2script::globals::separator . 
$onefile;
-            $destfile = $destdir . $par2script::globals::separator . $onefile;
-            if ( -f $sourcefile )   # only files, no directories
-            {
-                copy_one_file($sourcefile, $destfile);
-            }
-        }
-    }
-}
-
-
-1;
diff --git a/solenv/bin/modules/pre2par/files.pm 
b/solenv/bin/modules/pre2par/files.pm
index af216e11a148..ddc8d9157eae 100644
--- a/solenv/bin/modules/pre2par/files.pm
+++ b/solenv/bin/modules/pre2par/files.pm
@@ -24,16 +24,6 @@ use pre2par::exiter;
 # File Operations
 ############################################
 
-sub check_file
-{
-    my ($arg) = @_;
-
-    if(!( -f $arg ))
-    {
-        pre2par::exiter::exit_program("ERROR: Cannot find file $arg", 
"check_file");
-    }
-}
-
 sub read_file
 {
     my ($localfile) = @_;
diff --git a/solenv/bin/modules/pre2par/parameter.pm 
b/solenv/bin/modules/pre2par/parameter.pm
index 08c2d7cd02ff..aa9fe3399a57 100644
--- a/solenv/bin/modules/pre2par/parameter.pm
+++ b/solenv/bin/modules/pre2par/parameter.pm
@@ -20,7 +20,6 @@
 package pre2par::parameter;
 
 use Cwd;
-use pre2par::files;
 use pre2par::globals;
 use pre2par::systemactions;
 
@@ -89,6 +88,15 @@ sub control_parameter
         exit(-1);
     }
 
+    if (!(-f $pre2par::globals::prefilename))
+    {
+        print "\n************************************************\n";
+        print "Error: Input file does not exist!";
+        print "\n************************************************\n";
+        usage();
+        exit(-1);
+    }
+
     if ($pre2par::globals::parfilename eq "")
     {
         print "\n************************************************\n";
@@ -115,10 +123,6 @@ sub control_parameter
         usage();
         exit(-1);
     }
-
-    # The input file has to exist
-
-    pre2par::files::check_file($pre2par::globals::prefilename);
 }
 
 #####################################
diff --git a/solenv/bin/modules/pre2par/remover.pm 
b/solenv/bin/modules/pre2par/remover.pm
index aa90ec4a56c4..029d09a4e419 100644
--- a/solenv/bin/modules/pre2par/remover.pm
+++ b/solenv/bin/modules/pre2par/remover.pm
@@ -31,28 +31,4 @@ sub remove_leading_and_ending_whitespaces
     $$stringref =~ s/\s*$//g;
 }
 
-sub remove_leading_and_ending_quotationmarks
-{
-    my ( $stringref ) = @_;
-
-    $$stringref =~ s/^\s*\"//g;
-    $$stringref =~ s/\"\s*$//g;
-}
-
-sub remove_leading_and_ending_slashes
-{
-    my ( $stringref ) = @_;
-
-    $$stringref =~ s/^\s*\///g;
-    $$stringref =~ s/\/\s*$//g;
-}
-
-sub remove_leading_and_ending_backslashes
-{
-    my ( $stringref ) = @_;
-
-    $$stringref =~ s/^\s*\\//g;
-    $$stringref =~ s/\\\s*$//g;
-}
-
 1;
diff --git a/solenv/bin/modules/pre2par/systemactions.pm 
b/solenv/bin/modules/pre2par/systemactions.pm
index c5f5e44f5f41..7848b64950ec 100644
--- a/solenv/bin/modules/pre2par/systemactions.pm
+++ b/solenv/bin/modules/pre2par/systemactions.pm
@@ -78,115 +78,4 @@ sub create_directory
     }
 }
 
-#######################################################################
-# Creating the directories, in which files are generated or unzipped
-#######################################################################
-
-sub create_directories
-{
-    my ($directory, $languagesref) =@_;
-
-    $pre2par::globals::unpackpath =~ s/\Q$pre2par::globals::separator\E\s*$//; 
 # removing ending slashes and backslashes
-
-    my $path = $pre2par::globals::unpackpath;    # this path already exists
-
-    $path = $path . $pre2par::globals::separator . $pre2par::globals::build . 
$pre2par::globals::separator;
-    create_directory($path);
-
-    $path = $path . $pre2par::globals::minor . $pre2par::globals::separator;
-    create_directory($path);
-
-    if ($directory eq "unzip" )
-    {
-    }
-    else
-    {
-        $path = $path . $pre2par::globals::platformid . 
$pre2par::globals::separator;
-        create_directory($path);
-
-        $path = $path . $pre2par::globals::product . 
$pre2par::globals::separator;
-        create_directory($path);
-
-        $path = $path . $directory . $pre2par::globals::separator;
-        create_directory($path);
-
-        if (!($$languagesref eq "" ))   # this will be a path like "01_49", 
for Profiles and ConfigurationFiles, idt-Files
-        {
-            $path = $path . $$languagesref . $pre2par::globals::separator;
-            create_directory($path);
-        }
-    }
-
-    $path =~ s/\Q$pre2par::globals::separator\E\s*$//;
-
-    return $path;
-}
-
-########################
-# Copying one file
-########################
-
-sub copy_one_file
-{
-    my ($source, $dest) = @_;
-
-    my ($copyreturn, $returnvalue, $infoline);
-
-    $copyreturn = copy($source, $dest);
-
-    if ($copyreturn)
-    {
-        $infoline = "Copy: $source to $dest\n";
-        $returnvalue = 1;
-    }
-    else
-    {
-        $infoline = "Error: Could not copy $source to $dest $!\n";
-        $returnvalue = 0;
-    }
-
-    push(@pre2par::globals::logfileinfo, $infoline);
-
-    return $returnvalue;
-}
-
-##########################################
-# Copying all files from one directory
-# to another directory
-##########################################
-
-sub copy_directory
-{
-    my ($sourcedir, $destdir) = @_;
-
-    my ($onefile, $sourcefile, $destfile);
-    my @sourcefiles = ();
-
-    $sourcedir =~ s/\Q$pre2par::globals::separator\E\s*$//;
-    $destdir =~ s/\Q$pre2par::globals::separator\E\s*$//;
-
-    $infoline = "\n";
-    push(@pre2par::globals::logfileinfo, $infoline);
-    $infoline = "Copying files from directory $sourcedir to directory 
$destdir\n";
-    push(@pre2par::globals::logfileinfo, $infoline);
-
-    opendir(DIR, $sourcedir);
-    @sourcefiles = readdir(DIR);
-    closedir(DIR);
-
-    foreach $onefile (@sourcefiles)
-    {
-        if ((!($onefile eq ".")) && (!($onefile eq "..")))
-        {
-            $sourcefile = $sourcedir . $pre2par::globals::separator . $onefile;
-            $destfile = $destdir . $pre2par::globals::separator . $onefile;
-            if ( -f $sourcefile )   # only files, no directories
-            {
-                copy_one_file($sourcefile, $destfile);
-            }
-        }
-    }
-}
-
-
 1;
diff --git a/solenv/bin/modules/pre2par/work.pm 
b/solenv/bin/modules/pre2par/work.pm
index b05940c441b2..6ee87ec3786b 100644
--- a/solenv/bin/modules/pre2par/work.pm
+++ b/solenv/bin/modules/pre2par/work.pm
@@ -271,45 +271,6 @@ sub formatter
     }
 }
 
-###################################################
-# Returning the language file name
-###################################################
-
-sub getlangfilename
-{
-    return $pre2par::globals::langfilename;
-}
-
-############################################
-# Checking if a file exists
-############################################
-
-sub fileexists
-{
-    my ($langfilename) = @_;
-
-    my $fileexists = 0;
-
-    if( -f $langfilename ) { $fileexists = 1; }
-
-    return $fileexists;
-}
-
-############################################
-# Checking the existence of ulf file
-############################################
-
-sub check_existence_of_langfiles
-{
-    my ($langfilename) = @_;
-
-    my $do_localize = 0;
-
-    if ( fileexists($langfilename) ) { $do_localize = 1; }
-
-    return $do_localize;
-}
-
 ############################################
 # Checking that the pre file has content
 ############################################
diff --git a/solenv/bin/pre2par.pl b/solenv/bin/pre2par.pl
index 42c319dd2c08..28ca73312dc7 100644
--- a/solenv/bin/pre2par.pl
+++ b/solenv/bin/pre2par.pl
@@ -43,11 +43,9 @@ my $parfile = pre2par::work::convert($prefile);
 
 pre2par::work::formatter($parfile);
 
-my $langfilename = pre2par::work::getlangfilename();
+my $langfilename = $pre2par::globals::langfilename;
 
-my $dolocalization = 
pre2par::work::check_existence_of_langfiles($langfilename);
-
-if ( $dolocalization )
+if ( -f $langfilename )
 {
     my $langfile = pre2par::files::read_file($langfilename);
     pre2par::language::localize($parfile, $langfile);
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to