Skip to content

Commit d57e250

Browse files
committed
add librdata to github workflow
1 parent e1eccc0 commit d57e250

File tree

4 files changed

+7
-8
lines changed

4 files changed

+7
-8
lines changed

.github/workflows/ubuntu.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -43,6 +43,7 @@ jobs:
4343
sudo apt install libglpk-dev libcurl4-openssl-dev libmpfr-dev libfontconfig1-dev libcairo2-dev #required by some r packages
4444
sudo apt install jags
4545
sudo apt install libminizip-dev # required by freexl
46+
sudo apt install librdata-dev
4647
git clone https://github.com/jasp-stats/freexl.git
4748
cd freexl && ./configure && make && sudo make install
4849

Desktop/CMakeLists.txt

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -107,16 +107,16 @@ target_include_directories(
107107
${PROJECT_SOURCE_DIR}/QMLComponents
108108
# ReadStat
109109
$<$<PLATFORM_ID:Windows>:${RTOOLS_LIBREADSTAT_H}>
110+
${LIBREADSTAT_INCLUDE_DIRS}
110111
# librdata
111112
$<$<PLATFORM_ID:Windows>:${RTOOLS_LIBRDATA_H}>
112-
${LIBREADSTAT_INCLUDE_DIRS}
113+
${LIBRDATA_INCLUDE_DIRS}
113114
# JSONCPP
114115
$<$<PLATFORM_ID:Linux>:${_PKGCONFIG_LIB_JSONCPP_INCLUDEDIR}>
115116
$<$<BOOL:${FLATPAK_USED}>:/app/include/QtCore5Compat>
116117
$<$<BOOL:${FLATPAK_USED}>:/app/include/QtWebEngineQuick>
117118
$<$<BOOL:${FLATPAK_USED}>:/app/include/QtWebEngineCore>
118119
${LIBFREEXL_INCLUDE_DIRS}
119-
${LIBRDATA_INCLUDE_DIRS}
120120
)
121121

122122
target_link_libraries(
@@ -169,8 +169,6 @@ target_link_libraries(
169169
#$<$<PLATFORM_ID:Linux>:${_PKGCONFIG_LIB_JSONCPP_LINK_LIBRARIES}>
170170
# R-Framework --------------------------------
171171
$<$<PLATFORM_ID:Darwin>:${_R_Framework}>
172-
# librdata
173-
${LIBRDATA_LIBRARIES}
174172
)
175173

176174
target_compile_definitions(JASP PUBLIC JASP_USES_QT_HERE)

Desktop/data/importers/rdata/readrdata.h

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
//
2-
// Copyright (C) 2013-2024 University of Amsterdam
2+
// Copyright (C) 2013-2025 University of Amsterdam
33
//
44
// This program is free software: you can redistribute it and/or modify
55
// it under the terms of the GNU General Public License as published by
@@ -36,8 +36,8 @@ class RDataReader
3636
private:
3737
struct RDataCtx
3838
{
39-
long column_count = 0;
40-
long row_count = 0;
39+
size_t column_count = 0;
40+
size_t row_count = 0;
4141
const char *table_name = nullptr;
4242
stringvec column_names;
4343
stringvecvec column_data;

Desktop/data/importers/rdataimporter.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ ImportDataSet *RDataImporter::loadFile(const std::string &locator, std::function
5353
if (colCount == 0)
5454
throw std::runtime_error(fq(tr("0 valid columns were read from the file, please check your data file.")));
5555

56-
importColumns.reserve(ccolCount);
56+
importColumns.reserve(colCount);
5757

5858
for (size_t colIndex = 0; colIndex < colCount; ++colIndex)
5959
{

0 commit comments

Comments
 (0)