Skip to content

Conversation

@luoliwoshang
Copy link
Member

@luoliwoshang luoliwoshang commented Mar 7, 2025

llcppg v0.2.0

conan install --requires cjson/1.7.18 --generator PkgConfigDeps --options=*:shared=True --options=utils=True --build=missing --output-folder=./build && export PKG_CONFIG_PATH=$PKG_CONFIG_PATH:$PWD/build

linux & macos use same conan config have same result

@luoliwoshang luoliwoshang marked this pull request as draft March 7, 2025 09:09
@luoliwoshang luoliwoshang changed the title cjson cjson/1.7.18 Mar 7, 2025
@luoliwoshang luoliwoshang force-pushed the cjson branch 5 times, most recently from 7709307 to a76f1de Compare March 13, 2025 10:27
@luoliwoshang luoliwoshang mentioned this pull request Mar 13, 2025
@MeteorsLiu
Copy link
Contributor

This llpkg missed llpkg.cfg, which cannot pass Github Action verfication.
Here's the examplellpkg.cfg for this llpkg:

{
  "upstream": {
    "package": {
      "name": "cjson",
      "version": "1.7.18"
    },
    "installer":{
      "name": "conan",
      "config" : {
        "options": "utils=True"
      }
    }
  }
}

@luoliwoshang luoliwoshang force-pushed the cjson branch 2 times, most recently from 681684b to 0a89206 Compare March 18, 2025 03:40
@luoliwoshang luoliwoshang force-pushed the cjson branch 2 times, most recently from 66c3ea8 to 86f2657 Compare April 1, 2025 03:51
@luoliwoshang luoliwoshang marked this pull request as ready for review April 1, 2025 03:52
}

/* Render a cJSON entity to text using a buffer already allocated in memory with given length. Returns 1 on success and 0 on failure. */
/* NOTE: cJSON is not always 100% accurate in estimating how much memory it will use, so to be safe allocate 5 bytes more than you actually need */
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A Note is recommended to use "MARKER(uid): note body" format.

Details

lint 解释

这个lint结果提示在注释中使用“MARKER(uid): note body”格式是一个建议。这意味着在编写注释时,应该遵循这种特定的格式来提高代码的可读性和一致性。

错误用法

以下是一个错误的注释示例:

// 这是一个注释

正确用法

以下是一个正确的注释示例:

// MARKER(12345): 这是一个注释

💡 以上内容由 AI 辅助生成,如有疑问欢迎反馈交流

}

/* Implement RFC6902 (https://tools.ietf.org/html/rfc6902) JSON Patch spec. */
/* NOTE: This modifies objects in 'from' and 'to' by sorting the elements by their key */
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A Note is recommended to use "MARKER(uid): note body" format.

Details

lint 解释

这个lint结果提示在注释中使用“MARKER(uid): note body”格式是一个建议。这意味着在编写注释时,应该遵循这种特定的格式来提高代码的可读性和一致性。

错误用法

以下是一个错误的注释示例:

// 这是一个错误的注释示例

正确用法

以下是一个正确的注释示例:

// MARKER(12345): 这是一个正确的注释示例

💡 以上内容由 AI 辅助生成,如有疑问欢迎反馈交流

}

/* generates a patch to move from -> to */
/* NOTE: This modifies objects in 'from' and 'to' by sorting the elements by their key */
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A Note is recommended to use "MARKER(uid): note body" format.

Details

lint 解释

这个lint结果提示在注释中使用“MARKER(uid): note body”格式是一个建议。这意味着在编写注释时,应该遵循这种特定的格式来提高代码的可读性和一致性。

错误用法

以下是一个错误的注释示例:

// 这是一个错误的注释示例

正确用法

以下是一个正确的注释示例:

// MARKER(12345): 这是一个正确的注释示例

💡 以上内容由 AI 辅助生成,如有疑问欢迎反馈交流

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants