670487 - Fixing rl_ttyset prototype in js/src/editline.

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Cyril Roelandt]

Attachment: Fix the prototype of rl_ttyset().

User Agent: Mozilla/5.0 (X11; Linux i686; rv:5.0) Gecko/20100101 Firefox/5.0
Build ID: 2011061500

Steps to reproduce:

I compiled the trunk version of Firefox.


Actual results:

It compiles fine and Firefox works perfectly, but the compiler (GCC 4.5.1) prints out these warnings :

mozilla/js/src/editline/editline.c:1026:5: warning: call to function ‘rl_ttyset’ without a real prototype
mozilla/js/src/editline/editline.h:118:13: note: ‘rl_ttyset’ was declared here
mozilla/js/src/editline/editline.c:1037:5: warning: call to function ‘rl_ttyset’ without a real prototype
mozilla/js/src/editline/editline.h:118:13: note: ‘rl_ttyset’ was declared here


Expected results:

These warnings should not appear. Though one should not really worry about them, they pollute the compiler's output. The attached patch should fix this.

Comment 1

[Brendan Eich [:brendan]]

Comment on attachment 545043 [details] [diff] [review]
Fix the prototype of rl_ttyset().

Thanks for the patch -- wow, this code is old.

Ask around on irc.mozilla.org #jsapi for help landing this, please.

/be

Comment 2

[(no longer active)]

http://hg.mozilla.org/mozilla-central/rev/9bc361598877