unofficial mirror of notmuch@notmuchmail.org
 help / color / mirror / code / Atom feed
* [PATCH] ruby: query: fix get sort
@ 2023-04-03 20:27 Felipe Contreras
  2023-05-25 10:05 ` David Bremner
  0 siblings, 1 reply; 2+ messages in thread
From: Felipe Contreras @ 2023-04-03 20:27 UTC (permalink / raw)
  To: notmuch

The order was wrong, right now `query.sort` doesn't return a number.

Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>
---
 bindings/ruby/query.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/bindings/ruby/query.c b/bindings/ruby/query.c
index 8a2b4d3d..077def02 100644
--- a/bindings/ruby/query.c
+++ b/bindings/ruby/query.c
@@ -45,7 +45,7 @@ notmuch_rb_query_get_sort (VALUE self)
 
     Data_Get_Notmuch_Query (self, query);
 
-    return FIX2INT (notmuch_query_get_sort (query));
+    return INT2FIX (notmuch_query_get_sort (query));
 }
 
 /*
-- 
2.40.0+fc1

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

* Re: [PATCH] ruby: query: fix get sort
  2023-04-03 20:27 [PATCH] ruby: query: fix get sort Felipe Contreras
@ 2023-05-25 10:05 ` David Bremner
  0 siblings, 0 replies; 2+ messages in thread
From: David Bremner @ 2023-05-25 10:05 UTC (permalink / raw)
  To: Felipe Contreras, notmuch

Felipe Contreras <felipe.contreras@gmail.com> writes:

> The order was wrong, right now `query.sort` doesn't return a number.
>

applied to master

d

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

end of thread, other threads:[~2023-05-25 10:06 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-04-03 20:27 [PATCH] ruby: query: fix get sort Felipe Contreras
2023-05-25 10:05 ` David Bremner

Code repositories for project(s) associated with this public inbox

	https://yhetil.org/notmuch.git/

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).