Use process.argv only in CLI version

In the web version, process.argv does not contain the font conversion options.
lvgl · Mar 23, 2024 · 91a14b1 · 91a14b1
1 parent 7f9f29f
commit 91a14b1
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 1 deletion.
diff --git a/lib/cli.js b/lib/cli.js
@@ -288,6 +288,7 @@ List of characters to copy, belongs to previously declared "--font". Examples:
   //
 
   let args = parser.parse_args(argv.length ? argv : [ '-h' ]);
+  args.opts_string = process.argv.slice(2).join(' ');
 
   for (let font of args.font) {
     if (font.ranges.length === 0) {

diff --git a/lib/writers/lvgl/lv_font.js b/lib/writers/lvgl/lv_font.js
@@ -74,7 +74,7 @@ class LvFont extends Font {
     return `/*******************************************************************************
  * Size: ${this.src.size} px
  * Bpp: ${this.opts.bpp}
- * Opts: ${process.argv.slice(2).join(' ')}
+ * Opts: ${this.opts.opts_string}
  ******************************************************************************/
 
 #ifdef LV_LVGL_H_INCLUDE_SIMPLE

diff --git a/web/index.js b/web/index.js
@@ -115,6 +115,8 @@ document.querySelector('#converterForm').addEventListener('submit', function han
     lv_fallback: _fallback
   };
 
+  args.opts_string = '';
+
   convert(args).then(result => {
     const blob = new Blob([ result[_name] ], { type: 'text/plain;charset=utf-8' });