如何对开源项目提出代码变更建议？

Question

我正在读一些开源项目的代码。我发现其中一个地方的代码写得不好，可以稍微修改一下，以提高可读性/设计/其他任何可以改进代码并使其更容易使用的东西。

有没有一种通用的方式来报告这样的变化？我知道你可以成为一个贡献者，做一个补丁，提出一些建议。但是如果你不想在这个项目上花费太多的时间，你只需要下载，阅读，想出一个改变，写下它，把它作为一个建议发布在某个地方。

实际上，我更喜欢发布建议的能力，让所有人都能看到，这就像是与社区一起学习，但只是变得灵活了。

Answer 1

您可以在项目论坛、bug跟踪器或邮件列表中发布带有评论的代码片段。每个项目都以不同的方式管理通信和报告。例如:我听说Mozilla开发者倾向于忽略Bugzilla上的评论，所以这可能不是一个评论Firefox变化的好地方。

如果你想写一篇关于它的很长的文章，你最好把它写在博客上，然后链接到它。我会当心你说话的语气。事情可能是这样的，这是有充分理由的。试着理解他们在项目中的进展和他们的日程安排。也许可以问一下为什么会出现这种情况，并清楚地说明问题可能是什么以及谁可能会受到影响。

为更改打补丁被认为是一种良好的实践，并鼓励人们接受您的输入。如果你没有积极地为项目创建补丁，那么通常不会那么快地为这样的事情创建补丁，但它可能会给你更大的影响力。我个人的观点是，这是一件好事。它在提交方面区分了小麦和谷壳，人们需要坚持不懈。

Answer 2

除非你修复一个特定项目的努力满足了某种最低限度的要求，否则你只能在那里做出贡献。这需要一些时间来适应你的改变，看看你的补丁，验证，集成等等。如果你花的时间太少，你的补丁更有可能质量低下，解决已经讨论过的问题，或者仅仅是对概念的误解。

因此，大多数免费工作的开发人员，冒着宝贵的时间冒险查看你的补丁，最终可能会完全重写它，浪费时间，他们将致力于更重要的问题。但如果你花更多的时间工作，对他们来说风险就会更小。如何确定你做了很多工作？那么，联系开发人员所需的工作量将只占您总工作量的一小部分，因此您会发现在报告问题时克服某种官僚作风是不成问题的。

最快的方法是：

• 通过项目的简短description.
• Go to IRC频道将补丁发布到邮件列表，并粘贴补丁，确保符合条件的人会照顾它。
• 在错误跟踪器中提交错误，简要描述错误或改进，并将补丁附加到那里。

太复杂了？哈！要为OpenJDK项目做出贡献，您必须签署一份协议并将其发送给Sun (通过普通邮件或传真)。好吧，这就是“复杂”的意思！并在bugzilla注册，激活您的帐户并使用补丁打开问题并不复杂。

Answer 3

首先，在荷兰有一句话是这样说的：

“没有人问过的建议很少被欣赏”

如果你想帮忙，那就确保你是乐于助人的。

不要试图仅仅通过指出别人的错误而看起来很聪明，而没有进一步的帮助。这不会给项目带来好处，它会打击开发人员的积极性，而且会让你看起来很糟糕。此外，最有可能的是，软件的作者比你更清楚威胁项目的问题。

好的示例：

“我看到你在代码中使用了技巧x和y。我对z有很好的经验。你有没有考虑过使用它？”

好的示例：

“嘿，我每天都很高兴地使用这个软件。我注意到文件加载有点慢。我检查了代码，我建议在迭代之间缓存x和y的结果。不幸的是，我没有时间做补丁，所以我已经在你的bugtracker中报告了我的建议。”

错误的示例：

“伙计，我只是在看你的代码。伙计，真是一团糟。任何菜鸟都知道你应该为变量使用有用的名称。难怪软件都不稳定。”

简而言之，确保你的反馈是积极的。