ParserBase could be optimized
@ezio-melotti 已经在做这个了。
开始于 2022年5月1日。
- 主要语言
- Python
- 星标
- 77.2k
- 派生
- 36k
- 平均合并
- 1 天 9 小时
- 30 天内合并 PR
- 558
描述
There is a lot of wasteful CPU code in this fashion (seen multiple times) :-
if ")" in rawdata[j:]:
j = rawdata.find(")", j) + 1
search performed in string twice, even if we could just do it once.
Consider the strings immutable, then on every slice a new string is created on the fly.
Moreover, the slicing done at rawdata[j:], is bound to be quite expensive depending on the size of the string.
We could eliminate slicing altogether and only have one find operation in this style :-
RPAREN_pos = rawdata.find(")", j)
if find_RPAREN != -1:
j = RPAREN_pos + 1
I'm new to Open Source Code Contributions. Would love to learn from other's coding style & know other's point of view.
Originally posted by @be-thomas in https://github.com/python/cpython/issues/92084#issuecomment-1114023102
贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
评估
这个 Issue 还没有评估数据。