Merge pull request 'pifile_readall' (#190) from pifile_readall into master

Reviewed-on: #190

libs/main/io_devices/pifile.cpp

@@ -22,6 +22,7 @@
 #include "pidir.h"
 #include "piincludes_p.h"
 #include "piiostream.h"
+#include "piliterals_bytes.h"
 #include "pitime_win.h"
 #include "pitranslator.h"
 #ifdef WINDOWS
@@ -251,28 +252,18 @@ llong PIFile::readAll(void * data) {
 }
 
 
-PIByteArray PIFile::readAll(bool forceRead) {
-	PIByteArray a;
-	llong cp = pos();
-	if (forceRead) {
+PIByteArray PIFile::readAll() {
+	if (!isOpened()) return {};
+	llong prev_pos = pos();
+	PIByteArray ret, buffer(4_KiB);
 	seekToBegin();
 	for (;;) {
-			uchar byte = static_cast<uchar>(fgetc(PRIVATE->fd));
-			if (feof(PRIVATE->fd) || ferror(PRIVATE->fd)) break;
-			a.push_back(byte);
+		size_t readed = fread(buffer.data(), 1, buffer.size(), PRIVATE->fd);
+		if (readed == 0) break;
+		ret.append(buffer.data(), readed);
 	}
-		seek(cp);
-		return a;
-	}
-	llong s = size();
-	if (s < 0) return a;
-	a.resize(s);
-	seekToBegin();
-	auto _r = fread(a.data(), 1, s, PRIVATE->fd);
-	NO_UNUSED(_r);
-	seek(cp);
-	if (s >= 0) a.resize(s);
-	return a;
+	seek(prev_pos);
+	return ret;
 }
 
 
@@ -631,10 +622,10 @@ bool PIFile::applyFileInfo(const PIString & path, const PIFile::FileInfo & info)
 }
 
 
-PIByteArray PIFile::readAll(const PIString & path, bool forceRead) {
+PIByteArray PIFile::readAll(const PIString & path) {
 	PIFile f(path, PIIODevice::ReadOnly);
 	if (!f.isOpened()) return PIByteArray();
-	return f.readAll(forceRead);
+	return f.readAll();
 }
 
 

libs/main/io_devices/pifile.h

@@ -225,8 +225,7 @@ public:
 
 	//! \~english Read all file content to byte array and return it. Position leaved unchanged
 	//! \~russian Читает всё содержимое файла и возвращает его как массив байтов. Позиция остаётся неизменной
-	PIByteArray readAll(bool forceRead = false);
-
+	PIByteArray readAll();
 
 	//! \~english Set file path to "path" and reopen file if need
 	//! \~russian Устанавливает путь файла на "path" и переоткрывает его при необходимости
@@ -295,7 +294,7 @@ public:
 
 	//! \~english Read all file content at path "path" to byte array and return it.
 	//! \~russian Читает всё содержимое файла по пути "path" и возвращает его как массив байтов.
-	static PIByteArray readAll(const PIString & path, bool forceRead = false);
+	static PIByteArray readAll(const PIString & path);
 
 	//! \~english Clear file at path "path" and write "data", returns written bytes.
 	//! \~russian Очищает файл по пути "path", пишет туда "data" и возвращает количество записанных байт.

libs/main/system/pisysteminfo.cpp

@@ -234,7 +234,7 @@ PIString PISystemInfo::machineKey() {
 		PISystemInfo * si = instance();
 		PIByteArray salt;
 		PIString conf = confDir() + "/.pip_machine_salt";
-		if (PIFile::isExists(conf)) salt = PIFile::readAll(conf, false);
+		if (PIFile::isExists(conf)) salt = PIFile::readAll(conf);
 		if (salt.size_s() != SALT_SIZE) {
 			salt = generateSalt();
 			PIFile::writeAll(conf, salt);

main.cpp

@@ -83,6 +83,15 @@ void piDeserializeJSON(S & v, const PIJSON & js) {
 PIKbdListener kbd;
 
 int main(int argc, char * argv[]) {
+	if (argc < 2) return 0;
+	PIFile f(argv[1], PIIODevice::ReadOnly);
+	piCout << "read" << f.path();
+	auto fc = f.readAll();
+	piCout << fc.size();
+	if (!fc.isEmpty()) piCout << PIString::fromUTF8(fc.resized(32));
+	return 0;
+
+
 	PIHTTPServer server;
 	server.listen({"127.0.0.1:7777"});
 	// server.setBasicAuthRealm("pip");
@@ -102,11 +111,13 @@ int main(int argc, char * argv[]) {
 	};
 	// });
 
-	reg("/*/3/get");
-	reg("/*/{ID}/get");
+	reg(" /*/a/get ");
+	reg(" /*/{ID}/get ");
+	reg(" /*/*/get ");
+	reg(" /*/*_b/get ");
+
 	reg("/api/{ID}/get");
 	reg("/api/1/get");
-	server.unregisterPath("*/{ID}/get");
 	// reg("/api/1/bort{bortID}/get");
 	// reg("/api/**");
 	// reg("/api/1/bort2/get");
@@ -164,7 +175,7 @@ int main(int argc, char * argv[]) {
 	// return 0;
 	PIRegularExpression pire("(?:\\/\\/\\s*)?.*\\n?(?:\\bfunction\\b)\\s*(?<name>\\b\\w+\\b)\\s*(?:\\((?<args>[^;()]*?)\\))",
 							 PIRegularExpression::Multiline);
-	PIString subj = PIString::fromUTF8(PIFile::readAll("telegram.qs", false));
+	PIString subj = PIString::fromUTF8(PIFile::readAll("telegram.qs"));
 
 	piCout << "Pattern:" << pire.pattern();
 	piCout << "Valid:" << pire.isValid();

utils/deploy_tool/main.cpp

@@ -463,12 +463,15 @@ bool procDpkg(const PIString & l) {
 	fi.path      = l;
 	PIString cmd = dpkg + dpkgdir + " -S " + fi.name() + ign_err_suffix;
 	// PICout(true) << cmd;
-	PIString vs  = execute(cmd);
+	PIString vs  = execute(cmd).trim();
 	if (!vs.isEmpty()) {
-		vs = vs.left(vs.find(":"));
-		if (!vs.isEmpty() && !vs.endsWith("-cross")) all_deps << vs;
+		PIStringList lines = vs.split('\n').reverse();
+		for (auto l: lines) {
+			l = l.left(l.find(":"));
+			if (!l.isEmpty() && !l.endsWith("-cross")) all_deps << vs;
 			return true;
 		}
+	}
 	// piCout << "No dep on" << l;
 	return false;
 }
@@ -787,7 +790,16 @@ int main(int argc, char * argv[]) {
 	out_dir.replaceAll("/", "\\");
 #endif
 	PIVector<PIString> clibs = all_libs.toVector();
+	static PIStringList ignore_libs({"libc.so"});
 	for (auto l: clibs) {
+		bool ignore_lib = false;
+		for (auto il: ignore_libs) {
+			if (l.startsWith(il)) {
+				ignore_lib = true;
+				break;
+			}
+		}
+		if (ignore_lib) continue;
 		PIFile::FileInfo fi;
 		fi.path = l;
 #ifdef WINDOWS