From d1cb424ce904a5b70fb357a4f1c61f5229a4bc14 Mon Sep 17 00:00:00 2001 From: Sinan Date: Fri, 28 Sep 2018 12:26:13 +0200 Subject: [PATCH 2/5] Remove the ascii-string default serializer Thus a string will always be be serialized in a utf-8-string and thus utf8 will be favored. A strategy to resolve multiple converters for a single type is missing. --- rsb-cil/Rsb/Converter/DefaultConverterRepository.cs | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/rsb-cil/Rsb/Converter/DefaultConverterRepository.cs b/rsb-cil/Rsb/Converter/DefaultConverterRepository.cs index 9cf1b8e..c3c5a52 100644 --- a/rsb-cil/Rsb/Converter/DefaultConverterRepository.cs +++ b/rsb-cil/Rsb/Converter/DefaultConverterRepository.cs @@ -45,9 +45,9 @@ namespace Rsb.Converter new IsExactMatch(signature.Schema), converter )); - - // this will add a second string converter in the informer list. - // I assume the first one wins and i don't think, that this is intended. Checks and Exceptions should be added at sometime + + //To prevent informers to send ascii-strings. A strategy to choose multiple solution for one type is still missing. + if (converter.GetSignature().Schema.Equals("ascii-string")) continue; serializationConverterList.Add( new Tuple.Predicate, IConverter>( -- 2.14.2.windows.1