--- lbbs/src/article_cache.c	2025/06/14 09:20:13	1.16
+++ lbbs/src/article_cache.c	2025/06/21 06:52:24	1.22
@@ -15,18 +15,17 @@
  ***************************************************************************/
 
 #include "article_cache.h"
-#include "log.h"
 #include "lml.h"
+#include "log.h"
 #include <errno.h>
-#include <stdio.h>
 #include <fcntl.h>
+#include <stdio.h>
 #include <stdlib.h>
+#include <string.h>
+#include <time.h>
 #include <unistd.h>
 #include <sys/mman.h>
 #include <sys/stat.h>
-#include <strings.h>
-#define _POSIX_C_SOURCE 200809L
-#include <string.h>
 
 #define ARTICLE_HEADER_MAX_LEN 4096
 #define ARTICLE_CONTENT_MAX_LEN 1024 * 1024 * 4 // 4MB
@@ -44,6 +43,8 @@ static const char *BBS_article_footer_co
 };
 static const int BBS_article_footer_color_count = 7;
 
+static char *content_f; // static buffer in large size
+
 inline static int article_cache_path(char *file_path, size_t buf_len, const char *cache_dir, const ARTICLE *p_article)
 {
 	if (file_path == NULL || cache_dir == NULL || p_article == NULL)
@@ -60,8 +61,6 @@ inline static int article_cache_path(cha
 int article_cache_generate(const char *cache_dir, const ARTICLE *p_article, const SECTION_LIST *p_section,
 						   const char *content, const char *sub_ip, int overwrite)
 {
-	static char *content_f; // static buffer in large size
-
 	char data_file[FILE_PATH_LEN];
 	int fd;
 	ARTICLE_CACHE cache;
@@ -108,7 +107,7 @@ int article_cache_generate(const char *c
 		return -2;
 	}
 
-	bzero(&cache, sizeof(cache));
+	memset(&cache, 0, sizeof(cache));
 
 	// Generate article header / footer
 	localtime_r(&(p_article->sub_dt), &tm_sub_dt);
@@ -125,11 +124,13 @@ int article_cache_generate(const char *c
 	header_len = strnlen(header, sizeof(header));
 	footer_len = strnlen(footer, sizeof(footer));
 
-	header_line_cnt = split_data_lines(header, SCREEN_COLS, cache.line_offsets, MAX_SPLIT_FILE_LINES);
+	header_line_cnt = split_data_lines(header, SCREEN_COLS, cache.line_offsets, MAX_SPLIT_FILE_LINES, 1);
 
 	if (header_len != cache.line_offsets[header_line_cnt])
 	{
-		log_common("Header of article(aid=%d) is truncated from %ld to %ld\n", p_article->aid, header_len, cache.line_offsets[header_line_cnt]);
+#ifdef _DEBUG
+		log_error("Header of article(aid=%d) is truncated from %ld to %ld\n", p_article->aid, header_len, cache.line_offsets[header_line_cnt]);
+#endif
 		header_len = (size_t)cache.line_offsets[header_line_cnt];
 	}
 
@@ -137,12 +138,14 @@ int article_cache_generate(const char *c
 	cache.data_len = header_len + (size_t)lml_plain(content, content_f, ARTICLE_CONTENT_MAX_LEN);
 
 	cache.line_total = header_line_cnt +
-					   split_data_lines(content_f, SCREEN_COLS, cache.line_offsets + header_line_cnt, MAX_SPLIT_FILE_LINES - header_line_cnt);
+					   split_data_lines(content_f, SCREEN_COLS, cache.line_offsets + header_line_cnt, MAX_SPLIT_FILE_LINES - header_line_cnt, 1);
 
 	if (cache.data_len - header_len != (size_t)cache.line_offsets[cache.line_total])
 	{
-		log_common("Body of article(aid=%d) is truncated from %ld to %ld\n",
-				p_article->aid, cache.data_len - header_len, cache.line_offsets[cache.line_total]);
+#ifdef _DEBUG
+		log_error("Body of article(aid=%d) is truncated from %ld to %ld\n",
+				  p_article->aid, cache.data_len - header_len, cache.line_offsets[cache.line_total]);
+#endif
 		cache.data_len = header_len + (size_t)(cache.line_offsets[cache.line_total]);
 	}
 
@@ -151,12 +154,14 @@ int article_cache_generate(const char *c
 		cache.line_offsets[i] += (long)header_len;
 	}
 
-	footer_line_cnt = split_data_lines(footer, SCREEN_COLS, cache.line_offsets + cache.line_total, MAX_SPLIT_FILE_LINES - cache.line_total);
+	footer_line_cnt = split_data_lines(footer, SCREEN_COLS, cache.line_offsets + cache.line_total, MAX_SPLIT_FILE_LINES - cache.line_total, 1);
 
 	if (footer_len != cache.line_offsets[cache.line_total + footer_line_cnt])
 	{
-		log_common("Footer of article(aid=%d) is truncated from %ld to %ld\n",
-				p_article->aid, footer_len, cache.line_offsets[cache.line_total + footer_line_cnt]);
+#ifdef _DEBUG
+		log_error("Footer of article(aid=%d) is truncated from %ld to %ld\n",
+				  p_article->aid, footer_len, cache.line_offsets[cache.line_total + footer_line_cnt]);
+#endif
 		footer_len = (size_t)(cache.line_offsets[cache.line_total + footer_line_cnt]);
 	}
 
@@ -203,6 +208,15 @@ int article_cache_generate(const char *c
 	return 0;
 }
 
+void article_cache_cleanup(void)
+{
+	if (content_f)
+	{
+		free(content_f);
+		content_f = NULL;
+	}
+}
+
 int article_cache_load(ARTICLE_CACHE *p_cache, const char *cache_dir, const ARTICLE *p_article)
 {
 	char data_file[FILE_PATH_LEN];
@@ -256,7 +270,7 @@ int article_cache_load(ARTICLE_CACHE *p_
 		return -3;
 	}
 
-	bzero(p_cache, sizeof(*p_cache));
+	memset(p_cache, 0, sizeof(*p_cache));
 	memcpy((void *)p_cache, p_mmap, (size_t)(((ARTICLE_CACHE *)p_mmap)->mmap_len - ((ARTICLE_CACHE *)p_mmap)->data_len));
 
 	p_cache->p_mmap = p_mmap;