{"action":"created","issue":{"url":"https://api.github.com/repos/btcsuite/btcutil/issues/58","labels_url":"https://api.github.com/repos/btcsuite/btcutil/issues/58/labels{/name}","comments_url":"https://api.github.com/repos/btcsuite/btcutil/issues/58/comments","events_url":"https://api.github.com/repos/btcsuite/btcutil/issues/58/events","html_url":"https://github.com/btcsuite/btcutil/pull/58","id":112901007,"number":58,"title":"Transaction input and output sorting","user":{"login":"T909","id":3424974,"avatar_url":"https://avatars.githubusercontent.com/u/3424974?v=3","gravatar_id":"","url":"https://api.github.com/users/T909","html_url":"https://github.com/T909","followers_url":"https://api.github.com/users/T909/followers","following_url":"https://api.github.com/users/T909/following{/other_user}","gists_url":"https://api.github.com/users/T909/gists{/gist_id}","starred_url":"https://api.github.com/users/T909/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/T909/subscriptions","organizations_url":"https://api.github.com/users/T909/orgs","repos_url":"https://api.github.com/users/T909/repos","events_url":"https://api.github.com/users/T909/events{/privacy}","received_events_url":"https://api.github.com/users/T909/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-10-22T22:03:48Z","updated_at":"2015-10-22T22:54:45Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/btcsuite/btcutil/pulls/58","html_url":"https://github.com/btcsuite/btcutil/pull/58","diff_url":"https://github.com/btcsuite/btcutil/pull/58.diff","patch_url":"https://github.com/btcsuite/btcutil/pull/58.patch"},"body":"Here is the sorting function moved from btcd/wire (previous pull request: btcsuite/btcd#523 )\r\n\r\nI have added txsort_test.go but there are no transactions in block 10000 which are \"unsorted\" (most have only 1 input).  Transaction[1] from block 10001 is unsorted and would be changed by this function, but writing that all in by hand is annoying :) \r\n\r\nAre there other blocks formatted like block 10000 for testing already in btcd?  Or is there some utility to format blocks or txs?  If there is I can put more tests that actually change the txid."},"comment":{"url":"https://api.github.com/repos/btcsuite/btcutil/issues/comments/150378438","html_url":"https://github.com/btcsuite/btcutil/pull/58#issuecomment-150378438","issue_url":"https://api.github.com/repos/btcsuite/btcutil/issues/58","id":150378438,"user":{"login":"T909","id":3424974,"avatar_url":"https://avatars.githubusercontent.com/u/3424974?v=3","gravatar_id":"","url":"https://api.github.com/users/T909","html_url":"https://github.com/T909","followers_url":"https://api.github.com/users/T909/followers","following_url":"https://api.github.com/users/T909/following{/other_user}","gists_url":"https://api.github.com/users/T909/gists{/gist_id}","starred_url":"https://api.github.com/users/T909/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/T909/subscriptions","organizations_url":"https://api.github.com/users/T909/orgs","repos_url":"https://api.github.com/users/T909/repos","events_url":"https://api.github.com/users/T909/events{/privacy}","received_events_url":"https://api.github.com/users/T909/received_events","type":"User","site_admin":false},"created_at":"2015-10-22T22:54:45Z","updated_at":"2015-10-22T22:54:45Z","body":"Great point there, I modified it.  Using sort.IsSorted() should be faster and the code is cleaner looking too."}}