{"action":"created","issue":{"url":"https://api.github.com/repos/seqan/seqan/issues/1310","labels_url":"https://api.github.com/repos/seqan/seqan/issues/1310/labels{/name}","comments_url":"https://api.github.com/repos/seqan/seqan/issues/1310/comments","events_url":"https://api.github.com/repos/seqan/seqan/issues/1310/events","html_url":"https://github.com/seqan/seqan/issues/1310","id":121521339,"number":1310,"title":"PackedStrings for non-packables should be regular strings","user":{"login":"h-2","id":2996863,"avatar_url":"https://avatars.githubusercontent.com/u/2996863?v=3","gravatar_id":"","url":"https://api.github.com/users/h-2","html_url":"https://github.com/h-2","followers_url":"https://api.github.com/users/h-2/followers","following_url":"https://api.github.com/users/h-2/following{/other_user}","gists_url":"https://api.github.com/users/h-2/gists{/gist_id}","starred_url":"https://api.github.com/users/h-2/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/h-2/subscriptions","organizations_url":"https://api.github.com/users/h-2/orgs","repos_url":"https://api.github.com/users/h-2/repos","events_url":"https://api.github.com/users/h-2/events{/privacy}","received_events_url":"https://api.github.com/users/h-2/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/seqan/seqan/labels/bug","name":"bug","color":"fc2929"}],"state":"open","locked":false,"assignee":{"login":"smehringer","id":7955878,"avatar_url":"https://avatars.githubusercontent.com/u/7955878?v=3","gravatar_id":"","url":"https://api.github.com/users/smehringer","html_url":"https://github.com/smehringer","followers_url":"https://api.github.com/users/smehringer/followers","following_url":"https://api.github.com/users/smehringer/following{/other_user}","gists_url":"https://api.github.com/users/smehringer/gists{/gist_id}","starred_url":"https://api.github.com/users/smehringer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/smehringer/subscriptions","organizations_url":"https://api.github.com/users/smehringer/orgs","repos_url":"https://api.github.com/users/smehringer/repos","events_url":"https://api.github.com/users/smehringer/events{/privacy}","received_events_url":"https://api.github.com/users/smehringer/received_events","type":"User","site_admin":false},"milestone":{"url":"https://api.github.com/repos/seqan/seqan/milestones/7","html_url":"https://github.com/seqan/seqan/milestones/Release%202.0.2","labels_url":"https://api.github.com/repos/seqan/seqan/milestones/7/labels","id":1330833,"number":7,"title":"Release 2.0.2","description":"Hotfixes and minor changes.","creator":{"login":"esiragusa","id":172085,"avatar_url":"https://avatars.githubusercontent.com/u/172085?v=3","gravatar_id":"","url":"https://api.github.com/users/esiragusa","html_url":"https://github.com/esiragusa","followers_url":"https://api.github.com/users/esiragusa/followers","following_url":"https://api.github.com/users/esiragusa/following{/other_user}","gists_url":"https://api.github.com/users/esiragusa/gists{/gist_id}","starred_url":"https://api.github.com/users/esiragusa/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/esiragusa/subscriptions","organizations_url":"https://api.github.com/users/esiragusa/orgs","repos_url":"https://api.github.com/users/esiragusa/repos","events_url":"https://api.github.com/users/esiragusa/events{/privacy}","received_events_url":"https://api.github.com/users/esiragusa/received_events","type":"User","site_admin":false},"open_issues":26,"closed_issues":24,"state":"open","created_at":"2015-09-30T10:21:02Z","updated_at":"2015-12-11T11:08:34Z","due_on":"2015-12-10T23:00:00Z","closed_at":null},"comments":2,"created_at":"2015-12-10T16:11:12Z","updated_at":"2015-12-11T15:03:03Z","closed_at":null,"body":"One can create `String<short, Packed<Alloc<> > >` which does not make sense, because short cant be packed. Doing this not only increases overhead, but does not work, i.e. the tests fail in some cases.\r\n\r\nTODO:\r\n  * rename `String<TValue, Packed<THostspec> >` to `PackedString_<TValue, THostspec >`\r\n  * make a metafunction that `PackedType_<String<TValue, TSpec> >` that sets `Type` `PackedString_<TValue, Spec >` **iff** `BitsPerValue<TValue>::VALUE < sizeof(TValue) * 8`\r\n  * otherwise the metafunction should set `Type` to  `String<TValue, THostspec >`, i.e. the `Packed<>` is stripped\r\n  * finally `String<TValue, Packed<THostspec> >` can be defined as inheriting from the metafunctions `Type`\r\n * ultimately the tests can be added again"},"comment":{"url":"https://api.github.com/repos/seqan/seqan/issues/comments/163957941","html_url":"https://github.com/seqan/seqan/issues/1310#issuecomment-163957941","issue_url":"https://api.github.com/repos/seqan/seqan/issues/1310","id":163957941,"user":{"login":"h-2","id":2996863,"avatar_url":"https://avatars.githubusercontent.com/u/2996863?v=3","gravatar_id":"","url":"https://api.github.com/users/h-2","html_url":"https://github.com/h-2","followers_url":"https://api.github.com/users/h-2/followers","following_url":"https://api.github.com/users/h-2/following{/other_user}","gists_url":"https://api.github.com/users/h-2/gists{/gist_id}","starred_url":"https://api.github.com/users/h-2/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/h-2/subscriptions","organizations_url":"https://api.github.com/users/h-2/orgs","repos_url":"https://api.github.com/users/h-2/repos","events_url":"https://api.github.com/users/h-2/events{/privacy}","received_events_url":"https://api.github.com/users/h-2/received_events","type":"User","site_admin":false},"created_at":"2015-12-11T15:03:03Z","updated_at":"2015-12-11T15:03:03Z","body":"Ok, can you provide a few more hints to @smehringer on this? Thanks!"}}