Bug 5127 - Reference to config/tc-alpha.c:736 missing in gas-2.17.90.pot
Summary: Reference to config/tc-alpha.c:736 missing in gas-2.17.90.pot
Status: RESOLVED INVALID
Alias: None
Product: binutils
Classification: Unclassified
Component: gas (show other bugs)
Version: 2.17
: P2 normal
Target Milestone: ---
Assignee: unassigned
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-10-05 21:10 UTC by Jorma Karvonen
Modified: 2007-10-08 15:43 UTC (History)
1 user (show)

See Also:
Host:
Target:
Build:
Last reconfirmed:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Jorma Karvonen 2007-10-05 21:10:27 UTC 
Reference to config/tc-alpha.c:736 missing in gas-2.17.90.pot (here below)

#: config/tc-mcore.c:1987
msgid "unknown"
Comment 1 Nick Clifton 2007-10-08 15:16:58 UTC 
Hi Jorma,

  I do not understand what you mean with this bug report.  Please could you
explain in some more detail ?

Cheers
  Nick
Comment 2 Jorma Karvonen 2007-10-08 15:36:07 UTC 
(In reply to comment #1)
> Hi Jorma,
> 
>   I do not understand what you mean with this bug report.  Please could you
> explain in some more detail ?
> 
> Cheers
>   Nick
> 

Sorry,

I did not explain this correctly. The source code looks like this:

static void
debug_exp (expressionS tok[], int ntok)
{
  int i;

  fprintf (stderr, "debug_exp: %d tokens", ntok);
  for (i = 0; i < ntok; i++)
    {
      expressionS *t = &tok[i];
      const char *name;

      switch (t->X_op)
	{
	default:	name = "unknown";	break;

In gas-2.17.90.pot is only one "unknown" msgid (for config/tc-more.c:1987)

I think that this debug message should be translated also (and there should be a
reference to config/tc-alpha.c:736 in the commment of the msgid "unknown" in
gas-2.17.90.pot).

best regards,

Jorma
Comment 3 Nick Clifton 2007-10-08 15:43:53 UTC 
Hi Jorma,

  OK, I understand now, but a fix is not necessary.  This code is used only when
debugging the alpha port of the assembler.  It is not enabled by default and no
normal user will ever see it.  Thus it does not need any translation.

Cheers
  Nick