public inbox for libc-alpha@sourceware.org
 help / color / mirror / Atom feed
* [PATCH] Fix monetary.h comment
@ 2015-05-26 15:53 Marko Myllynen
  2015-05-26 19:16 ` Siddhesh Poyarekar
  0 siblings, 1 reply; 4+ messages in thread
From: Marko Myllynen @ 2015-05-26 15:53 UTC (permalink / raw)
  To: GNU C Library

Hi,

this should be obvious, please commit if looks to be ok.

2015-05-26  Marko Myllynen  <myllynen@redhat.com>

	* stdlib/monetary.h: Fix comment.


diff --git a/stdlib/monetary.h b/stdlib/monetary.h
index c0f254b..c78795b 100644
--- a/stdlib/monetary.h
+++ b/stdlib/monetary.h
@@ -42,7 +42,7 @@ extern ssize_t strfmon (char *__restrict __s, size_t __maxsize,
 #ifdef __USE_XOPEN2K8
 # include <xlocale.h>

-/* Formatting a monetary value according to the current locale.  */
+/* Formatting a monetary value according to the given locale.  */
 extern ssize_t strfmon_l (char *__restrict __s, size_t __maxsize,
                          __locale_t __loc,
                          const char *__restrict __format, ...)

Thanks,

-- 
Marko Myllynen

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] Fix monetary.h comment
  2015-05-26 15:53 [PATCH] Fix monetary.h comment Marko Myllynen
@ 2015-05-26 19:16 ` Siddhesh Poyarekar
  2015-05-26 20:44   ` Marko Myllynen
  0 siblings, 1 reply; 4+ messages in thread
From: Siddhesh Poyarekar @ 2015-05-26 19:16 UTC (permalink / raw)
  To: Marko Myllynen; +Cc: GNU C Library

[-- Attachment #1: Type: text/plain, Size: 314 bytes --]

On Tue, May 26, 2015 at 05:13:07PM +0300, Marko Myllynen wrote:
> this should be obvious, please commit if looks to be ok.
> 
> 2015-05-26  Marko Myllynen  <myllynen@redhat.com>
> 
> 	* stdlib/monetary.h: Fix comment.

Patch didn't apply to master, but I've fixed it up and pushed it.

Thanks,
Siddhesh

[-- Attachment #2: Type: application/pgp-signature, Size: 473 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] Fix monetary.h comment
  2015-05-26 19:16 ` Siddhesh Poyarekar
@ 2015-05-26 20:44   ` Marko Myllynen
  2015-05-27  9:12     ` Siddhesh Poyarekar
  0 siblings, 1 reply; 4+ messages in thread
From: Marko Myllynen @ 2015-05-26 20:44 UTC (permalink / raw)
  To: Siddhesh Poyarekar; +Cc: GNU C Library

Hi,

On 2015-05-26 21:24, Siddhesh Poyarekar wrote:
> On Tue, May 26, 2015 at 05:13:07PM +0300, Marko Myllynen wrote:
>> this should be obvious, please commit if looks to be ok.
>>
>> 2015-05-26  Marko Myllynen  <myllynen@redhat.com>
>>
>> 	* stdlib/monetary.h: Fix comment.
> 
> Patch didn't apply to master, but I've fixed it up and pushed it.

sorry about that (a whitespace hickup) - but your patch changed the wrong comment, so here's a new patch to fix the fix.

diff --git a/stdlib/monetary.h b/stdlib/monetary.h
index f3dafcd..c78795b 100644
--- a/stdlib/monetary.h
+++ b/stdlib/monetary.h
@@ -34,7 +34,7 @@ typedef __ssize_t ssize_t;
 
 __BEGIN_DECLS
 
-/* Formatting a monetary value according to the given locale.  */
+/* Formatting a monetary value according to the current locale.  */
 extern ssize_t strfmon (char *__restrict __s, size_t __maxsize,
 			const char *__restrict __format, ...)
      __THROW __attribute_format_strfmon__ (3, 4);
@@ -42,7 +42,7 @@ extern ssize_t strfmon (char *__restrict __s, size_t __maxsize,
 #ifdef __USE_XOPEN2K8
 # include <xlocale.h>
 
-/* Formatting a monetary value according to the current locale.  */
+/* Formatting a monetary value according to the given locale.  */
 extern ssize_t strfmon_l (char *__restrict __s, size_t __maxsize,
 			  __locale_t __loc,
 			  const char *__restrict __format, ...)

Thanks,

-- 
Marko Myllynen

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] Fix monetary.h comment
  2015-05-26 20:44   ` Marko Myllynen
@ 2015-05-27  9:12     ` Siddhesh Poyarekar
  0 siblings, 0 replies; 4+ messages in thread
From: Siddhesh Poyarekar @ 2015-05-27  9:12 UTC (permalink / raw)
  To: Marko Myllynen; +Cc: GNU C Library

[-- Attachment #1: Type: text/plain, Size: 216 bytes --]

On Tue, May 26, 2015 at 10:22:18PM +0300, Marko Myllynen wrote:
> sorry about that (a whitespace hickup) - but your patch changed the
> wrong comment, so here's a new patch to fix the fix.

Thanks, pushed.

Siddhesh

[-- Attachment #2: Type: application/pgp-signature, Size: 473 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2015-05-27  2:40 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-05-26 15:53 [PATCH] Fix monetary.h comment Marko Myllynen
2015-05-26 19:16 ` Siddhesh Poyarekar
2015-05-26 20:44   ` Marko Myllynen
2015-05-27  9:12     ` Siddhesh Poyarekar

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).