refactor: xmake in use, compiles and tests fine
This commit is contained in:
6
.gitignore
vendored
6
.gitignore
vendored
@@ -7,6 +7,12 @@ out/
|
||||
out/Debug/
|
||||
out/Release/
|
||||
|
||||
# xmake
|
||||
xmake/
|
||||
xmake/*
|
||||
.xmake
|
||||
.xmake/*
|
||||
|
||||
# Cmake files
|
||||
CMakeCache.txt
|
||||
CMakeFiles/
|
||||
|
||||
@@ -1,98 +0,0 @@
|
||||
cmake_minimum_required(VERSION 3.20)
|
||||
project(
|
||||
arraylist
|
||||
VERSION 1.0
|
||||
LANGUAGES C)
|
||||
|
||||
# ------------------------------------------------
|
||||
# Estandard C
|
||||
# ------------------------------------------------
|
||||
set(CMAKE_C_STANDARD 11)
|
||||
set(CMAKE_C_STANDARD_REQUIRED ON)
|
||||
set(CMAKE_C_EXTENSIONS OFF)
|
||||
|
||||
# Exportar compile_commands.json (clangd / IDEs)
|
||||
set(CMAKE_EXPORT_COMPILE_COMMANDS ON)
|
||||
|
||||
# ------------------------------------------------
|
||||
# Is Top project?
|
||||
# ------------------------------------------------
|
||||
if(CMAKE_SOURCE_DIR STREQUAL PROJECT_SOURCE_DIR)
|
||||
set(ARRAYLIST_IS_TOP_LEVEL ON)
|
||||
else()
|
||||
set(ARRAYLIST_IS_TOP_LEVEL OFF)
|
||||
endif()
|
||||
|
||||
# ------------------------------------------------
|
||||
# Opciones
|
||||
# ------------------------------------------------
|
||||
option(ARRAYLIST_BUILD_TESTS "Build tests" ${ARRAYLIST_IS_TOP_LEVEL})
|
||||
option(ARRAYLIST_BUILD_EXAMPLE "Build example" ${ARRAYLIST_IS_TOP_LEVEL})
|
||||
option(ARRAYLIST_ENABLE_INSTALL "Enable install" ${ARRAYLIST_IS_TOP_LEVEL})
|
||||
option(ARRAYLIST_ENABLE_SANITIZERS "Enable Sanitizers for test" ON)
|
||||
|
||||
# ------------------------------------------------
|
||||
# Warning Flags (GCC/Clang vs MSVC)
|
||||
# ------------------------------------------------
|
||||
if(MSVC)
|
||||
set(ARRAYLIST_WARNING_FLAGS /W4)
|
||||
else()
|
||||
set(ARRAYLIST_WARNING_FLAGS -Wall -Wextra -Wpedantic)
|
||||
endif()
|
||||
|
||||
# ------------------------------------------------
|
||||
# Main Library
|
||||
# ------------------------------------------------
|
||||
add_library(arraylist src/arraylist.c)
|
||||
|
||||
# Alias con namespace → los consumidores via CPM/FetchContent pueden escribir
|
||||
# target_link_libraries(mi_app arraylist::arraylist)
|
||||
add_library(arraylist::arraylist ALIAS arraylist)
|
||||
|
||||
target_include_directories(
|
||||
arraylist PUBLIC $<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/include>
|
||||
$<INSTALL_INTERFACE:include>)
|
||||
|
||||
# Flagas only for compile lib
|
||||
target_compile_options(arraylist PRIVATE ${ARRAYLIST_WARNING_FLAGS})
|
||||
|
||||
# Propagates c11 as requirement
|
||||
target_compile_features(arraylist PUBLIC c_std_11)
|
||||
|
||||
# ------------------------------------------------
|
||||
# Example only built when is main proyect
|
||||
# ------------------------------------------------
|
||||
if(ARRAYLIST_BUILD_EXAMPLE)
|
||||
add_executable(arraylist_example src/main.c)
|
||||
target_link_libraries(arraylist_example PRIVATE arraylist)
|
||||
endif()
|
||||
|
||||
# ------------------------------------------------
|
||||
# Tests
|
||||
# ------------------------------------------------
|
||||
if(ARRAYLIST_BUILD_TESTS)
|
||||
enable_testing()
|
||||
add_subdirectory(test)
|
||||
endif()
|
||||
|
||||
# ------------------------------------------------
|
||||
# Installation rules
|
||||
# ------------------------------------------------
|
||||
if(ARRAYLIST_ENABLE_INSTALL)
|
||||
include(GNUInstallDirs)
|
||||
|
||||
install(
|
||||
TARGETS arraylist
|
||||
EXPORT arraylistTargets
|
||||
ARCHIVE DESTINATION ${CMAKE_INSTALL_LIBDIR}
|
||||
LIBRARY DESTINATION ${CMAKE_INSTALL_LIBDIR}
|
||||
RUNTIME DESTINATION ${CMAKE_INSTALL_BINDIR})
|
||||
|
||||
install(DIRECTORY include/ DESTINATION ${CMAKE_INSTALL_INCLUDEDIR})
|
||||
|
||||
install(
|
||||
EXPORT arraylistTargets
|
||||
FILE arraylistTargets.cmake
|
||||
NAMESPACE arraylist::
|
||||
DESTINATION ${CMAKE_INSTALL_LIBDIR}/cmake/arraylist)
|
||||
endif()
|
||||
@@ -1,9 +1,9 @@
|
||||
#ifndef ARRAYLIST_H
|
||||
#define ARRAYLIST_H
|
||||
#ifndef LAE_ARRAYLIST_H
|
||||
#define LAE_ARRAYLIST_H
|
||||
|
||||
#include <inttypes.h>
|
||||
#include <stddef.h>
|
||||
#include <stdbool.h>
|
||||
#include <stddef.h>
|
||||
|
||||
typedef struct ArrayList ArrayList;
|
||||
|
||||
@@ -35,8 +35,8 @@ size_t arraylist_capacity(const ArrayList *arr);
|
||||
bool arraylist_is_empty(const ArrayList *arr);
|
||||
|
||||
ArrayListErr arraylist_push_back(ArrayList *arr, void *data);
|
||||
ArrayListErr arraylist_insert(ArrayList*arr, size_t index, void *data);
|
||||
ArrayListErr arraylist_push_front(ArrayList* arr, void *data);
|
||||
ArrayListErr arraylist_insert(ArrayList *arr, size_t index, void *data);
|
||||
ArrayListErr arraylist_push_front(ArrayList *arr, void *data);
|
||||
|
||||
// Here out can be null for not having anything writen
|
||||
ArrayListErr arraylist_pop_back(ArrayList *arr, void *out);
|
||||
@@ -49,9 +49,13 @@ ArrayListErr arraylist_set(ArrayList *arr, size_t index, void *data);
|
||||
ArrayListErr arraylist_resize(ArrayList *arr, size_t new_capacity);
|
||||
ArrayListErr arraylist_reserve(ArrayList *arr, size_t size_to_reserve);
|
||||
|
||||
//Slice stuff
|
||||
ArrayListErr arraylist_slice(ArraySlice **slice, ArrayList *arr, size_t start, size_t len);
|
||||
ArrayListErr arraylist_slice_unsafe(ArraySlice **slice, ArrayList *arr, size_t start, size_t len);
|
||||
// Slice stuff
|
||||
ArrayListErr
|
||||
arraylist_slice(ArraySlice **slice, ArrayList *arr, size_t start, size_t len);
|
||||
|
||||
ArrayListErr arraylist_slice_unsafe(
|
||||
ArraySlice **slice, ArrayList *arr, size_t start, size_t len);
|
||||
|
||||
ArrayListErr arrayslice_destroy(ArraySlice **slice);
|
||||
|
||||
bool arrayslice_is_valid(const ArraySlice *slice);
|
||||
@@ -1,10 +1,10 @@
|
||||
#include "arraylist.h"
|
||||
#include "lae_arraylist.h"
|
||||
#include <stdbool.h>
|
||||
#include <stdint.h>
|
||||
#include <stdlib.h>
|
||||
#include <string.h>
|
||||
|
||||
struct ArrayList{
|
||||
struct ArrayList {
|
||||
uint8_t *buffer;
|
||||
size_t capacity;
|
||||
size_t len;
|
||||
@@ -26,7 +26,8 @@ struct ArraySlice {
|
||||
static ArrayListErr arraylist_grow(ArrayList *arr);
|
||||
static ArrayListErr arraylist_shrink(ArrayList *arr);
|
||||
|
||||
ArrayListErr arraylist_init(ArrayList **arr, size_t capacity, size_t elem_size) {
|
||||
ArrayListErr
|
||||
arraylist_init(ArrayList **arr, size_t capacity, size_t elem_size) {
|
||||
if (arr == NULL) {
|
||||
return ARRLIST_NULL_ARG;
|
||||
}
|
||||
@@ -122,7 +123,7 @@ static ArrayListErr arraylist_grow(ArrayList *arr) {
|
||||
size_t new_capacity = arr->capacity * 2;
|
||||
|
||||
if (new_capacity > SIZE_MAX / arr->elem_size) {
|
||||
return ARRLIST_ALLOC_OVERFLOW;
|
||||
return ARRLIST_ALLOC_OVERFLOW;
|
||||
}
|
||||
|
||||
void *tmp = realloc(arr->buffer, new_capacity * arr->elem_size);
|
||||
@@ -139,10 +140,10 @@ static ArrayListErr arraylist_shrink(ArrayList *arr) {
|
||||
size_t new_capacity = arr->capacity / 2;
|
||||
|
||||
if (new_capacity > SIZE_MAX / arr->elem_size) {
|
||||
return ARRLIST_ALLOC_OVERFLOW;
|
||||
return ARRLIST_ALLOC_OVERFLOW;
|
||||
}
|
||||
|
||||
void *tmp = realloc(arr->buffer, new_capacity *arr->elem_size);
|
||||
void *tmp = realloc(arr->buffer, new_capacity * arr->elem_size);
|
||||
if (tmp == NULL) {
|
||||
return ARRLIST_BAD_ALLOC;
|
||||
}
|
||||
@@ -195,13 +196,9 @@ ArrayListErr arraylist_push_front(ArrayList *arr, void *data) {
|
||||
memmove(
|
||||
arr->buffer + arr->elem_size,
|
||||
arr->buffer,
|
||||
(arr->len) * arr->elem_size
|
||||
);
|
||||
(arr->len) * arr->elem_size);
|
||||
|
||||
memcpy(
|
||||
arr->buffer,
|
||||
data,
|
||||
arr->elem_size);
|
||||
memcpy(arr->buffer, data, arr->elem_size);
|
||||
|
||||
arr->len++;
|
||||
|
||||
@@ -233,9 +230,7 @@ ArrayListErr arraylist_insert(ArrayList *arr, size_t index, void *data) {
|
||||
arr->buffer + (index * arr->elem_size),
|
||||
(arr->len - index) * arr->elem_size);
|
||||
|
||||
memcpy(arr->buffer + (index * arr->elem_size),
|
||||
data,
|
||||
arr->elem_size);
|
||||
memcpy(arr->buffer + (index * arr->elem_size), data, arr->elem_size);
|
||||
|
||||
arr->len++;
|
||||
|
||||
@@ -295,11 +290,7 @@ ArrayListErr arraylist_pop_front(ArrayList *arr, void *out) {
|
||||
}
|
||||
|
||||
if (out != NULL) {
|
||||
memcpy(
|
||||
out,
|
||||
arr->buffer,
|
||||
arr->elem_size
|
||||
);
|
||||
memcpy(out, arr->buffer, arr->elem_size);
|
||||
}
|
||||
|
||||
memmove(
|
||||
@@ -330,11 +321,7 @@ ArrayListErr arraylist_remove_at(ArrayList *arr, size_t index, void *out) {
|
||||
}
|
||||
|
||||
if (out != NULL) {
|
||||
memcpy(
|
||||
out,
|
||||
arr->buffer + (index * arr->elem_size),
|
||||
arr->elem_size
|
||||
);
|
||||
memcpy(out, arr->buffer + (index * arr->elem_size), arr->elem_size);
|
||||
}
|
||||
|
||||
memmove(
|
||||
@@ -356,10 +343,7 @@ ArrayListErr arraylist_get(const ArrayList *arr, size_t index, void *out) {
|
||||
return ARRLIST_OUT_OF_BOUNDS;
|
||||
}
|
||||
|
||||
memcpy(
|
||||
out,
|
||||
arr->buffer + (index * arr->elem_size),
|
||||
arr->elem_size);
|
||||
memcpy(out, arr->buffer + (index * arr->elem_size), arr->elem_size);
|
||||
|
||||
return ARRLIST_OK;
|
||||
}
|
||||
@@ -373,10 +357,7 @@ ArrayListErr arraylist_set(ArrayList *arr, size_t index, void *data) {
|
||||
return ARRLIST_OUT_OF_BOUNDS;
|
||||
}
|
||||
|
||||
memcpy(
|
||||
arr->buffer + (index * arr->elem_size),
|
||||
data,
|
||||
arr->elem_size);
|
||||
memcpy(arr->buffer + (index * arr->elem_size), data, arr->elem_size);
|
||||
|
||||
return ARRLIST_OK;
|
||||
}
|
||||
@@ -395,7 +376,7 @@ ArrayListErr arraylist_resize(ArrayList *arr, size_t new_capacity) {
|
||||
}
|
||||
|
||||
if (new_capacity > SIZE_MAX / arr->elem_size) {
|
||||
return ARRLIST_ALLOC_OVERFLOW;
|
||||
return ARRLIST_ALLOC_OVERFLOW;
|
||||
}
|
||||
|
||||
uint8_t *tmp = realloc(arr->buffer, new_capacity * arr->elem_size);
|
||||
@@ -420,7 +401,7 @@ ArrayListErr arraylist_reserve(ArrayList *arr, size_t size_to_reserve) {
|
||||
if (arr->capacity + size_to_reserve > SIZE_MAX / arr->elem_size ||
|
||||
size_to_reserve > SIZE_MAX / arr->elem_size) {
|
||||
|
||||
return ARRLIST_ALLOC_OVERFLOW;
|
||||
return ARRLIST_ALLOC_OVERFLOW;
|
||||
}
|
||||
|
||||
uint8_t *tmp = realloc(
|
||||
@@ -436,7 +417,8 @@ ArrayListErr arraylist_reserve(ArrayList *arr, size_t size_to_reserve) {
|
||||
return ARRLIST_OK;
|
||||
}
|
||||
|
||||
ArrayListErr arraylist_slice(ArraySlice **slice, ArrayList *arr, size_t start, size_t len) {
|
||||
ArrayListErr
|
||||
arraylist_slice(ArraySlice **slice, ArrayList *arr, size_t start, size_t len) {
|
||||
if (arr == NULL || slice == NULL) {
|
||||
return ARRLIST_NULL_ARG;
|
||||
}
|
||||
@@ -469,7 +451,8 @@ ArrayListErr arraylist_slice(ArraySlice **slice, ArrayList *arr, size_t start, s
|
||||
return ARRLIST_OK;
|
||||
}
|
||||
|
||||
ArrayListErr arraylist_slice_unsafe(ArraySlice **slice, ArrayList *arr, size_t start, size_t len) {
|
||||
ArrayListErr arraylist_slice_unsafe(
|
||||
ArraySlice **slice, ArrayList *arr, size_t start, size_t len) {
|
||||
if (arr == NULL || slice == NULL) {
|
||||
return ARRLIST_NULL_ARG;
|
||||
}
|
||||
@@ -606,8 +589,7 @@ ArrayListErr arrayslice_to_arraylist(ArrayList **arr, const ArraySlice *slice) {
|
||||
ArrayListErr init_err = arraylist_init(
|
||||
&new_arr,
|
||||
arraylist_capacity(slice->arr),
|
||||
slice->arr->elem_size
|
||||
);
|
||||
slice->arr->elem_size);
|
||||
|
||||
if (init_err != ARRLIST_OK) {
|
||||
return init_err;
|
||||
@@ -616,8 +598,7 @@ ArrayListErr arrayslice_to_arraylist(ArrayList **arr, const ArraySlice *slice) {
|
||||
memcpy(
|
||||
new_arr->buffer,
|
||||
slice->arr->buffer + (slice->start * slice->arr->elem_size),
|
||||
slice->arr->elem_size * len
|
||||
);
|
||||
slice->arr->elem_size * len);
|
||||
|
||||
new_arr->len = len;
|
||||
|
||||
@@ -1,121 +0,0 @@
|
||||
# ---------------------------------------------------------------
|
||||
# First search cmocka and automatically download if necessary
|
||||
# ---------------------------------------------------------------
|
||||
find_package(cmocka QUIET)
|
||||
|
||||
if(NOT cmocka_FOUND)
|
||||
message(
|
||||
STATUS
|
||||
"[arraylist] cmocka was not fount in the system — downloading with FetchContent..."
|
||||
)
|
||||
|
||||
include(FetchContent)
|
||||
FetchContent_Declare(
|
||||
cmocka
|
||||
GIT_REPOSITORY https://git.cryptomilk.org/projects/cmocka.git
|
||||
GIT_TAG cmocka-1.1.7
|
||||
GIT_SHALLOW TRUE)
|
||||
|
||||
set(WITH_STATIC_LIB
|
||||
ON
|
||||
CACHE BOOL "" FORCE)
|
||||
set(WITH_SHARED_LIB
|
||||
OFF
|
||||
CACHE BOOL "" FORCE)
|
||||
set(WITH_CMOCKERY_SUPPORT
|
||||
OFF
|
||||
CACHE BOOL "" FORCE)
|
||||
set(WITH_EXAMPLES
|
||||
OFF
|
||||
CACHE BOOL "" FORCE)
|
||||
set(UNIT_TESTING
|
||||
OFF
|
||||
CACHE BOOL "" FORCE)
|
||||
set(PICKY_DEVELOPER
|
||||
OFF
|
||||
CACHE BOOL "" FORCE)
|
||||
|
||||
FetchContent_MakeAvailable(cmocka)
|
||||
|
||||
# cmocka can export its target as "cmocka" or "cmocka-static" create alias if
|
||||
# it doesnt exist
|
||||
if(NOT TARGET cmocka::cmocka)
|
||||
if(TARGET cmocka-static)
|
||||
add_library(cmocka::cmocka ALIAS cmocka-static)
|
||||
elseif(TARGET cmocka)
|
||||
add_library(cmocka::cmocka ALIAS cmocka)
|
||||
else()
|
||||
message(
|
||||
WARNING
|
||||
"[arraylist] cmocka::cmocka target could not be tested — skipping tests"
|
||||
)
|
||||
return()
|
||||
endif()
|
||||
endif()
|
||||
endif()
|
||||
|
||||
# ---------------------------------------------------------------
|
||||
# Verifiy ASAN support
|
||||
# ---------------------------------------------------------------
|
||||
set(ARRAYLIST_USE_ASAN OFF)
|
||||
|
||||
if(ARRAYLIST_ENABLE_SANITIZERS)
|
||||
if(MSVC)
|
||||
if(MSVC_VERSION GREATER_EQUAL 1928)
|
||||
set(ARRAYLIST_USE_ASAN ON)
|
||||
else()
|
||||
message(
|
||||
STATUS
|
||||
"[arraylist] MSVC < 16.9 doesnt support ASAN — sanitizers deactivated."
|
||||
)
|
||||
endif()
|
||||
elseif(CMAKE_C_COMPILER_ID MATCHES "GNU|Clang" AND NOT WIN32)
|
||||
include(CheckCCompilerFlag)
|
||||
check_c_compiler_flag(-fsanitize=address ARRAYLIST_COMPILER_HAS_ASAN)
|
||||
if(ARRAYLIST_COMPILER_HAS_ASAN)
|
||||
set(ARRAYLIST_USE_ASAN ON)
|
||||
else()
|
||||
message(
|
||||
STATUS
|
||||
"[arraylist] Compiler doesnt support -fsanitize=address — sanitizers deactivated."
|
||||
)
|
||||
endif()
|
||||
else()
|
||||
message(
|
||||
STATUS
|
||||
"[arraylist] Platform/compiler doesnt support ASAN — sanitizers deactivated."
|
||||
)
|
||||
endif()
|
||||
|
||||
if(ARRAYLIST_USE_ASAN)
|
||||
message(STATUS "[arraylist] ASAN Enabled for tests")
|
||||
endif()
|
||||
endif()
|
||||
|
||||
# ---------------------------------------------------------------
|
||||
# Test execuatable
|
||||
# ---------------------------------------------------------------
|
||||
add_executable(test_arraylist test_arraylist.c)
|
||||
|
||||
target_link_libraries(test_arraylist PRIVATE arraylist cmocka::cmocka)
|
||||
|
||||
# Same as lib
|
||||
target_compile_features(test_arraylist PRIVATE c_std_11)
|
||||
|
||||
# Apply asan
|
||||
if(ARRAYLIST_USE_ASAN)
|
||||
if(MSVC)
|
||||
target_compile_options(test_arraylist PRIVATE /fsanitize=address)
|
||||
else()
|
||||
target_compile_options(test_arraylist PRIVATE -fsanitize=address
|
||||
-fno-omit-frame-pointer)
|
||||
target_link_options(test_arraylist PRIVATE -fsanitize=address)
|
||||
endif()
|
||||
endif()
|
||||
|
||||
# ---------------------------------------------------------------
|
||||
# Register with CTests
|
||||
# ---------------------------------------------------------------
|
||||
add_test(NAME arraylist_tests COMMAND test_arraylist)
|
||||
|
||||
set_tests_properties(arraylist_tests PROPERTIES TIMEOUT 30)
|
||||
File diff suppressed because it is too large
Load Diff
94
xmake.lua
Normal file
94
xmake.lua
Normal file
@@ -0,0 +1,94 @@
|
||||
set_project("lae_arraylist")
|
||||
set_version("1.0.0")
|
||||
|
||||
set_languages("c11")
|
||||
|
||||
add_rules("mode.debug", "mode.release")
|
||||
|
||||
-- clang compile commands
|
||||
add_rules("plugin.compile_commands.autoupdate")
|
||||
|
||||
-- =========================================================
|
||||
-- Options
|
||||
-- =========================================================
|
||||
|
||||
option("tests")
|
||||
set_default(false)
|
||||
set_showmenu(true)
|
||||
option_end()
|
||||
|
||||
option("example")
|
||||
set_default(false)
|
||||
set_showmenu(true)
|
||||
option_end()
|
||||
|
||||
option("asan")
|
||||
set_default(false)
|
||||
set_showmenu(true)
|
||||
option_end()
|
||||
|
||||
-- =========================================================
|
||||
-- Packages
|
||||
-- =========================================================
|
||||
|
||||
-- Run xmake repo --add lae-repo https://laentropia-homelab.tail7368da.ts.net/laentropia/xmake-repo.git
|
||||
-- to add my repo :)
|
||||
|
||||
add_requires("cmocka", { optional = true }) -- For tests
|
||||
add_requires("lae_allocator")
|
||||
|
||||
-- =========================================================
|
||||
-- Library
|
||||
-- =========================================================
|
||||
|
||||
target("lae_arraylist")
|
||||
set_kind("static")
|
||||
|
||||
add_packages("lae_allocator", { public = true })
|
||||
|
||||
add_files("src/lae_arraylist.c")
|
||||
|
||||
add_headerfiles("include/*.h")
|
||||
|
||||
add_includedirs("include", { public = true })
|
||||
|
||||
if is_mode("debug") then
|
||||
add_cflags("-Wall", "-Wextra", "-Wpedantic")
|
||||
end
|
||||
|
||||
-- =========================================================
|
||||
-- Example executable
|
||||
-- =========================================================
|
||||
|
||||
if has_config("example") then
|
||||
target("example")
|
||||
set_kind("binary")
|
||||
|
||||
add_deps("lae_arraylist")
|
||||
add_packages("lae_allocator")
|
||||
|
||||
add_files("src/main.c")
|
||||
end
|
||||
|
||||
-- =========================================================
|
||||
-- Tests
|
||||
-- =========================================================
|
||||
|
||||
if has_config("tests") then
|
||||
target("test_arena")
|
||||
set_kind("binary")
|
||||
|
||||
add_files("test/test_arraylist.c")
|
||||
|
||||
add_deps("lae_arraylist")
|
||||
|
||||
add_packages("cmocka")
|
||||
|
||||
add_tests("default")
|
||||
|
||||
if has_config("asan") then
|
||||
add_cflags("-fsanitize=address", "-fno-omit-frame-pointer", { force = true })
|
||||
|
||||
add_ldflags("-fsanitize=address", { force = true })
|
||||
end
|
||||
end
|
||||
Reference in New Issue
Block a user