[rfc][icedtea-web] rememberable dialogues general solution - impl part 1 - safe types returned from getUserResponse
Andrew Azores
aazores at redhat.com
Fri Jun 5 15:36:59 UTC 2015
Hi! Generally liking where this patch is going.
Jiri Vanek <jvanek at ...> writes:
>
> hi!
>
> So this is first part.
> Tho most nasty and error prone.
>
> It is changing type of securityMEssage.response from Object to regular
interface, and is providing
> useful implementations of this interface.
>
> The legacy object was integer at first, and was used to preselect button,
and tro return id of
> pressed button. LAter was missuesd and returned yes/no/always/never even
later sanbox was added and
> last drop was complex result about shortcuts.
I much prefer your new approach, obviously much better than ints and Object[]s.
>
> In this patch, there are few wrappers for types of yAnNS, which contains
also remember logic. This
> will be fixed in another changeset, and those wrappers will be gone.
Does this mean the Yes, No, YesNo, YesNoCancel, etc. classes? And "Garbage"?
>
> Approach of this patch i slightly different, less verbose, with a lot of
shared code and different
> typesafety. I hope it is generally better. If somebody found tech-review's
solution better, or
> generally have better ideas, pelase let me know.
>
> J.
>
> Attachment (makeTypesOfSecurityDialogMessageReturnValueSafe.patch):
text/x-patch, 84 KiB
Thanks,
Andrew Azores
More information about the distro-pkg-dev
mailing list