mimeapps - finding association between MIME types and applications
FreeSlave via Digitalmars-d-announce
digitalmars-d-announce at puremagic.com
Sat Apr 16 16:10:13 PDT 2016
On Saturday, 16 April 2016 at 21:58:35 UTC, ag0aep6g wrote:
> On 16.04.2016 20:48, FreeSlave wrote:
>> github repo: https://github.com/MyLittleRobo/mimeapps
>> (examples included)
>
> You've got some bad `@trusted`s in your code:
>
>
> <https://github.com/MyLittleRobo/mimeapps/blob/03b53ce35d2f4f56fdf76e0b6f5c681660b9d23a/source/mimeapps.d#L29>:
>
> The constraint allows user-defined types that implicitly
> convert to const(char)[] via alias this. Such a type may, for
> example, have an unsafe opAssign, which you're mistakenly
> trusting here.
I would better use inout here, but it has problems with Tuples. I
don't know how to rewrite this code to inout so it still remained
buildable. The other way of course is to rewrite constraints to
match real strings only not types with alias this.
><https://github.com/MyLittleRobo/mimeapps/blob/03b53ce35d2f4f56fdf76e0b6f5c681660b9d23a/source/mimeapps.d#L191>,
> <https://github.com/MyLittleRobo/mimeapps/blob/03b53ce35d2f4f56fdf76e0b6f5c681660b9d23a/source/mimeapps.d#L308>:
>
> Similarly, IniLikeReader may do unsafe stuff and can't be
> trusted.
This is usually constructed from iniLikeStringReader or
iniLikeFileReader from inilike library, but yeah, generally it
can be non-trusted user-defined type. Thanks for pointing out.
Will fix it.
More information about the Digitalmars-d-announce
mailing list