Merge branch 'next'

Author: chardoncs

.github/qna.md

@@ -9,15 +9,23 @@ Since v0.2.0, *libpgfe* has been "self-sufficient" and does not depend on any th
 ## Q2: Interested in supporting Big Endian?
 
 I will think about it eventually, but not now! Because byte order compatibility will dramatically increase complexity of the code,
-and I currently don't have enough time or effort to tackle it. Also, I do not have Big Endian machines or virtual machines, so the additional code would be left untested, which is what I don't want to happen.
+and I currently don't have enough time or effort to tackle it. Also, I do not have Big Endian machines or virtual machines, so the additional code would be left untested, which is not what I expect.
 
-## Q3: Why not MSVC?
+## Q3: Why not [Assembly Language](https://en.wikipedia.org/wiki/Assembly_language)?
+
+1. I'm not familiar with Assembly.
+
+2. Assembly is machine-dependent.
+
+3. C is fast enough for *libpgfe*. It's no need to use Assembly to do those optimizations.
+
+## Q4: Why not MSVC?
 
 > ***PS:** MSVC = Microsoft Visual C++*
 
-Firstly, this library prioritizes POSIX compatibility, instead of Windows compatibility.
+1. This library prioritizes POSIX compatibility, instead of Windows compatibility.
 
-Secondly, *libpgfe* need some necessary features that are not included in *MSVC*. For instance, like the C code below:
+2. *libpgfe* needs some necessary features that are not included in *MSVC*. For instance, like the C code below. *MSVC* will refuse to compile that code, because a variable is used as the array's size, while *Clang* and *GCC* are OK with it.
 
 ```c
 int main() {
@@ -27,5 +35,3 @@ int main() {
     return 0;
 }
 ```
-
-*MSVC* will refuse to compile that code, because of a variable is used as the array's size, while *Clang* and *GCC* are OK with it.

CMakeLists.txt

@@ -3,7 +3,7 @@ cmake_minimum_required(VERSION 3.16.0)
 set(CMAKE_C_COMPILER clang)
 set(CMAKE_CXX_COMPILER clang++)
 
-project(libpgfe VERSION 0.4.0 LANGUAGES C CXX)
+project(libpgfe VERSION 0.5.0 LANGUAGES C CXX)
 
 set(CMAKE_C_STANDARD 11)
 set(CMAKE_CXX_STANDARD 14)
@@ -12,51 +12,53 @@ include(CTest)
 enable_testing()
 
 set(src_dir "src")
+set(src_c_dir "${src_dir}/c")
+set(src_cpp_dir "${src_dir}/cpp")
 set(include_dir "include")
 set(test_dir "test")
 
 add_library(pgfe SHARED 
-    ${src_dir}/generic.c 
-    ${src_dir}/generic-internal.c
-    ${src_dir}/templates.c
-    ${src_dir}/utils.c
-    ${src_dir}/sha-internal.c
-    ${src_dir}/sha2-backend.c
-    ${src_dir}/keccak-backend.c
-    ${src_dir}/md5-backend.c
-    ${src_dir}/sha1.c ${src_dir}/sha224.c ${src_dir}/sha256.c ${src_dir}/sha384.c ${src_dir}/sha512.c
-    ${src_dir}/sha3-224.c ${src_dir}/sha3-256.c ${src_dir}/sha3-384.c ${src_dir}/sha3-512.c 
-    ${src_dir}/md5.c 
-    ${src_dir}/shake.c
-    ${src_dir}/hmac.c
-    ${src_dir}/otp-generic.c
-    ${src_dir}/hotp.c ${src_dir}/totp.c
-    ${src_dir}/base-encoding-internal.c
-    ${src_dir}/base64.c ${src_dir}/base32.c ${src_dir}/base16.c
-    ${src_dir}/sequential_data.cpp
-    ${src_dir}/utils.cpp
-    ${src_dir}/algorithm_selectable.cpp
-    ${src_dir}/hash_encoder.cpp
-    ${src_dir}/hmac_encoder.cpp
-    ${src_dir}/abstract_base_encoding.cpp
-    ${src_dir}/base16.cpp
-    ${src_dir}/base32.cpp
-    ${src_dir}/base64.cpp
-    ${src_dir}/abstract_otp.cpp
-    ${src_dir}/hotp.cpp ${src_dir}/totp.cpp)
+  ${src_c_dir}/generic.c 
+  ${src_c_dir}/generic-internal.c
+  ${src_c_dir}/utils.c
+  ${src_c_dir}/hash/sha-internal.c
+  ${src_c_dir}/hash/sha2-backend.c
+  ${src_c_dir}/hash/keccak-backend.c
+  ${src_c_dir}/hash/md5-backend.c
+  ${src_c_dir}/hash/sha1.c ${src_c_dir}/hash/sha224.c ${src_c_dir}/hash/sha256.c ${src_c_dir}/hash/sha384.c ${src_c_dir}/hash/sha512.c
+  ${src_c_dir}/hash/sha3-224.c ${src_c_dir}/hash/sha3-256.c ${src_c_dir}/hash/sha3-384.c ${src_c_dir}/hash/sha3-512.c 
+  ${src_c_dir}/hash/md5.c 
+  ${src_c_dir}/hash/shake.c
+  ${src_c_dir}/hmac/hmac.c
+  ${src_c_dir}/otp/otp-generic.c
+  ${src_c_dir}/otp/hotp.c ${src_c_dir}/otp/totp.c
+  ${src_c_dir}/base_encoding/base-encoding-internal.c
+  ${src_c_dir}/base_encoding/base64.c ${src_c_dir}/base_encoding/base32.c ${src_c_dir}/base_encoding/base16.c
+  ${src_cpp_dir}/generic.cpp
+  ${src_cpp_dir}/sequential_data.cpp
+  ${src_cpp_dir}/utils.cpp
+  ${src_cpp_dir}/algorithm_selectable.cpp
+  ${src_cpp_dir}/hash/hash_encoder.cpp
+  ${src_cpp_dir}/hmac/hmac_encoder.cpp
+  ${src_cpp_dir}/base_encoding/abstract_base_encoding.cpp
+  ${src_cpp_dir}/base_encoding/base16.cpp
+  ${src_cpp_dir}/base_encoding/base32.cpp
+  ${src_cpp_dir}/base_encoding/base64.cpp
+  ${src_cpp_dir}/otp/abstract_otp.cpp
+  ${src_cpp_dir}/otp/hotp.cpp ${src_cpp_dir}/otp/totp.cpp)
 add_executable(pgfetest ${test_dir}/test.c)
+add_executable(pgfetestcpp ${test_dir}/test.cpp)
 # add_executable(totptest ${test_dir}/totptest.c)
 # add_executable(totptestcpp ${test_dir}/totptest.cpp)
-add_executable(pgfetestcpp ${test_dir}/test.cpp)
 
 target_include_directories(pgfe PRIVATE
   ${src_dir}
   ${include_dir}
 )
 
 target_link_libraries(pgfetest pgfe)
-# target_link_libraries(totptest pgfe)
 target_link_libraries(pgfetestcpp pgfe)
+# target_link_libraries(totptest pgfe)
 # target_link_libraries(totptestcpp pgfe)
 
 include(${test_dir}/cmake/test_meta.cmake)

Makefile

@@ -1,6 +1,6 @@
-include meta.mak
+include metadata.mak
 
-.PHONY: all rebuild clean test install uninstall
+.PHONY: all rebuild clean test install uninstall init-scripts
 
 all: 
 	@echo 'Building...'
@@ -37,3 +37,6 @@ uninstall:
 	@echo 'Removing shared library...'
 	@rm -vrf $(LIB_DIR)/$(TARGET_FILE)
 	@echo done
+
+init-scripts:
+	@chmod u+x ./scripts/{comment_header,update_meta,deploy}

README.md

@@ -11,7 +11,9 @@
 
 *libpgfe* currently supports hash encoding (e.g. SHA256, MD5), HMAC encoding, HOTP/TOTP and Base 16/32/64.
 
-| Item         | Content             |
+*libpgfe* is currently under heavy development, more features and optimization will be added in the future.
+
+| Entry        | Info                |
 | :----------- | :------------------ |
 | C Standard   | [C11 (ISO/IEC 9899:2011)](https://en.wikipedia.org/wiki/C11_(C_standard_revision))  |
 | C++ Standard | [C++14 (ISO/IEC 14882:2014)](https://en.wikipedia.org/wiki/C++14) |
@@ -20,9 +22,7 @@
 
 [*Any questions?*](.github/qna.md)
 
-## Endianness (Byte order)
-
-> [*What is it anyway??*](https://en.wikipedia.org/wiki/Endianness)
+## [Endianness (Byte order)](https://en.wikipedia.org/wiki/Endianness)
 
 The implementation philosophy of *libpgfe* assumes that the systems running this library are **Little Endian**, since it is widely used by architectures and OS. Therefore, this library should not run properly on Big Endian systems.
 

include/algorithm_selectable.hpp

@@ -28,7 +28,7 @@ class AlgorithmSelectable
     void select_algorithm(pgfe_algorithm_choice choice);
 
 public:
-    pgfe_algorithm_choice algorithm();
+    pgfe_algorithm_choice algorithm() const;
 };
 } // namespace PGFE
 } // namespace chardon55

include/base-encoding-internal.h include/backend/base-encoding-internal.hinclude/base-encoding-internal.h renamed to include/backend/base-encoding-internal.h

@@ -10,7 +10,7 @@
 
 #include <stdbool.h>
 
-#include "generic.h"
+#include "../generic.h"
 
 #ifdef __cplusplus
 extern "C" {

include/generic-internal.h include/backend/generic-internal.hinclude/generic-internal.h renamed to include/backend/generic-internal.h

@@ -8,7 +8,7 @@
 #ifndef LIBPGFE_GENERIC_INTERNAL_H
 #define LIBPGFE_GENERIC_INTERNAL_H
 
-#include "generic.h"
+#include "../generic.h"
 
 #ifdef __cplusplus
 extern "C" {

include/keccak-backend.h include/backend/keccak-backend.hinclude/keccak-backend.h renamed to include/backend/keccak-backend.h

@@ -10,8 +10,8 @@
 
 #include <stdint.h>
 
-#include "generic.h"
-#include "hmac.h"
+#include "../generic.h"
+#include "../hmac.h"
 
 #ifdef __cplusplus
 extern "C" {