{"action":"created","comment":{"url":"https://api.github.com/repos/wkearn/RasterIO.jl/pulls/comments/40447626","id":40447626,"diff_hunk":"@@ -12,7 +13,24 @@ function Raster(dataset::GDALDatasetH)\n                     _getrastercount(dataset)) # number of bands\n     finalizer(raster, closeraster)\n     raster\n-end \n+end\n+\n+function Base.show(io::IO,raster::Raster)\n+     if (raster.dataset == C_NULL)\n+         println(io, raster.dataset)\n+         return null;","path":"src/raster_types.jl","position":15,"original_position":15,"commit_id":"cd7c5e57369470884fb98176ef78c517dfb9717f","original_commit_id":"cd7c5e57369470884fb98176ef78c517dfb9717f","user":{"login":"garborg","id":2823840,"avatar_url":"https://avatars.githubusercontent.com/u/2823840?v=3","gravatar_id":"","url":"https://api.github.com/users/garborg","html_url":"https://github.com/garborg","followers_url":"https://api.github.com/users/garborg/followers","following_url":"https://api.github.com/users/garborg/following{/other_user}","gists_url":"https://api.github.com/users/garborg/gists{/gist_id}","starred_url":"https://api.github.com/users/garborg/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/garborg/subscriptions","organizations_url":"https://api.github.com/users/garborg/orgs","repos_url":"https://api.github.com/users/garborg/repos","events_url":"https://api.github.com/users/garborg/events{/privacy}","received_events_url":"https://api.github.com/users/garborg/received_events","type":"User","site_admin":false},"body":"This is returning a function (making the return type unstable). Rather than explicitly returning `nothing`, either move it into the main if block or `return print...`.","created_at":"2015-09-25T16:20:01Z","updated_at":"2015-09-25T16:20:01Z","html_url":"https://github.com/wkearn/RasterIO.jl/pull/13#discussion_r40447626","pull_request_url":"https://api.github.com/repos/wkearn/RasterIO.jl/pulls/13","_links":{"self":{"href":"https://api.github.com/repos/wkearn/RasterIO.jl/pulls/comments/40447626"},"html":{"href":"https://github.com/wkearn/RasterIO.jl/pull/13#discussion_r40447626"},"pull_request":{"href":"https://api.github.com/repos/wkearn/RasterIO.jl/pulls/13"}}},"pull_request":{"url":"https://api.github.com/repos/wkearn/RasterIO.jl/pulls/13","id":45961378,"html_url":"https://github.com/wkearn/RasterIO.jl/pull/13","diff_url":"https://github.com/wkearn/RasterIO.jl/pull/13.diff","patch_url":"https://github.com/wkearn/RasterIO.jl/pull/13.patch","issue_url":"https://api.github.com/repos/wkearn/RasterIO.jl/issues/13","number":13,"state":"open","locked":false,"title":"Show and Print","user":{"login":"marciolegal","id":14054543,"avatar_url":"https://avatars.githubusercontent.com/u/14054543?v=3","gravatar_id":"","url":"https://api.github.com/users/marciolegal","html_url":"https://github.com/marciolegal","followers_url":"https://api.github.com/users/marciolegal/followers","following_url":"https://api.github.com/users/marciolegal/following{/other_user}","gists_url":"https://api.github.com/users/marciolegal/gists{/gist_id}","starred_url":"https://api.github.com/users/marciolegal/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/marciolegal/subscriptions","organizations_url":"https://api.github.com/users/marciolegal/orgs","repos_url":"https://api.github.com/users/marciolegal/repos","events_url":"https://api.github.com/users/marciolegal/events{/privacy}","received_events_url":"https://api.github.com/users/marciolegal/received_events","type":"User","site_admin":false},"body":"Second try... Please take a look. This version should print dimensions and band types.","created_at":"2015-09-24T23:13:17Z","updated_at":"2015-09-25T16:20:01Z","closed_at":null,"merged_at":null,"merge_commit_sha":"e344096811ef26f7598711df9bc23cc007ca1ccf","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/wkearn/RasterIO.jl/pulls/13/commits","review_comments_url":"https://api.github.com/repos/wkearn/RasterIO.jl/pulls/13/comments","review_comment_url":"https://api.github.com/repos/wkearn/RasterIO.jl/pulls/comments{/number}","comments_url":"https://api.github.com/repos/wkearn/RasterIO.jl/issues/13/comments","statuses_url":"https://api.github.com/repos/wkearn/RasterIO.jl/statuses/cd7c5e57369470884fb98176ef78c517dfb9717f","head":{"label":"marciolegal:master","ref":"master","sha":"cd7c5e57369470884fb98176ef78c517dfb9717f","user":{"login":"marciolegal","id":14054543,"avatar_url":"https://avatars.githubusercontent.com/u/14054543?v=3","gravatar_id":"","url":"https://api.github.com/users/marciolegal","html_url":"https://github.com/marciolegal","followers_url":"https://api.github.com/users/marciolegal/followers","following_url":"https://api.github.com/users/marciolegal/following{/other_user}","gists_url":"https://api.github.com/users/marciolegal/gists{/gist_id}","starred_url":"https://api.github.com/users/marciolegal/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/marciolegal/subscriptions","organizations_url":"https://api.github.com/users/marciolegal/orgs","repos_url":"https://api.github.com/users/marciolegal/repos","events_url":"https://api.github.com/users/marciolegal/events{/privacy}","received_events_url":"https://api.github.com/users/marciolegal/received_events","type":"User","site_admin":false},"repo":{"id":43029956,"name":"RasterIO.jl","full_name":"marciolegal/RasterIO.jl","owner":{"login":"marciolegal","id":14054543,"avatar_url":"https://avatars.githubusercontent.com/u/14054543?v=3","gravatar_id":"","url":"https://api.github.com/users/marciolegal","html_url":"https://github.com/marciolegal","followers_url":"https://api.github.com/users/marciolegal/followers","following_url":"https://api.github.com/users/marciolegal/following{/other_user}","gists_url":"https://api.github.com/users/marciolegal/gists{/gist_id}","starred_url":"https://api.github.com/users/marciolegal/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/marciolegal/subscriptions","organizations_url":"https://api.github.com/users/marciolegal/orgs","repos_url":"https://api.github.com/users/marciolegal/repos","events_url":"https://api.github.com/users/marciolegal/events{/privacy}","received_events_url":"https://api.github.com/users/marciolegal/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/marciolegal/RasterIO.jl","description":"Simple Raster Formats for Julia","fork":true,"url":"https://api.github.com/repos/marciolegal/RasterIO.jl","forks_url":"https://api.github.com/repos/marciolegal/RasterIO.jl/forks","keys_url":"https://api.github.com/repos/marciolegal/RasterIO.jl/keys{/key_id}","collaborators_url":"https://api.github.com/repos/marciolegal/RasterIO.jl/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/marciolegal/RasterIO.jl/teams","hooks_url":"https://api.github.com/repos/marciolegal/RasterIO.jl/hooks","issue_events_url":"https://api.github.com/repos/marciolegal/RasterIO.jl/issues/events{/number}","events_url":"https://api.github.com/repos/marciolegal/RasterIO.jl/events","assignees_url":"https://api.github.com/repos/marciolegal/RasterIO.jl/assignees{/user}","branches_url":"https://api.github.com/repos/marciolegal/RasterIO.jl/branches{/branch}","tags_url":"https://api.github.com/repos/marciolegal/RasterIO.jl/tags","blobs_url":"https://api.github.com/repos/marciolegal/RasterIO.jl/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/marciolegal/RasterIO.jl/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/marciolegal/RasterIO.jl/git/refs{/sha}","trees_url":"https://api.github.com/repos/marciolegal/RasterIO.jl/git/trees{/sha}","statuses_url":"https://api.github.com/repos/marciolegal/RasterIO.jl/statuses/{sha}","languages_url":"https://api.github.com/repos/marciolegal/RasterIO.jl/languages","stargazers_url":"https://api.github.com/repos/marciolegal/RasterIO.jl/stargazers","contributors_url":"https://api.github.com/repos/marciolegal/RasterIO.jl/contributors","subscribers_url":"https://api.github.com/repos/marciolegal/RasterIO.jl/subscribers","subscription_url":"https://api.github.com/repos/marciolegal/RasterIO.jl/subscription","commits_url":"https://api.github.com/repos/marciolegal/RasterIO.jl/commits{/sha}","git_commits_url":"https://api.github.com/repos/marciolegal/RasterIO.jl/git/commits{/sha}","comments_url":"https://api.github.com/repos/marciolegal/RasterIO.jl/comments{/number}","issue_comment_url":"https://api.github.com/repos/marciolegal/RasterIO.jl/issues/comments{/number}","contents_url":"https://api.github.com/repos/marciolegal/RasterIO.jl/contents/{+path}","compare_url":"https://api.github.com/repos/marciolegal/RasterIO.jl/compare/{base}...{head}","merges_url":"https://api.github.com/repos/marciolegal/RasterIO.jl/merges","archive_url":"https://api.github.com/repos/marciolegal/RasterIO.jl/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/marciolegal/RasterIO.jl/downloads","issues_url":"https://api.github.com/repos/marciolegal/RasterIO.jl/issues{/number}","pulls_url":"https://api.github.com/repos/marciolegal/RasterIO.jl/pulls{/number}","milestones_url":"https://api.github.com/repos/marciolegal/RasterIO.jl/milestones{/number}","notifications_url":"https://api.github.com/repos/marciolegal/RasterIO.jl/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/marciolegal/RasterIO.jl/labels{/name}","releases_url":"https://api.github.com/repos/marciolegal/RasterIO.jl/releases{/id}","created_at":"2015-09-23T22:08:42Z","updated_at":"2015-09-23T22:08:44Z","pushed_at":"2015-09-24T23:03:35Z","git_url":"git://github.com/marciolegal/RasterIO.jl.git","ssh_url":"git@github.com:marciolegal/RasterIO.jl.git","clone_url":"https://github.com/marciolegal/RasterIO.jl.git","svn_url":"https://github.com/marciolegal/RasterIO.jl","homepage":"","size":44413,"stargazers_count":0,"watchers_count":0,"language":"Julia","has_issues":false,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"wkearn:master","ref":"master","sha":"e352394555f23470401c7cfc5098170c1a6b73b9","user":{"login":"wkearn","id":4359487,"avatar_url":"https://avatars.githubusercontent.com/u/4359487?v=3","gravatar_id":"","url":"https://api.github.com/users/wkearn","html_url":"https://github.com/wkearn","followers_url":"https://api.github.com/users/wkearn/followers","following_url":"https://api.github.com/users/wkearn/following{/other_user}","gists_url":"https://api.github.com/users/wkearn/gists{/gist_id}","starred_url":"https://api.github.com/users/wkearn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/wkearn/subscriptions","organizations_url":"https://api.github.com/users/wkearn/orgs","repos_url":"https://api.github.com/users/wkearn/repos","events_url":"https://api.github.com/users/wkearn/events{/privacy}","received_events_url":"https://api.github.com/users/wkearn/received_events","type":"User","site_admin":false},"repo":{"id":16704804,"name":"RasterIO.jl","full_name":"wkearn/RasterIO.jl","owner":{"login":"wkearn","id":4359487,"avatar_url":"https://avatars.githubusercontent.com/u/4359487?v=3","gravatar_id":"","url":"https://api.github.com/users/wkearn","html_url":"https://github.com/wkearn","followers_url":"https://api.github.com/users/wkearn/followers","following_url":"https://api.github.com/users/wkearn/following{/other_user}","gists_url":"https://api.github.com/users/wkearn/gists{/gist_id}","starred_url":"https://api.github.com/users/wkearn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/wkearn/subscriptions","organizations_url":"https://api.github.com/users/wkearn/orgs","repos_url":"https://api.github.com/users/wkearn/repos","events_url":"https://api.github.com/users/wkearn/events{/privacy}","received_events_url":"https://api.github.com/users/wkearn/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/wkearn/RasterIO.jl","description":"Simple Raster Formats for Julia","fork":false,"url":"https://api.github.com/repos/wkearn/RasterIO.jl","forks_url":"https://api.github.com/repos/wkearn/RasterIO.jl/forks","keys_url":"https://api.github.com/repos/wkearn/RasterIO.jl/keys{/key_id}","collaborators_url":"https://api.github.com/repos/wkearn/RasterIO.jl/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/wkearn/RasterIO.jl/teams","hooks_url":"https://api.github.com/repos/wkearn/RasterIO.jl/hooks","issue_events_url":"https://api.github.com/repos/wkearn/RasterIO.jl/issues/events{/number}","events_url":"https://api.github.com/repos/wkearn/RasterIO.jl/events","assignees_url":"https://api.github.com/repos/wkearn/RasterIO.jl/assignees{/user}","branches_url":"https://api.github.com/repos/wkearn/RasterIO.jl/branches{/branch}","tags_url":"https://api.github.com/repos/wkearn/RasterIO.jl/tags","blobs_url":"https://api.github.com/repos/wkearn/RasterIO.jl/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/wkearn/RasterIO.jl/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/wkearn/RasterIO.jl/git/refs{/sha}","trees_url":"https://api.github.com/repos/wkearn/RasterIO.jl/git/trees{/sha}","statuses_url":"https://api.github.com/repos/wkearn/RasterIO.jl/statuses/{sha}","languages_url":"https://api.github.com/repos/wkearn/RasterIO.jl/languages","stargazers_url":"https://api.github.com/repos/wkearn/RasterIO.jl/stargazers","contributors_url":"https://api.github.com/repos/wkearn/RasterIO.jl/contributors","subscribers_url":"https://api.github.com/repos/wkearn/RasterIO.jl/subscribers","subscription_url":"https://api.github.com/repos/wkearn/RasterIO.jl/subscription","commits_url":"https://api.github.com/repos/wkearn/RasterIO.jl/commits{/sha}","git_commits_url":"https://api.github.com/repos/wkearn/RasterIO.jl/git/commits{/sha}","comments_url":"https://api.github.com/repos/wkearn/RasterIO.jl/comments{/number}","issue_comment_url":"https://api.github.com/repos/wkearn/RasterIO.jl/issues/comments{/number}","contents_url":"https://api.github.com/repos/wkearn/RasterIO.jl/contents/{+path}","compare_url":"https://api.github.com/repos/wkearn/RasterIO.jl/compare/{base}...{head}","merges_url":"https://api.github.com/repos/wkearn/RasterIO.jl/merges","archive_url":"https://api.github.com/repos/wkearn/RasterIO.jl/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/wkearn/RasterIO.jl/downloads","issues_url":"https://api.github.com/repos/wkearn/RasterIO.jl/issues{/number}","pulls_url":"https://api.github.com/repos/wkearn/RasterIO.jl/pulls{/number}","milestones_url":"https://api.github.com/repos/wkearn/RasterIO.jl/milestones{/number}","notifications_url":"https://api.github.com/repos/wkearn/RasterIO.jl/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/wkearn/RasterIO.jl/labels{/name}","releases_url":"https://api.github.com/repos/wkearn/RasterIO.jl/releases{/id}","created_at":"2014-02-10T18:29:04Z","updated_at":"2015-09-25T04:11:35Z","pushed_at":"2015-09-25T08:40:02Z","git_url":"git://github.com/wkearn/RasterIO.jl.git","ssh_url":"git@github.com:wkearn/RasterIO.jl.git","clone_url":"https://github.com/wkearn/RasterIO.jl.git","svn_url":"https://github.com/wkearn/RasterIO.jl","homepage":"","size":44967,"stargazers_count":12,"watchers_count":12,"language":"Julia","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":4,"mirror_url":null,"open_issues_count":5,"forks":4,"open_issues":5,"watchers":12,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/wkearn/RasterIO.jl/pulls/13"},"html":{"href":"https://github.com/wkearn/RasterIO.jl/pull/13"},"issue":{"href":"https://api.github.com/repos/wkearn/RasterIO.jl/issues/13"},"comments":{"href":"https://api.github.com/repos/wkearn/RasterIO.jl/issues/13/comments"},"review_comments":{"href":"https://api.github.com/repos/wkearn/RasterIO.jl/pulls/13/comments"},"review_comment":{"href":"https://api.github.com/repos/wkearn/RasterIO.jl/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/wkearn/RasterIO.jl/pulls/13/commits"},"statuses":{"href":"https://api.github.com/repos/wkearn/RasterIO.jl/statuses/cd7c5e57369470884fb98176ef78c517dfb9717f"}}}}