{"comment":{"url":"https://api.github.com/repos/chrisbahn/ITEC2545-rummy/comments/10074937","html_url":"https://github.com/chrisbahn/ITEC2545-rummy/commit/23055cc49ab3b9d599d5515d7b1e632b76df9317#commitcomment-10074937","id":10074937,"user":{"login":"chrisbahn","id":11052019,"avatar_url":"https://avatars.githubusercontent.com/u/11052019?v=3","gravatar_id":"","url":"https://api.github.com/users/chrisbahn","html_url":"https://github.com/chrisbahn","followers_url":"https://api.github.com/users/chrisbahn/followers","following_url":"https://api.github.com/users/chrisbahn/following{/other_user}","gists_url":"https://api.github.com/users/chrisbahn/gists{/gist_id}","starred_url":"https://api.github.com/users/chrisbahn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/chrisbahn/subscriptions","organizations_url":"https://api.github.com/users/chrisbahn/orgs","repos_url":"https://api.github.com/users/chrisbahn/repos","events_url":"https://api.github.com/users/chrisbahn/events{/privacy}","received_events_url":"https://api.github.com/users/chrisbahn/received_events","type":"User","site_admin":false},"position":null,"line":null,"path":"","commit_id":"23055cc49ab3b9d599d5515d7b1e632b76df9317","created_at":"2015-03-06T16:22:47Z","updated_at":"2015-03-06T16:22:47Z","body":"<!-- DIV {margin:0px;} -->I'm going to go backwards and restart from an earlier version. The CardSet superclass isn't working the way I thought it would, and I can't figure it out. I talked with one of my friends about what we'd written so far, and she suggested getting rid of CardSet and arranging things in a different way. My plan is now to get rid of CardSet, Run, DiscardPile and Hand as separate classes, and use Deck for all collections of LinkedList<Card>. Sorry to make such a big change, but my friend suggested the program was going to become way too complex for us otherwise.-----Original Message-----\nFrom: Lope922 \nSent: Mar 5, 2015 11:44 PM\nTo: chrisbahn/ITEC2545-rummy \nCc: Christopher Bahn \nSubject: Re: [ITEC2545-rummy] Changed all ArrayLists to LinkedLists (23055cc)\n\nChris i think the pick a card class will eventually become a menu of the possible melds the player will have to choose from that the sequence tester will print out the cards that fall into the meld criteria.  Looks pretty solid so far though. thanks for commenting your code.  \n\n—Reply to this email directly or view it on GitHub."}}