Lawliet
火狐狸
火狐狸
  • UID34414
  • 注册日期2010-11-03
  • 最后登录2017-04-02
  • 发帖数201
  • 经验13枚
  • 威望0点
  • 贡献值0点
  • 好评度0点
  • 社区居民
  • 忠实会员
90楼#
发布于:2011-12-27 13:45
minghegy:要不然我打包文件上来,你自己制作diff看一下,然后你再编译,如何?回到原帖

也可以,我diff後再把文件發上來
請有bugzilla帳號的人發上去
minghegy
火狐狸
火狐狸
  • UID37982
  • 注册日期2011-12-17
  • 最后登录2014-02-24
  • 发帖数132
  • 经验10枚
  • 威望0点
  • 贡献值0点
  • 好评度0点
91楼#
发布于:2011-12-27 13:45
这是那几个文件,进入到https://bugzilla.mozilla.org/show_bug.cgi?id=669026
点patch文件右边的Splinter Review,然后对着弄就行了(主要是点进去找到路径)

最好把做好的diff发上来,给dongyuanxun一份
我自己弄diff太慢,摸索需要时间
Lawliet
火狐狸
火狐狸
  • UID34414
  • 注册日期2010-11-03
  • 最后登录2017-04-02
  • 发帖数201
  • 经验13枚
  • 威望0点
  • 贡献值0点
  • 好评度0点
  • 社区居民
  • 忠实会员
92楼#
发布于:2011-12-27 13:45
我第一個patch已經做好了,前面發的那個就是了
minghegy
火狐狸
火狐狸
  • UID37982
  • 注册日期2011-12-17
  • 最后登录2014-02-24
  • 发帖数132
  • 经验10枚
  • 威望0点
  • 贡献值0点
  • 好评度0点
93楼#
发布于:2011-12-27 13:45
好,这里只包括第二个补丁
附件没帖上来,可能论坛做了限制
请发信告诉我邮箱
dongyuanxun
非常火狐
非常火狐
  • UID28632
  • 注册日期2009-04-19
  • 最后登录2013-02-14
  • 发帖数898
  • 经验10枚
  • 威望0点
  • 贡献值0点
  • 好评度0点
94楼#
发布于:2011-12-27 13:45
minghegy:好消息,第二个补丁我打成功了,现在正在用我编译的无优化版发帖
可以在贴吧用手势跳到页首页尾
我用的是干净的9.0.1代码,手工打的补丁
每个文件都进去手工改(除那几个test文件可以不管),一定要细心
而且有几个地方(nsGlobalWindowCommands.cpp文件)须要把补丁代码里的true改成PR_TRUE,false改成PR_FALSE

注:那个补丁是打到trunk上的,trunk和9.0的代码有些不同,所以行数经常对不上,但细心点就可以搞定
回到原帖

对,这茬我给忘了

PR_INT都改为int了
dongyuanxun
非常火狐
非常火狐
  • UID28632
  • 注册日期2009-04-19
  • 最后登录2013-02-14
  • 发帖数898
  • 经验10枚
  • 威望0点
  • 贡献值0点
  • 好评度0点
95楼#
发布于:2011-12-27 13:45
有bugzilla帐号的话也不允许发这个补丁

因为时间线不对
Lawliet
火狐狸
火狐狸
  • UID34414
  • 注册日期2010-11-03
  • 最后登录2017-04-02
  • 发帖数201
  • 经验13枚
  • 威望0点
  • 贡献值0点
  • 好评度0点
  • 社区居民
  • 忠实会员
96楼#
发布于:2011-12-27 13:45
dongyuanxun:有bugzilla帐号的话也不允许发这个补丁

因为时间线不对
回到原帖

什麼意思?
minghegy
火狐狸
火狐狸
  • UID37982
  • 注册日期2011-12-17
  • 最后登录2014-02-24
  • 发帖数132
  • 经验10枚
  • 威望0点
  • 贡献值0点
  • 好评度0点
97楼#
发布于:2011-12-27 13:45
我把文件 路径整理出来了,这样就不用去bugzilla上看了
请短信告诉我邮箱,我马上把这几个修改后的文件打包发给您

content/html/content/src/nsHTMLInputElement.cpp
content/html/content/src/nsHTMLTextAreaElement.cpp
content/xbl/builtin/browser-base.inc
4个platformHTMLBindings.xml在content/xbl/builtin/各平台目录下
dom/base/nsFocusManager.cpp
dom/base/nsGlobalWindowCommands.cpp
editor/composer/src/nsEditingSession.cpp
editor/libeditor/base/nsEditorController.cpp
editor/libeditor/base/nsEditorController.h
layout/build/nsLayoutModule.cpp
dongyuanxun
非常火狐
非常火狐
  • UID28632
  • 注册日期2009-04-19
  • 最后登录2013-02-14
  • 发帖数898
  • 经验10枚
  • 威望0点
  • 贡献值0点
  • 好评度0点
98楼#
发布于:2011-12-27 13:45
只针对开发版本

按一个周期会对以前的版本backport

所以你发了他也不看……
dongyuanxun
非常火狐
非常火狐
  • UID28632
  • 注册日期2009-04-19
  • 最后登录2013-02-14
  • 发帖数898
  • 经验10枚
  • 威望0点
  • 贡献值0点
  • 好评度0点
99楼#
发布于:2011-12-27 13:45
你们生成patch的时候
最好用

diff -ruNa
minghegy
火狐狸
火狐狸
  • UID37982
  • 注册日期2011-12-17
  • 最后登录2014-02-24
  • 发帖数132
  • 经验10枚
  • 威望0点
  • 贡献值0点
  • 好评度0点
100楼#
发布于:2011-12-27 13:45
dongyuanxun:你们生成patch的时候
最好用

diff -ruNa
回到原帖

我现在还在摸索diff,让Lawliet生成patch
他肯定会检查比对一下那些patch的
你跟他说
dongyuanxun
非常火狐
非常火狐
  • UID28632
  • 注册日期2009-04-19
  • 最后登录2013-02-14
  • 发帖数898
  • 经验10枚
  • 威望0点
  • 贡献值0点
  • 好评度0点
101楼#
发布于:2011-12-27 13:45
我下次再自己打打看看

是不是那些true类似的改成PR_TRUE的就行了
不知道10的一样不一样,到时再说

在mozilla bugzilla上搜了improve的一群东西,看着头疼,感觉没有gcc那边细节做得好,可能模块太多的缘故
minghegy
火狐狸
火狐狸
  • UID37982
  • 注册日期2011-12-17
  • 最后登录2014-02-24
  • 发帖数132
  • 经验10枚
  • 威望0点
  • 贡献值0点
  • 好评度0点
102楼#
发布于:2011-12-27 13:45
dongyuanxun:我下次再自己打打看看

是不是那些true类似的改成PR_TRUE的就行了
不知道10的一样不一样,到时再说

在mozilla bugzilla上搜了improve的一群东西,看着头疼,感觉没有gcc那边细节做得好,可能模块太多的缘故
回到原帖

应该是,按这个弄就行了
<!-- l --><a class="postlink-local" href="https://www.firefox.net.cn/forum/viewtopic.php?f=4&t=42966&start=75#p302106">viewtopic.php?f=4&t=42966&start=75#p302106</a><!-- l -->
Lawliet
火狐狸
火狐狸
  • UID34414
  • 注册日期2010-11-03
  • 最后登录2017-04-02
  • 发帖数201
  • 经验13枚
  • 威望0点
  • 贡献值0点
  • 好评度0点
  • 社区居民
  • 忠实会员
103楼#
发布于:2011-12-27 13:45
為了解決第一個patch打上會編譯錯誤的問題
我把我學生時代的c++課本又拿出來了
dongyuanxun
非常火狐
非常火狐
  • UID28632
  • 注册日期2009-04-19
  • 最后登录2013-02-14
  • 发帖数898
  • 经验10枚
  • 威望0点
  • 贡献值0点
  • 好评度0点
104楼#
发布于:2011-12-27 13:45
第一个没啥吧
我没记得有什么事故
游客

返回顶部