Fire tongs are the tool(s) of the smelter

Bug #717749 reported by Hans Joachim Desserud
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
widelands
Fix Released
Low
Unassigned

Bug Description

I noticed the three tribes have pretty similar descriptions of the ware "fire tongs", with one small exception.

~/widelands/tribes$ grep -ri "fire tongs are the" .
./empire/fire_tongs/conf:help=_The fire tongs are the tool of the smelter, working in the smelting works. They are produced by the toolsmith.
./atlanteans/fire_tongs/conf:help=_The fire tongs are the tools of the smelter, working in the smelting works. They are produced by the toolsmith.
./barbarians/fire_tongs/conf:help=_The fire tongs are the tool of the smelter, working in the smelting works. Produced in the metal works (but ceases to be produced by the building if it is enhanced to an axefactory and war mill).

Atlanteans describe it as the "tools of the smelter" in plural, while barbarian and empire describe it as "tool" in singular. Wikitionary (http://en.wiktionary.org/wiki/fire_tongs) indicates fire tongs is always in plural, so I guess tools should be used in all descriptions.
I can probably make a patch to fix these, however, could anyone (preferably native English speaker) verify this is actually the correct form to use? It would be a shame to change it, only to have to correct it again later.

Widelands bzr r5825.

Tags: string-fix

Related branches

Revision history for this message
Chuck Wilder (chuckw20) wrote :

In my experience as a native American, the word "tongs" is, indeed plural, i.e. you never see "tong". However, each of these phrases are acceptable:
"Fire tongs are tools of the smelter."
"Fire tongs are a tool of the smelter."
"The tool of the smelter is fire tongs." ("is" refers to "the tool"-singular in this case).

Hope that clarifies things. :)

Changed in widelands:
status: New → In Progress
assignee: nobody → Hans Joachim Desserud (hjd)
Revision history for this message
Hans Joachim Desserud (hjd) wrote :

Thanks Chuck!

I have attached a branch with my suggested fix.

Revision history for this message
SirVer (sirver) wrote :

See my comment on the merge request. basically your suggestions are in trunk now.

Changed in widelands:
status: In Progress → Fix Committed
assignee: Hans Joachim Desserud (hjd) → nobody
Revision history for this message
SirVer (sirver) wrote :

Released in build16-rc1

Changed in widelands:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.