OK! muy buen trabajo...si podemos refinar el uso de métodos en el futuro!!!


Thanks for the PR.
I thought, that git am does commit automatically.
Am I right?

From man git am:
Splits mail messages in a mailbox into commit log message, authorship information and patches, and applies them to the current branch.


badly named test method
also, returned value "false" is not tested (true could be hardcoded as far as the test goes"


ObjexxFCL_Update (DeadParrot) - i386-Windows-7-VisualStudio-12: OK (1909 of 1909 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Good Catch! Thanks!


@frenchcomp note that it only affects function and method Closures like you get from Reflection. Normal Closures (`$f = function(...) use (...) {};`) can still be rebound etc. This is exactly where `$newscope` parameter still has its use.

Also, in general, when writing a method, one should be able to assume that we are always referring to *the* self of *this* class and never anything else.

Also, randomly preventing getClosure() on some functions isn't an option. Would be weirdly inconsistent...
We don't really want to make exceptions based on some implementation detail of the function…


@branto1 Hmm, you're right: https://en.opensuse.org/SDB:SELinux . . . I agree, `%bcond_with selinux` for SUSE makes sense.


Aha, I see your point. You are right. For tracking propose and finding package is git commit log.


Problem is those links don't work with `https`


@mbellotti I think we need to discuss about these changes. I see that in mail_validation_controller.py
 it is removed the captcha validation.


Please use 4 spaces for indentation.


make it nice


!mustfix Don't entirely like that fact that `Module` interface is in `config/loader` package.


should be "when_there_is_no_snapshot" or something of the kind


I noticed you use this also in some other places in spec file. Just wondering: Why don't you use systemctl try-restart ceph.target directly, here?


account_job_in_status() is named in accordance to current implementation of Python worker (see file storage.py). When the group is BAD, the last chance is when we have a job on some group, in this case we turn the couple into SERVICE_ACTIVE/SERVICE_STALLED.


Thanks! adding this commit


probably if we can add `PhotoGalleryRenderGallery` hook once we have file captions parsed so we'll not have to parse them here. It's not very nice.


what if there is only one snapshot and it's not last = true?


ok cool


Great!


Yes they do... it even says on their site to use https and they suggest it as a default?

![screen shot 2015-10-05 at 11 01 57 pm](https://cloud.githubusercontent.com/assets/6525926/10280479/2630c8ba-6bb5-11e5-809f-c810e2b0f63d.png)


indent this block -2


Thanks Michal +1
One comment, I think one can differentiate two types of time outs
1) Timeouts after which the deploy exits all together with an error. I think these timeouts can be set fairly high.
2) Timeouts after which an alternative method is used, or when the error is not fatal. Here one need to be a little bit more careful on the timeout value.


indent -2


https://ghbtns.com/github-btn.html?user=mottie&repo=tablesorter&type=watch&count=true <-- change &amp to & and it works fine.


with this changes, `else` is now useless


Tag den, @frejsoya.


@arangari Please review changes.


mention amt of oil


!mustfix config.EnsureValueFilled: Handle nil pointers properly


Internal status is now removed.

As for SERVICE_ACTIVE_ServiceActive, yep, looks weird, this naming convention was used to understand in what reason group is in a certain state. For example, it can be BAD when one of groups has metadata different from other groups, or when some group is read-only. But there's only one case when it is in state SERVICE_ACTIVE.


!mustfix Using emptyModuleConfigContainer is super ugly, find a better solution.


I tend to put the `thrown` property initialization right before the statement which is supposed to raise the exception. This way, it the rest of the method throws a similar exception, the test will fail and won't false negative


update functions documentation to reflect the correct return type (iterable of Proteins)


!mustfix It is not clear whether the local of global config is affected and what section exactly.


!mustfix Don't entirely like that fact that Module interface is in config/loader package.


From what I see, no test has been added to check that aggregation is well working on project without file.


@bwoebi Else, why to not provide an additional method from the Reflecion api to get a real closure, to use at our perils ?


From what I see, no test has been added to check that aggregation is well working on empty views.


ObjexxFCL_Update (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (835 of 835 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-24.31%25-green.svg)


Usually I group headers of different origin: our headers (like Round.h), STL headers, and headers of external libraries.


Ja, definitiv.


Yeh, hmm.


buen intento y manejo de las opciones


OKです！


@lunaroyster @thel3l change all to dp. Device Pixels. All browsers make it look good


feito, vamos anunciar já o release!


Don't really have time to investigate this.


**[unitTests_js_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_js_branches/19138/testReport/) for this commit**


@frenchcomp Tell me … why?


Because it doesn't work :-)


we gone be alright (ngr)


As INT_SUM_FORMULA and INT_SUM_FORMULA_NULL_DEFAULT_INPUT_VALUE are the same, I suggest to keep only one


Not Knwoleg


fdfsdfs


Should be moved to the end


**[unitTests_js_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_js_branches/19139/testReport/) for this commit**


:+1: Thanks


Initialization stuff is made once at program's start, we don't have any lazy initialization. What exactly should I describe?


**[unitTests_js_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_js_branches/19140/testReport/) for this commit**


Nope.


To limit your BC.., sorry you don't want to listen the community... i search alternative solution


Because shitty boost macros begin to conflict when I include cocaine headers and elliptics headers in "wrong" order.


With 1 000 000 entry in the orders sales_flat_order_item table:
without index, without view => 50 sec to reindex one (demo) store
with index, with view => 23 sec
with index, without view => 3 sec

Results weren't similar on with only few records.
So letting the indices and removing the view seems like the way to go

cc @redox


I think that would be a great way to go.


All right, let's remove that.


This output format is used only for manual debug purposes. Monitoring scripts will use log files to get the same numbers and more (like round times, parsing times, etc.). So please do not waste your time discussing it anymore. @v-nikulichev, please mention it in comments.


We indicate error. Rapidjson stops processing when we return false.


line too long.


See comment above.


Nice work, looks like this resolved a couple of architectural challenges too!


> It's one from a number of ways. Why not print(std::ostream&)?

There are a lot of ways we can imagine to solve a problem. And there just few that are common. ```operator <<``` is a C++ way to print objects.


@frenchcomp I want to know for what actual reason (where) this possibility would be useful.


Unit tests are missing


Reposting in case it got lost in the rebase:

@rnewman: I'm having a problem with traversal of the folder hierarchy in navigation after rebasing this on your bookmarking code.

While profile.bookmarks.modelForFolder(BookmarkRoots.MobileFolderGUID).upon(onModelFetched) provides me with the correct model from MergedSQLiteBookmarks (treating mobile___ as root), the modelFactory that is returned for that model is a SQLiteBookmarkMirrorStorage, rather than a MergedSQLiteBookmarks. This mean that when I traverse back up the folder structure and fetch root again, it fetches the root from SQLiteBookmarkMirrorStorage which is empty.

Any pointer on how this can be resolved, while retaining the correct SQLiteBookmarkMirrorStorage functionality?


A bit short on the info in the commit message.


I'd better did, but I left as little number of raw pointers as possible after our discussion.


my bad will restore this


They are not. From a user point of view, they are not using the same constructor.


Ah, no.. updated to do an explicit remove_session.

Also added a con->is_connected check at the top so we can throw those out immediately.


not the the end but one method below, I agree


please use None, instead of default mutable types


Don't forget to push this commit :>


I'll take a look.


On Mon, 5 Oct 2015, branto1 wrote:
> Hmm, afaik there is no need to create the (/var)/run/ceph in ceph-common as no daemons will write there. Looking at the ceph-common file list
> that initiated the original move, /var/lib/ceph and /var/log/ceph directories require the ceph user/group. AFAIK, these are server-side
> directories and do not need to be created for client-side utils. Maybe, these all could be moved to the ceph package.
>
> It would be nice if @liewegas could confirm that we do not need /var/{lib,log}/ceph for client side utils?

I think they shoudl be in ceph-common so that librados users etc can log
there (e.g., radosgw).  Same goes for /var/run/ceph.  Note that client
side stuff can also set up the admin socket files that go in
/var/run/ceph...


index.html - wywalone zbędne logo, poprawka w główce strony


aside from questions of the `if (ringpop.<foo>)` checks, lgtm


;5 whoops


ну с АМО ты загнался, можно же было
link.setAttribu7e('onmouseover', 'vk_videos.init_animated_thumb(this);')


ok


Oh, good! Please update the pull request. Thanks!


better, indeed !


not bad


To perform like script in my first commit. It's a method to allow to write some library allowing final user to implemant easily some pattern. It's useful for fe other reasons, search on google and look it. Several projects use this method. (One of my libraries use it, https://github.com/UniAlteri/states by example , but i work on an alternative...)

But it is only because you introduce a major BC break to fix a minor BC break WITHOUT consulting the community ...


right


Use this instead:
    config_get: "show running-config ntp"

The "show runn <feature>" commands are more efficient than filtering the entire config.


please ensure jira version in setup.cfg
please update README accordingly


indeed !


Fixed in https://github.com/xwiki/xwiki-platform/commit/0d1a68588a849beb06ed2af06d4eb04dd7dc712f. Thanks!


Like in this amended commit? https://github.com/apjanke/homebrew/commit/5b0afb74d8576dae887445f402e7b319711fb8d0


indeed


This change is not complete. If TableLogAction/Cache is stored in the DI container, you do not need to use `Config::getInstance()` or `StaticContainer`. Should be changed to:

```
public function __construct(LoggerInterface $logger, Config $config)
{
        $this->isEnabled = (bool)$config->General['enable_segments_subquery_cache'];
        $this->limitActionIds = $config->General['segments_subquery_cache_limit'];
        $this->lifetime = $config->General['segments_subquery_cache_ttl'];
        $this->logger = $logger;
}
```

You can also get rid of `$hits` by creating a mock in tests.


This generates the SQL `not (this_.client_id in ())`. which works in H2 but not anywhere else.


Not managed by the code right now.


What is the difference ?


Nice shit


Though I appreciate the desire to completely rule out socketId conflicts, I think you can safely get away with using a monotonically increasing timestamp and save yourself quite a few bytes by using: like so:

var timestamp = require('monotonic-timestamp');
...
this.socketId = this.conf.prefix + '-' + timestamp().toString(36);


Could also be expressed this way:
```
if start_parsing
  #puts "#{row[0]} sys #{row[column]}"
  time.push(count - 7)
  values.push(row[column])
else
  start_parsing = true if count == 7
end
```


yeah poor choice of wording on my part :disappointed: will switch to `tail`


weird, no? I'll get the latest rc and try to track this down


:thumbsup:


I fixed thi sin player class.


It should never happen, and if it happen it means that there's no previous analysis (dashboard will be empty).


**Functionality - Awesome**
Great application of _dependency injection_.


commit message issues:
- the headline is not clear enough
- the content of the patch concerns the FloatingPoint parameters, not the FixedPoint.


I don't see the difference...


Yes, I think remove it and just report 0 if device is not there. I think we will document that this parameter reports the current size of base device and not the desired size (as specified by dm.basesize= option).


I would disable that urllib3 warnings for verify=False


**Code Quality - Critical**
We don't need `header_image`, and it should be camel cased.
It was created by @kn9ts a month ago, to test around Angular's data binding. We don't need it anymore.


How about calling it BaseDeviceSize?


I would keep True, in order to comply with previous behavior


We don't need this anymore since we support only Xcode 6 & 7 and they both come with XCTest. But I can clean it everywhere later.


Let us return 0 for the unlikely scenario of base device not being present.


You probably don't want to hard code this. Use `url_for`.


Oh shit, I'm sorry I forgot to uncomment that out ^_^;;; Fixed now


@frenchcomp If I see that right, you need random $this binding, but actually you don't need random scope binding (yeah, you are specifying a scope, but do you need it?)...?


"Starting"


It's not a data- attribute that's missing, you've got other JS problems if you're hitting this.


will update to have final newline


You mean like in teuthology which runs the daemons on its own or altogether? AFAICR, it worked fine for me if I started the daemons with systemd and it did not work with teuthology-started daemons even if I tried to restart the daemons with systemd directly via the ceph-mon/osd/mds services.


IllegalArgumentException


You don't handle octal base:

    [+-]?0[0-9]+


update to input type


いいね！他の関数リファクタリングするなら feature/refactor でやった方がいいのかな？
あと，このブランチ PR でてないね


In that case, lgtm. :)


Very exciting, thank you for working on this!


PHPCS: Opening brace should be on a new line


the same, as above regarding to mutable types


Thanks!


Ok, please look at https://github.com/pixijs/pixi-spine/commit/07f1902b0b4a726d6401d9c911a47c1e0cfe9b17 and use latest build


@pynicolas I think this extension is not necessary anymore. Also maybe the class could extend PHPCheckTest, which would avoid to prefix the call to ``PHPCheckTest#check()``, WDYT ?


This is the 3rd assignment for my Java class


Why is it here? Won't it work without it? I guess it won't because XCTest queries only for `test*` methods.

Do we want to support regular test method names to contain parameters? Like `testMethodWithParamater:(int)value`? I mean that won't require implementing `testInvocations`.


No, in SUSE we are not starting any daemons except via systemd. In SUSE, ceph.target unit works with start and stop only IIRC.


Where are the blobs from?
Will it break auto rotation on certain device?


Since this is just alpha-equivalence, my goal here is performance (at least asymptotic, plus easy-to-get constant factors). Hence:

> Since you never actually substitute, you can map bound names to their de Bruijn level (!) instead of mapping them to a fresh name. That is, use Map[Name, Int] instead of Map[Name, Name] and a simple counter instead of fresh().

Yep, that saves needless work.

> Or maybe you want to switch to the locally nameless style eventually?

Not learned that one yet. More importantly, that affects also the term representation, so there are more constraints.

> Why should ABTs ever expose the difference between alpha-equivalent terms?

Good question. I know two reasons why this design choice is interesting.
1. And in practice, each time I implemented a language I also wanted the original names when inspecting terms, and it was a pain when I didn't have them. So I'd keep around names for debugging. If I do that, I should probably also get their handling right, making other choices redundant. However, since I didn't yet prove this code correct, redundancy and defensive programming might make sense.
2. I understand that's part of Harper's interface (just double-checked), both in his math and in other implementations of ABTs I've seen. (In particular, see [neelk's blog post](http://semantic-domain.blogspot.de/2015/03/abstract-binding-trees.html)).


LGTM!


@bwoebi No, Currently, I extract a method from a class to add it on fly (during the execution, according this state) to a proxy... So, $this is bind on the proxy instance.


symlink /system/vendor /vendor
and
TW_IGNORE_ABS_MT_TRACKING_ID := true
are needed for fixing touch problem


"config:" is pretty vague since there are other ntp configurations:

/config:/source_interface:/


I don't think I understand. This code takes part in storage snapshot. What should it describe?


#2 Finished


dang, had it as resource first and changed it at the last minute.  Will fix


I wonder if this whole `DataSource` section can be eliminated via Boot autoconfig.


@frenchcomp Hmm

Because supporting random $this binding isn't an issue. Just changing scope is. (because self).
(I thought a fitting $this would be necessary, but actually it isn't.) I could relax that restriction.

Is this an okay-ish compromise here?


Could be… I didn’t see any commits in the base code that should affect it when I merged with latest codebase, but I could have missed something.

Thanks

From: nehresma [mailto:notifications@github.com]
Sent: Monday, October 05, 2015 5:39 AM
To: GSBCfamily/bvcms <bvcms@noreply.github.com>
Cc: Heath Kouns <hkouns@kounsfamily.net>
Subject: Re: [bvcms] Merge pull request #39 from nehresma/cc_box_on_email (391196f)

Hmm. Yep, will do. My email config is a little wonky -- I've got it set up to deliver to a personal email server with a self signed cert, so I have a few small changes locally that I didn't commit (to override the .Net TLS key errors from a self signed certificate). Possibly that's the difference here? I'll check it out.

—
Reply to this email directly or view it on GitHub<https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_GSBCfamily_bvcms_commit_391196fdc893d7223f2d18e5bb21f4663db840a4-23commitcomment-2D13587871&d=AwMCaQ&c=ByDrzdYw8tO08sJlHDO_Vg&r=iGjRQCxUMWIh2KzlFV0J1Q&m=CZXDzMBfiKq69GgbOMw5rEylxqRL8fTntjBwT-rEa1s&s=HF63rnyEHguv5ZPCL_4dmNo4_fJQKi9XyydKsrrXzvs&e=>.


#4 Finished


In recent version I removed all separate metadata getters and return a reference to Metadata structure.


I recommend using textMessage as well. Some mail readers will not render HTML.


PHPCS: There must be one blank line after the last USE statement; 2 found;


Method BuildService::addBuildToQueue does not have a Docblock comment.


Class BuildWorker does not have a Docblock comment.


I remember seeing the case when I would end up with 2 selections in the modal tree, both the old selection and the new one. As the old element was before my new element and the tree was configured to use a single selection, the old (first) selection was used and the test would fail.

My conclusion was that the deselection was faulty (i.e. not synchronous) so I focused on the time gap between the deselection and the new selection, making extra sure that until we go any further, the tree is clear both in the DOM and in the JSTree API.

Personally, I`m not ruling out any bugs in the JSTree widget, since the experienced behaviors should not be considered normal, but anyway.


Adenilton bom dia,
Nesta nova atualização não estou conseguindo carregar a Classe Impressao somente aparece as demais classes...

e na classe Certificado esta dando erro no seguinte evento

PropertyChanged?.Invoke(this, new PropertyChangedEventArgs(propertyName));

Pode ser incompatibilidade com o meu VS2012.


@claudiosmweb can you double check this and put in 2.4?


I will add comments. Mongo API is really strange.


@pynicolas very minor: FYI [VisitorCheck.java#visitClassDeclaration()](https://github.com/SonarCommunity/sonar-php/blob/master/php-frontend/src/main/java/org/sonar/plugins/php/api/visitors/VisitorCheck.java#L136) should visit both Class and Interface declarations.


@pynicolas idem: extension of CheckTest


LGTM!


let's extract "for/while" and othe magic strings into one class named TheRNames, so we can reuse them


No, good idea.


Please consider change this `class Admin <Index` to `class Admin < self`

When I use `Entities::Organization::Index::Admin.present({}).model` this error is showed: `NameError: uninitialized constant Entities::Organization::Index::Admin`


Just a preference -- this code never throws exceptions.


Ok


Fixes #1


Instead of checking `can_configure`, we should probably be checking something more like `$scope.canIInteractWithMultiplePlaceTypes`...


Eeee, czyli co? Chyba nie powinno być u mnie błędu, bo mi chodzi o wartości
mieszczące się w 8-iu bitach. Czy coś jeszcze?

2015-10-05 12:40 GMT+02:00 Marcin Kubik <notifications@github.com>:

> http://stackoverflow.com/questions/11600378/portable-code-bits-per-char
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/martakuzak/MP4_2/commit/ba2943e59ca175d3cf1d1bce232f69ea265f97ec#commitcomment-13588946>
> .
>


its like saying "it_rains_because_it_rains" or "it_rains_because_water_is_falling_from_the_sky"


Didn't get how commit message corresponds to this change )


cfc


!mustfix Rename back to `tool` again.


Yes, it's a good compromise, thanks :)


Text in some games will be broken by this change until we port over more changes.


At that point, the user's token has changed to the one provided from the confirmEmail() function. You'll want to update the user's cookie to reflect that change.


@hvelarde não esquecer de atualizar os buildouts do brasil.gov.portal, brasil.gov.temas e brasil.gov.paginadestaque.


well, if I change `setLast(true)` to `setLast(false)` there should be a test enforcing that we expect a different result
see my comment on previous method, you will see the point


unclear


ditto


The 'create' and 'destroy' methods shouldn't be used this way. You should either remove them altogether or use them to enable/disable 'feature ntp' (my preference).

For source_interface you need to create getter/setter/default methods, e.g.:

def source_interface
def source_interface=
def default_source_interface


@hasufell
I'm not sure what exactly you are proposing or what the real problem is. Please spell out what you propose for a change. Thanks!


!mustfix Change to `salsaflow.modules.codereview.github`.


Если развертка одна, то да.


Based on the wiki I think it's suppose to be ```ms_pictureBufferSize```?


this is my problem

sorry for my low englishbest regardFernando Fernando Valdes C.


@thongsav-usgs did you mean to overwrite this? I think this is changed, same with sublocation.


I like this kind of guards so much.

From uuidgen manpage:
> The new UUID can reasonably be considered unique among all UUIDs created on the local system, and among UUIDs created on other systems in the past and in the future.


@pynicolas idem: CheckTest


Same here: ```ms_movieErrorBufferSize```


@tedsharpe Can you offer any comments on why the file src/main/scala/org/broadinstitute/dsde/firecloud/service/MethodConfigurationService.scala does or doesn't need to be modified for the purpose of DSDEEPB-1343  .  I have a thought in my mind on yes/no and why, but I wonder what you have to say about it.  I hope after a brief comment from you that you can rebase and merger


Ok ja conseguir carregar a classe impressao...

Somente o evento q continua...


I'll pull the title/metadata change into my local copy and set a PR with it.


!mustfix Rename to `manager` or something...


See comment in th_movie.h


adcas


Please replace the star import with individual ones


Hmm.. Sorry for dumb question, but must not API server open from outside?


"throwed" should be "thrown"


I separate blocks of methods/data members this way.


**semi**: Missing semicolon.


**react/jsx-sort-props**: Props should be sorted alphabetically


Doesn't it build init.g2.power.rc as init.recovery.g2.rc? Check the mamefile.


I'm replacing it with set_first_analysis_to_true_when_no_last_snapshot


LGTM!


This is right as is according to: https://github.com/CorsixTH/CorsixTH/wiki/Coding-Conventions#variable-naming


There's a little rails helper you could use to generate links http://apidock.com/rails/ActionView/Helpers/UrlHelper/link_to


heh


I know its type, group id after Job::init(). Timestamp in constructor is a point when I connect to database.


Change this `nickname: "Acme"` to `nickname: params[:nickname]`


!mustfix Change the command to accept `-no_skeleton` so that people don't keep forgetting to specify the `-skeleton` flag.

This makes the situation better since you need to specify a flag explicitly, i.e. valid ways are `-skeleton` or `-no_skeleton`...


Only thing that changes from my version are the tests, I will merge the two.


@akbertram while implementing this locks concept it looks more natural to keep locks outside from FormClass and stick to resource instead:
```
class ResourceLock {
     ResourceId id;
     ResourceId ownerId;
     String expression;
}

Promise<List<ResourceLock>> getResourceLocator().queryLocks(ResourceId resourceId);
```
- id of the lock allows to get/save locks independently from FormClass and thus keep it loosely coupled (stick to Resource rather than FormClass)
- If we inject locks inside FormClass then we need to translate to/from json. We already have persisted jsons in activity table, with FormClass definition change we have to start track json changes on GetFormClassHandler. If keep lock outside of FormClass then we avoid all these problems.


Vou alterar
```cs
PropertyChanged?.Invoke(this, new PropertyChangedEventArgs(propertyName));
´´´
para:
```cs
 if (PropertyChanged != null) PropertyChanged.Invoke(this, new PropertyChangedEventArgs(propertyName));
```
Talvez o VS2012 não tenha suporte à checagem de valor nulo utilizando o operador ?.


ok, I'm adding a test


nits : You has になった場合おかしいね


No, the rest of this file and the coding convention use camalCase: https://github.com/CorsixTH/CorsixTH/wiki/Coding-Conventions#variable-naming (also see https://github.com/TheCycoONE/CorsixTH/blob/3b80286cbaacb078a8cfae4be6ce33f92f5e662a/CorsixTH/Src/th_lua_ui.cpp#L55)


Ou melhor, troque aí no seu projeto e me diga se deu certo.
Por que daí subo a alteração para o git.


是


Regarding the example above:
How should look an operationId for MyApp::Controller::Person->new()->list ?


区别主要就是XP了，剩下的都是开发方面的区别


!mustfix Would be better to make this return `action.Action`.


This code looked incomplete to me but Glenn mentioned that this was a netdev provider. (ah-ha!)
For future PRs of this type please also link to the netdev type. Thanks!


Unfortunately it isn't so straightforward. In particular, if you have the
`declarationsCapability` and you can get a particular `ClassMirror`, then
you can also get its declarations. This might well apply to other class
mirrors than the ones you have in mind, for instance class mirrors
describing return types of methods that you need to know in order to
support the correct semantics for mustache expressions.

One thing we are about to implement in order to cut down on the needed set
of class mirrors is exactly about this scenario: It will be possible to get
just the `Type` instance for such a return type, and not have support for
getting the class mirror itself. With that, you could check for equality
("does this method have return type `int` or not?") and not much more, but
that seems to be OK in this case.

But it is not trivial to check that this is the only case where we include
"too many mirrors" of one or the other kind.

I think it will be necessary for someone (Jacob and those of us who are
working on reflectable come to mind, but maybe others as well) to look at
the size of the generated code and maybe browse through the generated `new
r.ClassMirrorImpl(..)` expressions in order to check that the set of
classes does not include a large number of unwanted classes. Fixes may then
be applied by adjusting the capabilities, or maybe choosing different names
for certain members such that they are easily matched by a regexp, etc.

If no fixes will work then we can also enter into discussions about
enhancing the set of available capabilities in reflectable, but we
obviously have to use a rather general point of view rather than separately
supporting every special case that comes up.

Such is the nature of quantification: You cannot directly specify "give me
exactly the classes I need", because language-like mechanisms like the
reflectable capabilities has a certain amount of expressive power, and the
needs in any given usage setting may or may not be expressible with that,
so there will be cases where you have to live with quantification that
gives you too many class mirrors (or parameter mirrors, or whatever). The
`@foo` metadata annotation mechanism lets you specify a set of members very
precisely, though, which is the reason why you may want to use it, even
though it is not so convenient.

On Mon, Oct 5, 2015 at 1:37 PM, Günter Zöchbauer <notifications@github.com>
wrote:

> (easy, but makes the generated code larger)
> I assume this is only when not every member needed to be exposed, right?
>
> When I create a class only for the purpose to create Polymer bindings to
> its fields and methods, and only members needed for this purpose are added
> to the class in the first place, then I don't see why there would be a
> difference in the resulting code size.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dart-lang/polymer-dart/commit/4ebca8eb4f770cb233c7e0ccf8a72b99a43fdd85#commitcomment-13589868>
> .
>

--
Erik Ernst  -  Google Danmark ApS
Skt Petri Passage 5, 2 sal, 1165 København K, Denmark
CVR no. 28866984


I think what @shindo is saying is that you should describe the way this object is meant be used in your program. E.g. if you're using it as a singleton you should mention that to prevent somebody else trying to create another instance of this class.


!mustfix Wondering whether this works even though the directory exists already... Probably yes.


@bwoebi This is an "simplified behavior" of my library

<?php

class A
{
	public function hello($lastname)
	{
		echo $this->name.' '.$lastname;
	}
}

class B
{
	private $closure;

	private $name;

	public function __construct($name, $closure)
	{
		$this->closure = $closure;
		$this->name = $name;
	}

	public function __call($name, $arg)
	{
		$this->closure->call($this, ...$arg);
	}

}

$c = new ReflectionClass('A');
$m = $c->getMethod('hello');
$b = new B('first', $m->getClosure(new A));
$b->hello('last');
=> display "first last"


можно было и через && соединить


Jak się czujesz tak w ogóle? Trochę zdrowszy?

W dniu 5 października 2015 15:12 użytkownik Marta Kuzak <
marta.kuzak@gmail.com> napisał:

> Eeee, czyli co? Chyba nie powinno być u mnie błędu, bo mi chodzi o
> wartości mieszczące się w 8-iu bitach. Czy coś jeszcze?
>
> 2015-10-05 12:40 GMT+02:00 Marcin Kubik <notifications@github.com>:
>
>> http://stackoverflow.com/questions/11600378/portable-code-bits-per-char
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/martakuzak/MP4_2/commit/ba2943e59ca175d3cf1d1bce232f69ea265f97ec#commitcomment-13588946>
>> .
>>
>
>


What is this require for?


!mustfix Is this being used anywhere?


@lindamartin Indeed. I think my intention was to avoid covering traits, but they were already covered by the existing rule, so I should change the new implementation.


LGTM!


Allow å ä ö


extra space? what for


Lol


!mustfix The modules are already sorted.


@pynicolas oh I see.


Minor, but perhaps add a reboot utility function somewhere that does this?  Just so the way we reboot is captured in one place.


2015-10-04 21:32 GMT-03:00 Tiago Silva do Nascimento <
notifications@github.com>:

> fffff
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ProjetoInterdisciplinar/ProjetoInterdisciplinarIII/commit/7cc5046f80dc1fb068ac0483a17151fd9b1d1b88#commitcomment-13583279>
> .
>


This isn't an issue since it's just a demo app, but ideally you'll leave `.env.example` alone, since the person installing the app may use a different host/database.


You have two return statements here, so you'll actually split them with a `|`.


Remember the code standards, braces on the same line.


this hunk goes in the next commit


**semi**: Missing semicolon.


**react/jsx-sort-props**: Props should be sorted alphabetically


@lindamartin I will change these test classes so that they don't extend anything.


can you put odsync_write and do_fast_sync in the next commit?


Don't forget that you can use the `Guard` class here within the constructor, that way you don't need to call this.


4 = hardcoded constant, might be useful to give it a symbolic name


This brace should be on the same line as the `function`.


Missing a docblock.


If you import os, then you can use the module path without additional os.path imports. Or for isfile is any exception?
I would agree if you use only the path, then import os.path or from os import path as os_path would be acceptable, but if you are using import os, os.path then I not see any reason to use the additional os.path import.


I don't know the history on this, although I seem to recall that it was intentional, but we are going to have to investigate


Since this is a brand new feature, you'd have been better off branching it. This has the added benefits that it won't affect your app if it was in production and you'd catch the code standards (CS) issues.


Generally failed assertion means the program is completely broken. In this place I wanted to catch some obvious bugs during development period.


It would be great if you could describe the resulting json in the comments so anyone could see it right away without parsing the code. Like its structure and preconditions of certain subkeys.


`present.map!(&:last)` is the same as `present = present.map(&:last)` except that it’s done in-place so use less memory and might be slightly faster. The second code snippet creates a new array then assign it while the first modifies the existing array without creating a new one.


Space after the `//`.


Correct :yum: But why?


Maybe for brevity, `$user` would make more sense?


ObjexxFCL_Update (DeadParrot) - Win64-Windows-7-VisualStudio-12: OK (1909 of 1909 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


@jrchamp Looking back through some old commits I noticed we originally tried fixing XSS vulnerabilities by setting each variable with `htmlspecialchars()`, but the very next commit changed the function to `urlencode()` instead. I don't think we would have changed it if `htmlspecialchars()` was working. Can you explain the choice here?


To be removed


You right! This loop should start at 0 because 'total' need all pixels


Because:

- `tar.gz` is a default format for many things. Creating a `.orig.tar.gz` is a common task, and this tool *nearly* scratches that itch.
- `git-archive-all` doesn't mention any special about `--format=`, so it should work at least with those formats understood by `git-archive`

See #25


we have an 'Undefined' error when we use 'this.size'.


**require-jsdoc/require-jsdoc**: Missing @jsdoc for _customValidate


Remember that Laravel prefers underscores for table names, and tables should be plural.

In fact, this table might not make much sense. You could have gone with `friends` and then had:

- `user_id`
- `friend_id`
- `confirmed`

Also, maybe storing a timestamp of when the confirmation was done?


I'll remove it.


We should have general `Copy(Object o)` and `CreateSomethingWhichReturnsSomethingIWantToHave` anyway.
But splitting up the ops into a `ComputerOp` and a `FunctionOp` makes sense.


Ciao-robot is a simple hub where you can store all what you know about robotics. In this way other people will find interesting things for their projects and they will share what they do in this field.


Automatically :)
I would say "Trace packet going through Butterfly"


bool merge_jobs() is not descriptive. I think the return code is something about result of whole operation. For example, false might mean storage versions conflict.


Nice, thanks!


To be removed ?


 #[zstack Test Bot Report] P A S S
 Integration Test Result:
Test Summary:

 TestSuite|TestCase | Pass | Fail | Timeout | Skip
 ---------|---------| ---- | ---- | ------- | ----
 basic_test|__suite_setup__|0|1|0|0
basic_test|_basic/test_create_vm_|0|0|0|1
basic_test|_basic/test_stop_vm_|0|0|0|1
basic_test|_basic/test_start_vm_|0|0|0|1
basic_test|_basic/test_reboot_vm_|0|0|0|1
basic_test|_basic/test_add_volume_|0|0|0|1
basic_test|_basic/test_add_vol_to_stopvm_|0|0|0|1
basic_test|_basic/test_add_volume_reboot_vm_|0|0|0|1
basic_test|_basic/test_add_multi_volumes_|0|0|0|1
basic_test|_basic/test_add_volume_negative_|0|0|0|1
basic_test|_basic/test_crt_temp_from_volume_|0|0|0|1
basic_test|_basic/test_crt_vm_with_volume_|0|0|0|1
basic_test|_basic/test_vm_securitygroup_|0|0|0|1
basic_test|_basic/test_create_iso_vm_|0|0|0|1
basic_test|_account/test_crt_vm_by_normal_account_|0|0|0|1
basic_test|_account/test_crt_vm_by_normal_account2_|0|0|0|1
basic_test|_account/test_vm_ops_by_normal_account_|0|0|0|1
basic_test|_account/test_multi_accounts_vm_ops_|0|0|0|1
basic_test|_suite_teardown_|0|0|0|1
virtual_router_test|__suite_setup__|0|1|0|0
virtual_router_test|_host/test_reconnect_host_|0|0|0|1
virtual_router_test|_account/test_vm_ops_by_normal_account_|0|0|0|1
virtual_router_test|_account/test_multi_accounts_vm_ops_|0|0|0|1
virtual_router_test|_novlan/test_create_vm_|0|0|0|1
virtual_router_test|_novlan/test_reboot_vm_|0|0|0|1
virtual_router_test|_novlan/test_ping_vms_|0|0|0|1
virtual_router_test|_host/test_reconnect_host_|0|0|0|1
virtual_router_test|_vlan/test_create_vm_|0|0|0|1
virtual_router_test|_vlan/test_reboot_vm_|0|0|0|1
virtual_router_test|_vlan/test_snat_create_|0|0|0|1
virtual_router_test|_vlan/test_dnat_reboot_|0|0|0|1
virtual_router_test|_vlan/test_ping_vms_|0|0|0|1
virtual_router_test|_vlan/test_create_3vms_|0|0|0|1
virtual_router_test|_vlan/test_start_vm_|0|0|0|1
virtual_router_test|_vlan/test_3l3s_vm_|0|0|0|1
virtual_router_test|_vlan/test_remove_dhcp_|0|0|0|1
virtual_router_test|_vlan/test_2l3_vm_|0|0|0|1
virtual_router_test|_vlan/test_crt_vms_with_limited_ips_|0|0|0|1
virtual_router_test|_vlan/test_crt_vms_with_limited_ips2.0_|0|0|0|1
virtual_router_test|_vlan/test_crt_vms_with_limited_ips2.1_|0|0|0|1
virtual_router_test|_vlan/test_start_vms_simultaneously_|0|0|0|1
virtual_router_test|_vlan/test_crt_static_ip_vm_|0|0|0|1
virtual_router_test|_vlan/test_crt_vm_with_hostname_|0|0|0|1
virtual_router_test|_vlan/test_attach_nic_|0|0|0|1
virtual_router_test|_vlan/test_attach_nic_stopped_vm_|0|0|0|1
virtual_router_test|_vlan/test_attach_nic_reboot_vm_|0|0|0|1
virtual_router_test|_vlan/test_chg_static_ip_vm_|0|0|0|1
virtual_router_test|_vlan/test_attach_nic_with_static_ip_|0|0|0|1
virtual_router_test|_volume/test_add_volume_|0|0|0|1
virtual_router_test|_volume/test_add_m_vols_|0|0|0|1
virtual_router_test|_volume/test_add_volume_vr_|0|0|0|1
virtual_router_test|_volume/test_crt_vm_volume_|0|0|0|1
virtual_router_test|_volume/test_crt_vm_2_volumes_|0|0|0|1
virtual_router_test|_volume/test_crt_vm_24_volumes_|0|0|0|1
virtual_router_test|_volume/test_volume_ops_on_stopped_vm_|0|0|0|1
virtual_router_test|_volume/test_crt_vm_from_dvt_|0|0|0|1
virtual_router_test|_template_image/test_crt_temp_image_|0|0|0|1
virtual_router_test|_template_image/test_del_temp_image_|0|0|0|1
virtual_router_test|_sg/test_ingress_|0|0|0|1
virtual_router_test|_sg/test_egress_|0|0|0|1
virtual_router_test|_sg/test_icmp_|0|0|0|1
virtual_router_test|_sg/test_udp_|0|0|0|1
virtual_router_test|_sg/test_stopped_vm_|0|0|0|1
virtual_router_test|_sg/test_vm_ops_|0|0|0|1
virtual_router_test|_sg/test_2vm_egress_|0|0|0|1
virtual_router_test|_sg/test_2vm_ingress_|0|0|0|1
virtual_router_test|_sg/test_4vm_complx_|0|0|0|1
virtual_router_test|_sg/test_detach_l3_|0|0|0|1
virtual_router_test|_pf/test_2l3s_pfs_|0|0|0|1
virtual_router_test|_pf/test_attach_pf_|0|0|0|1
virtual_router_test|_pf/test_delete_pf_after_destroy_vm_|0|0|0|1
virtual_router_test|_pf/test_del_vip_before_add_pf_to_vm_|0|0|0|1
virtual_router_test|_pf/test_multi_pfs_sg_tcp_|0|0|0|1
virtual_router_test|_pf/test_multi_rules_|0|0|0|1
virtual_router_test|_pf/test_pf_after_destroy_vm_|0|0|0|1
virtual_router_test|_pf/test_pf_sg_tcp_|0|0|0|1
virtual_router_test|_pf/test_pfs_tcp_udp_same_vip_ports_|0|0|0|1
virtual_router_test|_pf/test_pfs_tcp_with_same_vip_|0|0|0|1
virtual_router_test|_pf/test_tcp_|0|0|0|1
virtual_router_test|_pf/test_udp_|0|0|0|1
virtual_router_test|_pf/test_vm_ops_|0|0|0|1
virtual_router_test|_eip/test_2_eips_|0|0|0|1
virtual_router_test|_eip/test_2l3s_eip_pf_|0|0|0|1
virtual_router_test|_eip/test_2l3s_eips_|0|0|0|1
virtual_router_test|_eip/test_attach_eip_|0|0|0|1
virtual_router_test|_eip/test_create_eip_vm_|0|0|0|1
virtual_router_test|_eip/test_del_vip_before_add_eip_to_vm_|0|0|0|1
virtual_router_test|_eip/test_eip_sg_tcp_|0|0|0|1
virtual_router_test|_eip/test_pf_eip_attach_|0|0|0|1
virtual_router_test|_eip/test_vm_ops_|0|0|0|1
virtual_router_test|_eip/test_vm_ops2_|0|0|0|1
virtual_router_test|_lb/test_create_lb_alone_vr_|0|0|0|1
virtual_router_test|_lb/test_create_lb_source_|0|0|0|1
virtual_router_test|_lb/test_create_lb_|0|0|0|1
virtual_router_test|_lb/test_lb_health_check_|0|0|0|1
virtual_router_test|_snapshot/test_create_snapshot_|0|0|0|1
virtual_router_test|_snapshot/test_create_sp_after_delete_|0|0|0|1
virtual_router_test|_snapshot/test_create_image_by_snapshot_|0|0|0|1
virtual_router_test|_snapshot/test_crt_sp_for_both_root_..._|0|0|0|1
virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm2_|0|0|0|1
virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm3_|0|0|0|1
virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm_|0|0|0|1
virtual_router_test|_snapshot/test_delete_snapshot_on_ru..._|0|0|0|1
virtual_router_test|_snapshot/test_crt_sp_in_live_vm_|0|0|0|1
virtual_router_test|_snapshot/test_crt_sp_from_vol_crt_f..._|0|0|0|1
virtual_router_test|_snapshot/test_change_max_sp_depth_l..._|0|0|0|1
virtual_router_test|_regression/delete_sg_with_2_attache..._|0|0|0|1
virtual_router_test|_regression/rm_nic_sg_on_stopped_vm_|0|0|0|1
virtual_router_test|_regression/test_data_volume_with_vm..._|0|0|0|1
virtual_router_test|_vr/test_ping_vms_reboot_vr_|0|0|0|1
virtual_router_test|_vr/test_reconnect_vr_|0|0|0|1
virtual_router_test|_vr/test_start_vrs_simultaneously_|0|0|0|1
virtual_router_test|_host/test_reconnect_host_|0|0|0|1
virtual_router_test|_suite_teardown_|0|0|0|1
local_ps_virtual_router_test|__suite_setup__|0|1|0|0
local_ps_virtual_router_test|_host/test_reconnect_host_|0|0|0|1
local_ps_virtual_router_test|_account/test_vm_ops_by_normal_account_|0|0|0|1
local_ps_virtual_router_test|_account/test_multi_accounts_vm_ops_|0|0|0|1
local_ps_virtual_router_test|_novlan/test_create_vm_|0|0|0|1
local_ps_virtual_router_test|_novlan/test_reboot_vm_|0|0|0|1
local_ps_virtual_router_test|_novlan/test_ping_vms_|0|0|0|1
local_ps_virtual_router_test|_host/test_reconnect_host_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_create_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_reboot_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_snat_create_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_dnat_reboot_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_ping_vms_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_create_3vms_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_start_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_3l3s_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_remove_dhcp_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_2l3_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_crt_vms_with_limited_ips_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_crt_vms_with_limited_ips2.0_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_crt_vms_with_limited_ips2.1_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_start_vms_simultaneously_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_crt_static_ip_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_crt_vm_with_hostname_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_attach_nic_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_attach_nic_stopped_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_attach_nic_reboot_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_chg_static_ip_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_attach_nic_with_static_ip_|0|0|0|1
local_ps_virtual_router_test|_volume/test_add_volume_|0|0|0|1
local_ps_virtual_router_test|_volume/test_add_m_vols_|0|0|0|1
local_ps_virtual_router_test|_volume/test_add_volume_vr_|0|0|0|1
local_ps_virtual_router_test|_volume/test_crt_vm_volume_|0|0|0|1
local_ps_virtual_router_test|_volume/test_crt_vm_2_volumes_|0|0|0|1
local_ps_virtual_router_test|_volume/test_crt_vm_24_volumes_|0|0|0|1
local_ps_virtual_router_test|_volume/test_volume_ops_on_stopped_vm_|0|0|0|1
local_ps_virtual_router_test|_volume/test_crt_vm_from_dvt_|0|0|0|1
local_ps_virtual_router_test|_template_image/test_crt_temp_image_|0|0|0|1
local_ps_virtual_router_test|_template_image/test_del_temp_image_|0|0|0|1
local_ps_virtual_router_test|_sg/test_ingress_|0|0|0|1
local_ps_virtual_router_test|_sg/test_egress_|0|0|0|1
local_ps_virtual_router_test|_sg/test_icmp_|0|0|0|1
local_ps_virtual_router_test|_sg/test_udp_|0|0|0|1
local_ps_virtual_router_test|_sg/test_stopped_vm_|0|0|0|1
local_ps_virtual_router_test|_sg/test_vm_ops_|0|0|0|1
local_ps_virtual_router_test|_sg/test_2vm_egress_|0|0|0|1
local_ps_virtual_router_test|_sg/test_2vm_ingress_|0|0|0|1
local_ps_virtual_router_test|_sg/test_4vm_complx_|0|0|0|1
local_ps_virtual_router_test|_sg/test_detach_l3_|0|0|0|1
local_ps_virtual_router_test|_pf/test_2l3s_pfs_|0|0|0|1
local_ps_virtual_router_test|_pf/test_attach_pf_|0|0|0|1
local_ps_virtual_router_test|_pf/test_delete_pf_after_destroy_vm_|0|0|0|1
local_ps_virtual_router_test|_pf/test_del_vip_before_add_pf_to_vm_|0|0|0|1
local_ps_virtual_router_test|_pf/test_multi_pfs_sg_tcp_|0|0|0|1
local_ps_virtual_router_test|_pf/test_multi_rules_|0|0|0|1
local_ps_virtual_router_test|_pf/test_pf_after_destroy_vm_|0|0|0|1
local_ps_virtual_router_test|_pf/test_pf_sg_tcp_|0|0|0|1
local_ps_virtual_router_test|_pf/test_pfs_tcp_udp_same_vip_ports_|0|0|0|1
local_ps_virtual_router_test|_pf/test_pfs_tcp_with_same_vip_|0|0|0|1
local_ps_virtual_router_test|_pf/test_tcp_|0|0|0|1
local_ps_virtual_router_test|_pf/test_udp_|0|0|0|1
local_ps_virtual_router_test|_pf/test_vm_ops_|0|0|0|1
local_ps_virtual_router_test|_eip/test_2_eips_|0|0|0|1
local_ps_virtual_router_test|_eip/test_2l3s_eip_pf_|0|0|0|1
local_ps_virtual_router_test|_eip/test_2l3s_eips_|0|0|0|1
local_ps_virtual_router_test|_eip/test_attach_eip_|0|0|0|1
local_ps_virtual_router_test|_eip/test_create_eip_vm_|0|0|0|1
local_ps_virtual_router_test|_eip/test_del_vip_before_add_eip_to_vm_|0|0|0|1
local_ps_virtual_router_test|_eip/test_eip_sg_tcp_|0|0|0|1
local_ps_virtual_router_test|_eip/test_pf_eip_attach_|0|0|0|1
local_ps_virtual_router_test|_eip/test_vm_ops_|0|0|0|1
local_ps_virtual_router_test|_eip/test_vm_ops2_|0|0|0|1
local_ps_virtual_router_test|_lb/test_create_lb_alone_vr_|0|0|0|1
local_ps_virtual_router_test|_lb/test_create_lb_source_|0|0|0|1
local_ps_virtual_router_test|_lb/test_create_lb_|0|0|0|1
local_ps_virtual_router_test|_lb/test_lb_health_check_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_create_snapshot_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_create_sp_after_delete_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_create_image_by_snapshot_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_crt_sp_for_both_root_..._|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm2_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm3_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_delete_snapshot_on_ru..._|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_crt_sp_in_live_vm_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_crt_sp_from_vol_crt_f..._|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_change_max_sp_depth_l..._|0|0|0|1
local_ps_virtual_router_test|_regression/delete_sg_with_2_attache..._|0|0|0|1
local_ps_virtual_router_test|_regression/rm_nic_sg_on_stopped_vm_|0|0|0|1
local_ps_virtual_router_test|_regression/test_data_volume_with_vm..._|0|0|0|1
local_ps_virtual_router_test|_vr/test_ping_vms_reboot_vr_|0|0|0|1
local_ps_virtual_router_test|_vr/test_reconnect_vr_|0|0|0|1
local_ps_virtual_router_test|_vr/test_start_vrs_simultaneously_|0|0|0|1
local_ps_virtual_router_test|_host/test_reconnect_host_|0|0|0|1
local_ps_virtual_router_test|_suite_teardown_|0|0|0|1
local_nfs_virtual_router_test|__suite_setup__|0|1|0|0
local_nfs_virtual_router_test|_host/test_reconnect_host_|0|0|0|1
local_nfs_virtual_router_test|_account/test_vm_ops_by_normal_account_|0|0|0|1
local_nfs_virtual_router_test|_account/test_multi_accounts_vm_ops_|0|0|0|1
local_nfs_virtual_router_test|_novlan/test_create_vm_|0|0|0|1
local_nfs_virtual_router_test|_novlan/test_reboot_vm_|0|0|0|1
local_nfs_virtual_router_test|_novlan/test_ping_vms_|0|0|0|1
local_nfs_virtual_router_test|_host/test_reconnect_host_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_create_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_reboot_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_snat_create_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_dnat_reboot_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_ping_vms_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_create_3vms_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_start_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_3l3s_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_remove_dhcp_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_2l3_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_crt_vms_with_limited_ips_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_crt_vms_with_limited_ips2.0_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_crt_vms_with_limited_ips2.1_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_start_vms_simultaneously_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_crt_static_ip_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_crt_vm_with_hostname_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_attach_nic_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_attach_nic_stopped_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_attach_nic_reboot_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_chg_static_ip_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_attach_nic_with_static_ip_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_add_volume_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_add_m_vols_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_add_volume_vr_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_crt_vm_volume_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_crt_vm_2_volumes_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_crt_vm_24_volumes_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_volume_ops_on_stopped_vm_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_crt_vm_from_dvt_|0|0|0|1
local_nfs_virtual_router_test|_template_image/test_crt_temp_image_|0|0|0|1
local_nfs_virtual_router_test|_template_image/test_del_temp_image_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_ingress_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_egress_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_icmp_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_udp_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_stopped_vm_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_vm_ops_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_2vm_egress_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_2vm_ingress_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_4vm_complx_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_detach_l3_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_2l3s_pfs_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_attach_pf_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_delete_pf_after_destroy_vm_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_del_vip_before_add_pf_to_vm_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_multi_pfs_sg_tcp_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_multi_rules_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_pf_after_destroy_vm_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_pf_sg_tcp_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_pfs_tcp_udp_same_vip_ports_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_pfs_tcp_with_same_vip_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_tcp_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_udp_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_vm_ops_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_2_eips_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_2l3s_eip_pf_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_2l3s_eips_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_attach_eip_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_create_eip_vm_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_del_vip_before_add_eip_to_vm_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_eip_sg_tcp_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_pf_eip_attach_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_vm_ops_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_vm_ops2_|0|0|0|1
local_nfs_virtual_router_test|_lb/test_create_lb_alone_vr_|0|0|0|1
local_nfs_virtual_router_test|_lb/test_create_lb_source_|0|0|0|1
local_nfs_virtual_router_test|_lb/test_create_lb_|0|0|0|1
local_nfs_virtual_router_test|_lb/test_lb_health_check_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_create_snapshot_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_create_sp_after_delete_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_create_image_by_snapshot_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_crt_sp_for_both_root_..._|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm2_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm3_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_delete_snapshot_on_ru..._|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_crt_sp_in_live_vm_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_crt_sp_from_vol_crt_f..._|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_change_max_sp_depth_l..._|0|0|0|1
local_nfs_virtual_router_test|_regression/delete_sg_with_2_attache..._|0|0|0|1
local_nfs_virtual_router_test|_regression/rm_nic_sg_on_stopped_vm_|0|0|0|1
local_nfs_virtual_router_test|_regression/test_data_volume_with_vm..._|0|0|0|1
local_nfs_virtual_router_test|_vr/test_ping_vms_reboot_vr_|0|0|0|1
local_nfs_virtual_router_test|_vr/test_reconnect_vr_|0|0|0|1
local_nfs_virtual_router_test|_vr/test_start_vrs_simultaneously_|0|0|0|1
local_nfs_virtual_router_test|_host/test_reconnect_host_|0|0|0|1
local_nfs_virtual_router_test|_suite_teardown_|0|0|0|1
Total|304|0|4|0|300


For "intentional" usages I would expect `int QUERY_FORCE_DESCRIBE_PORTAL = QUERY_DISALLOW_BATCHING` kind of declaration.
However, "disallow_batching" is a rather unexpected strategy of "batch API" support.


PropertyChanged?.Invoke(this, new PropertyChangedEventArgs(propertyName));

so funciona no vs 2015+ isto é coisa nova do c# 6.0 no link abaxio algumas outras novidades.

http://blog.caelum.com.br/novidades-do-c-6-0/


`WITH_BODHI` becomes unused with this hunk.


Fine, by me.


Tabs ?


Yep, this fixes my problem, thanks :)


@sushobhit @mmatts MeshCoefficient and MeshCount appear to be somewhat arbitrary here.  Would it better to make them optional user inputs to allow more control over the solution?


This looks like only uniques can drop hide now?  Perhaps you wanted to say '|| uniq' on line 342.


ref #279


Thrown exception will break one request (or background task), occured assertion will break the application.


!mustfix "one-time action"


This looks very weird, I don't understand what mail API have to do with XWiki attachments...


I still think the above line can produce a confusing exception.


It's terrible~~


Awesome work @tobscure!


sa pet


While I agree the assumption is logical, We can't rely on that


the filestore_fast_commit name doesn't make sense to me.. maybe journal_dynamic_throttle or something?


pyhon3_3 is announced for deprecation, please don't add it to new ebuilds


Done


e.g. see `const MAX_LIKES = 4` below


I'm assuming it initalizes this in recovery? I'm not sure. That's a question for Ethan.


Is it worthwhile doing it in-place if I'd have to do it twice (once each for "head" and "head block")? Or is there a function that can check for "all of these are included"?


Better add pkgcore too then.


Then:
`struct have_newer_t { bool value; };`
or
`using have_newer_t = bool;`
for
`have_newer_t merge_jobs();`

In fact depends on context.


Minus is hard.


frm tehre


Hash # is not good here..


Ca va parce que c'est un test mais l'accolade devrait être à la ligne


When editing button title should be `Update`


I don't think that this patch is sufficient, see https://github.com/djfarrelly/MailDev/issues/86#issuecomment-137772354


The method getBuild() has a Cyclomatic Complexity of 11. The configured cyclomatic complexity threshold is 10.


fix #12


**Testing - Awesome**
This is was a good improvisation.


Why assert default is not True for BoolParameter?


**Code Quality - Nitpick**
Remember to leave spaces after and before concatenating periods (`.`).


shabang -> shebang


The testcase for #267 is in the test set (https://gist.github.com/ringerc/25d36e9f0044077c664b), and it passes with `QUERY_FORCE_DESCRIBE_PORTAL` having its own value (see #380)


console.log


@rushtong This can't be a button. It has to be a link with an `href` attribute that points to the download endpoint. I don't think there is a good way to mock this data either.


**Comments**
- Well commented code, and good use of `docBlock`.

**Functionality**
- Meets expectation

**Code Qaulaity**
- Good OOP design with dependency injection
- Good use of namespaces and autoloading


I think sublocations are supposed to be part of the data now (every row), not in the header.


What about
```
prefix += "?" + urllib.urlencode(params)
```
?


Yes, did a bunch of work last night on other related stuff.


At least it fixes the relay problem. For the unexpected response I'm
writing a response at the moment.

On 5 October 2015 at 15:42, Christoph Rissner <notifications@github.com>
wrote:

> I don't think that this patch is sufficient, see djfarrelly#86 (comment)
> <https://github.com/djfarrelly/MailDev/issues/86#issuecomment-137772354>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/diresi/MailDev/commit/a6ccce53feedcfdc380eb391c526bd7b1ea5031a#commitcomment-13592156>
> .
>

--
“Empty your mind, be formless, shapeless - like water. Now you put water
into a cup, it becomes the cup, you put water into a bottle, it becomes the
bottle, you put it in a teapot, it becomes the teapot. Now water can flow
or it can crash. Be water, my friend.” - Bruce Lee


It feels like the fact that `user` can be null is poorly documented.

What is the intended action in this case?


It's ok for me! thanks :)


I must emphasize that, in the scheme used here, anyone can log in with the email of someone else. It may be on purpose, but might cause issues, depending on how the rest of the system works.

I recommend that you use HSTS, which I believe is not the case yet.


You git checkout normally within localmh_docker. The sTeX installation is
under /path/to/localmh/ext/sTex .

On Mon, Oct 5, 2015, 12:13 angerhang <notifications@github.com> wrote:

> I apologize for introducing this error but this error makes me ponder
> about the workflow we have for future sTeX work.
>
> I think merging from a feature branch to master is an more elegant
> approach than merging from a fork. Otherwise every time I do something on
> the master inside the docker, I will need to manually make the changes
> inside my own fork without being able to do a regression test since it is
> not inside the docker.
>
> Perhaps lmh allows me to run my sTeX fork in the docker? @tkw1536
> <https://github.com/tkw1536>
> But again that might hinder the work of feature branch.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/KWARC/sTeX/commit/d994bda868cae147e01784dea411b1d6941baa6a#commitcomment-13588553>
> .
>


Not sure if I understand the question, but wild guess: "listPersons" ..?


feature/post がマージされたからちゃんと diff 出るはずなので PR 出してもいいかもね
おざきくんが出したいときに出せばいいとおもう

リファクタリングはお好きなようにやっていいとおもう


On my system, for homebrew-science/vtk, this commit breaks the build. The problem seems to be that VTK builds some binaries and then tries to run them during the build; when it runs the binaries it is unable to load the shared libraries because of the CMAKE_BUILD_WITH_INSTALL_RPATH option. What was the reasoning behind adding the CMAKE_BUILD_WITH_INSTALL_RPATH flag? I  understand the need for CMAKE_INSTALL_RPATH_USE_LINK_PATH - the  binaries need to be  linked properly for running. But at build time, the binaries are not present in the install directory, and therefore any attempt to run them from the build directory will fail.

I removed CMAKE_BUILD_WITH_INSTALL_RPATH from the args list and build vtk  again, and the build was successful. If there is no compelling reason to keep CMAKE_BUILD_WITH_INSTALL_RPATH set, would  you accept a pull request disabling it again?


so what is supposed to be correct?? locationNumber or stationId

and locationName vs stationName?


I anticipate it will be stationName on the next version of the JSON. I have pending code for both java and repgen stuff coming later this morning.


@faboschert execute `npm install` to add the after-brunch plugin.


OK, lets see if @ringerc  responds to this? He may remember better than I


ObjexxFCL_Update (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1373 of 1373 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-70.36%25-green.svg)


I'm responsible for creating a school table holding the details of a School (Back-end). . . .

TABLE/ENTITY name: SCHOOL Attributes: (PK)SCHL_ID, (CONSTRAINT)SCHL_NAME, SCHL_ABBR, SCHL_STREET_ADDR, SCHL_CITY, SCHL_COUNTRY, SCHL_POSTAL_CODE

SCHL_NAME, SCHL_CITY - (Unique Constraint)


我以为你会。


Actually, that change was not supposed to be part of this PR. Get rid of it if you need to test your next PR. And I'll change it when it's actually ready.


There's a module called 'mozinstall' that does just this. You can import it after the "SEARCH_PATHS" get added to sys.path below.


I understand there is a more recent version. Would you mind committing it so I can fork and help with development? Thanks!


Visiblement, c'est appliqué depuis le 1er mai 2007.
http://www.legifrance.gouv.fr/eli/decret/2007/4/13/SANS0721467D/jo
Merci pour la précision !


The 'requests' module is checked into the gecko tree. You can import it after the SEARCH_PATHS gets added to sys.path below. It's much better than urllib2.


!mustfix Not super pretty, could be done better.


@davidpearce You can use my repo for your request


pkgcore doesn't have full EAPI=5 support yet. I'd rather remove such notes completely and link to a wiki entry.


Each time a new view is loaded the `ng-view` directive will create a __copy__ of itself, download the template and append the contents.

![image](https://cloud.githubusercontent.com/assets/6700824/10282135/dd0a1528-6bc4-11e5-8817-e5b3b31f56b3.png)


I can not build Slicer on Mac and Linux :
```
vtkSlicerApplicationLogic.cxx:1760:60: error: expected ‘)’ before ‘Slicer_EXTENSIONS_DIRBASENAME’
    bool macExtension = (canonicalPath.find(".app/Contents/" Slicer_EXTENSIONS_DIRBASENAME "-") != std::string::npos);
                                                            ^

```


> I think the name above is unknown because I am using S3Represent but name is not foreign key?

Right, that would certainly cause the issue if you're feeding in names instead of FKs ;)


when a request comeS, if NO valid handler.


Ok Funcionou...


nit, please order


s/For instance/e. g./


It's being invoked by hand with cocaine-tool to understand the current state and metrics.

$ cocaine-tool call mastermind2.26-collector enqueue "'summary', ''" --timeout 10


We have to use `position: absolute;` when `.ng-enter` and `.ng-leave`, because we need to overlay the `.ng-enter` in front of the `.ng-leave`.

When elements are __positioned__ (better to be `absolute`), they can overlap other elements.

http://www.w3schools.com/css/css_positioning.asp


You're problem is the `%s`, because `scmp` returns an `int`. `printf` is looking for a null terminated string, so it's trying looking for the null at the end of the `int` which may or may not be there, but probably isn't, and it isn't finding it which is causing the `segfault`. Changing it to a `%d` fixes it.


Looks like it is missing `backendUri = backendUri.buildUpon()`, same below.


This just makes the "no documentation" closable by pressing the escape key for example.

Since it's a warming, it draws attention to it (it's yellow), so I would like to get rid of it as soon as possible when I realise that I made a mistake.

If you don't like it, I understand, that is just the way I see it.


Might be worth trying find for cursor rather than toArray.


Added normalize css


This will be an error if system is something other than 'Linux', 'Darwin' or 'Windows'.


This is currently not working, because instead of redirect to:
https://cwiki.apache.org/confluence/display/WICKET/Migration+to+Wicket+8.0
s.apache.org/wicket8migrate redirects to:
https://cwiki.apache.org/confluence/display/WICKET/Migrating+to+Wicket+8.0


**[phpUnit_trunkIntegration run failed](http://qa-s2.wikia-prod:8080/job/phpUnit_trunkIntegration/8450/testReport/) for this commit**


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/21871/testReport/) for this commit**


why do we close reader in startDebugging method? Looks suspicious


To clarify, Slicer fails to build if `Slicer_BUILD_EXTENSIONMANAGER_SUPPORT` is `False`. By default, Slicer will build without error.


Pronto, substituição feita no commit 904470c80ce2f4281ce1541df28b2e00cac36d50.


@ccx-testerbot runtests master


Unrelated note but best practice recommendation is to not start variables with underscores. Technically your usage will not have any issue since you do not use a capital alphabetic character after the underscore, but it's a gray area that's best to avoid:

http://stackoverflow.com/questions/228783/what-are-the-rules-about-using-an-underscore-in-a-c-identifier

The [Google style guide for C++](https://google-styleguide.googlecode.com/svn/trunk/cppguide.html) recommends the form `member_variable_` with an underscore at the *end*, which gives you that same usability but less risky as far as conflicts with the implementation go and less error prone.


As far I could understand, we don't need this anymore, right? If yes, you can remove the whole if block.


This is currently not working, because the user guide is generate to:
https://ci.apache.org/projects/wicket/guide/8.x/8.x/guide/contributing.html
instead of
https://ci.apache.org/projects/wicket/guide/8.x/guide/contributing.html


@tetron, trying to run `bcbio_nextgen` on docker here.

Unless I take the second output from this path:

`/Users/romanvg/dev/bcbio-nextgen/cwl/file:/usr/local/bcbio-nextgen/config/testinput-bcbio_system.yaml`

The workflow does not find the file(s). Is pathmapper on docker interpreting the `File` directive as a file (see `(...)/cwl/file` which does not exist nor is referred anywhere)?

https://github.com/brainstorm/bcbio-nextgen/blob/master/cwl/testinput-args.json

/cc: @chapmanb


Ah, I see. It makes sense.


Thanks for reporting that this broke things for you. Let's keep discussing in #1268


TeamCity pull requests :: netty [Build 368](https://garage.netty.io/teamcity/viewLog.html?buildId=2048&buildTypeId=pulls_netty&guest=1) is now running


We have 2 approaches to solve:
(A) Have the name column & do a search on name
- issue here is that we may have 2 names the same in English but different in translation
(B) Add an extra id column & do the search on ID using the std S3Represent

I would prefer option (B).
An example of this is here:
https://github.com/flavour/eden/blob/master/modules/s3db/project.py#L6344
https://github.com/flavour/eden/blob/master/modules/s3db/project.py#L6318


Javadocs also left ot be generated to:

http://ci.apache.org/projects/wicket/apidocs/8.x/index.html


Всё ровно медленно, можно делать это по dword'ам. [Вот](https://gist.github.com/RIscRIpt/2efd838b68e9904e5837#file-com_munge2-asm-L49) полностью размотанный цикл :)


ah I'm sorry, you need to get it earlier. I suggest you pass this.size to the otsu method in a second argument, at this line: https://github.com/image-js/core/commit/1a4f41699e6f60479a3a137b8543aa056a1726c3#diff-f14335101321717ca151d6f6faeae71dR36


lengthy, add global variable somewhere


I don't have an integer build to hand.  Take -0.5°C = 0xFFF8 so `t=-5000`. Now -5000 % 10000 = 5000, but does -5000 % 10000 == -1?


This `absolute` positioning will only apply in the animation, i.e. when the `ng-enter`, `ng-enter-active`, `ng-leave` and `ng-leave-active` classes are still applied at the `<ui-view>` element.

Once the leave animation is over then element is removed and once the enter animation is complete then the ng-enter and ng-enter-active CSS classes are removed from the element, causing it to rerender and reposition itself with its default CSS code (so no more absolute positioning once the animation is over). This works fluidly so that pages flow naturally between route changes without anything jumping around.


Right on, @nambok !  Nice looking out for the community.  :+1:


I guess it is time to write a wiki page...


Thank you for the request. It has been added to the queue (id: 110). To see the progress you can go to the [status](http://ccextractor.canihavesome.coffee/test/110) page. Please note that depending on the current queue, it could take a while before any results will be visible. In each case I will report back here once the tests are done.


Isn't this a bit to specific? cant we just copy all files?


replace with findPalindromes(2)[palindromes.length-1], remove line 36


Git checkout only works with branch but not with fork and that was the problem.


you can use equals - better for maintainence


Use: os.path.join(b2g_sdk, os.path.basename(urlparse.parse(url).path))


Can we add support for a list of metric ids too? Not sure if we should do it one or the other; or obtain a list of metric based on both. But I see a use case where somebody wants to run this over a predefined list of metric ids.


Yep, it is very dangerous to overvalue a problem and have crashes in prod. This assertion is for development period. So yes -- I want to crash it.


@rtyley I'm wondering if removing this was the right thing to do. Is it not for `awscli` rather than the AWS SDK for Java? http://docs.aws.amazon.com/cli/latest/userguide/cli-chap-getting-started.html


Ah, good spot, that's a copy/paste error :) Thanks


@asmecher
Both $request->url and $dispatcher->url are deprecated. They refer to PKPRouter->url.

Would you use it for the series links or the image links or both? Would you still use display for the template?


I may be confused but [`Uri.buildUpon()`](http://developer.android.com/reference/android/net/Uri.html#buildUpon%28%29) should be fine to create a `Uri.Builder`.


As discussed offline, this method is necessary to fill in the argument, not just to support a custom prefix.


@pynicolas Why not use existing PHPSubscriptionCheck?


Checked and looks ok


how about this
http://stackoverflow.com/questions/3425906/creating-temporary-files-in-android?


Should probably look for the extracted bundle and not the zipfile, as the zipfile could get deleted.


Perfect! In this case, the first loop can start at 1


> s/merge_jobs/try_merge_jobs/

You haven't read my comment.

>  have_newer_t merge_jobs();

Isn't it too geeky? :)


**Name** Antonio Jackson
 - **GitHub**:@antonio015

 ## Completed Courses


Yes, I believe it had some positive effects on the source code as well.


Fwiw, I was resisting this for a long time, but the the size benefits of forcing the annotations was simply to great. Even if I remove the `typeAnnotationQuantifyCapability` completely (to make sure I wasn't pulling in extra class mirrors for return/parameter types), I still was seeing an additional 2x reduction in size by additionally requiring the annotations on the TodoMVC app. This size reduction takes hundreds of milliseconds off the initial render time on mobile, not even counting the download time.

Ideally we get to the point with reflectable where we can get close to the same size performance without requiring the annotations, but its easier to allow people to remove them in the future then forcing them to add them if that doesn't pan out.


@rvonstange DWMR (done with my review). Checkpoints 19-22 are done. Checked off. Press ahead!


問題なさそうです :+1:


@pynicolas I would suggest to use this pattern https://github.com/SonarCommunity/sonar-php/blob/master/php-checks/src/main/java/org/sonar/php/checks/ClassNameCheck.java#L48 when message is presented as static final field (it's easy to find it and creation of issue will be shorter).


It would be better if we have a bool property called "camera2marker" that enables this if block if it is true instead of have a fixed "false" value. You can keep the property default value as "false".

Besides that, in order to keep the coding style, use the braces in another line as you can see in the other if blocks.


Didn't this use to be part of the build system? What's the rationale for moving it here?


This also should of been in a branch >.<


Holding on this PR for a good solution on docker UID mapping:

https://github.com/docker/docker/pull/12648

But for now I'm leaving this as euid 0 (insecure, but just for dev purposes for now), otherwise, the id mapping between an OSX host, boot2docker and its docker machine is rather tricky (uid's 501, 1000 and NA, respectively).

Some Dockerfiles resolve this creating a `docker` user with uid 1000 and gid 100 inside the (bcbio-nextgen) docker image, which is another workaround I tried:

https://github.com/jupyter/docker-stacks/pull/31/files

But then there were several permission issues which I will take care of later on:

```
  File "/usr/local/share/bcbio-nextgen/anaconda/lib/python2.7/os.py", line 157, in makedirs
    mkdir(name, mode)
OSError: [Errno 13] Permission denied: 'log'
```

@tetron, @chapmanb


@joelochlann Hey I *wasn't* going mad when I added this in the first place. This is for the cli, so you might be right perhaps we shouldn't have removed this after all.


Yeah but builder created by `buildUpon` does not modify original uri.


These 3 lines are very important because it ensures that the `<ng-view>` is stretching out to the whole viewport.

If we don't have these lines (especially `left` and `right`) while using the `absolute positioning`, `<ng-view>` will just take the __width__ of its child, the `div` with a `Bootstrap` `container` class. In this case, the whole `<ng-view>` will be pulled to the left most side of its nearest positioned ancestor, which would be `<div class="view-container">` (it's using `relative positioning`), so that the whole view wouldn't be placed in the centre of the viewport.


So far, haven't noticed any difference of using `z-index: 100;` and `z-index: 99;` below than not.


Okay, I will fix this.


@pynicolas why this change?


No worries, my names Edward Maxwell-Lyte. Thanks for updating.


looks good


I can't think (off the top of my head) of another place where we use a getter like this. Would it be more consistent (and simpler) to assign .id at instantiation? Or is there another value to functionizing it?


Nice catch! I think CMake defaults to Debug, but it would be better to set it to Release by default... no one wants huge log files, do they?


This should not be necessary. The `outdated`-event will not be emitted when there are no outdated modules. Instead, [the `noop`-event is emitted](https://github.com/peerigon/updtr/blob/f2c8d61a581f4217da672863d4da70f23db0c240/lib/run.js#L79) (standing for "no operation").

Furthermore, I don't think that it is the reporter's job to exit the process. I think, it's even best practice to not call `process.exit()` at all. When there is nothing to do, the process will exit anyway. When an error is thrown and not caught, the process exit with a non-zero exit code. So why should we call it?


sucks but i guess it'll do


@cekees, I think I will do it this way, changing only the tank3D.py file. Otherwise, I get all these unnecessary commits.


ACK


Why ignore *.md?


good catch. Thanks Brian.


I removed it locally and didn't push it.

Sorry.

Thanks Brian.


As mentioned above, we have to use `absolute positioning` to enabling the overlapping between different `<ui-view>`s during the animation.

We added a `position: relative` style to `<div class="view-container">` so that the positioning of the `<ng-view>` would be relative to this parent as it animates transitions.


Maybe right place for these helper functions is in [here](https://github.com/tangrams/tangram-es/blob/master/core/src/util/yamlHelper.h).


Bench 8073614


Yes, this would be obsolete (as well as the EulerToQuaternion implementation).


In general, we only need to consider whether a pointer is NULL or NOT NULL.
We do not need to consider the sign.


Perhaps... where could we put them though?


any reason all of this is commented out ?


@pynicolas IMHO i would rename previousClause into current, and look into next clause (it's weird to check previous clause and raise issues for it while you have a current one)


Calling sudo from PHP is ridiculous.  Nobody should be doing that, and they shouldn't be using your platform until this is changed.


Shall this change be in prefsEditor.js, on ```fluid.prefs.prefsEditor``` component, so the aria attributes for the reset button can apply to all instances that use prefsEditorLoader, such as full page versions in the infusion repo.


Wow, really dislike this `hh`-prefixed stuff. Can we do something with this? Just an idea: shallow copy `hh` utils to our object and add additional methods. So we will have extended  hh utils. Can we rename `util` dir to `utils` for consistency as well?


It would still be quite interesting to take a look at the generated classes
in order to see if there are some significant subsets of them that will not
be needed and which can be specified out.

On Mon, Oct 5, 2015 at 4:08 PM, Jacob MacDonald <notifications@github.com>
wrote:

> Fwiw, I was resisting this for a long time, but the the size benefits of
> forcing the annotations was simply to great. Even if I remove the
> typeAnnotationQuantifyCapability completely (to make sure I wasn't
> pulling in extra class mirrors for return/parameter types), I still was
> seeing an additional 2x reduction in size by additionally requiring the
> annotations on the TodoMVC app. This size reduction takes hundreds of
> milliseconds off the initial render time on mobile, not even counting the
> download time.
>
> Ideally we get to the point with reflectable where we can get close to the
> same size performance without requiring the annotations, but its easier to
> allow people to remove them in the future then forcing them to add them if
> that doesn't pan out.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dart-lang/polymer-dart/commit/4ebca8eb4f770cb233c7e0ccf8a72b99a43fdd85#commitcomment-13592765>
> .
>

--
Erik Ernst  -  Google Danmark ApS
Skt Petri Passage 5, 2 sal, 1165 København K, Denmark
CVR no. 28866984


@pynicolas why we need this additional class?


I bet you're right. There's a tactical issue of being compatible with
existing env variables, but maybe we can solve that in a shell script.


Faudrait rajouter dans les guidelines un truc du genre faîtes attention à bien placer dans le temps toutes vos modifications des impôts et prestations existants dans le code.


`surveywere`


Pull


This comment is stale.


@chuckbutler This is initially intended to unit test interface layers, but easing the testing of charm layer logic is also a goal, once a bit more mocking is added (specifically around the state functions in reactive.bus).

But, as @bcsaller pointed out, they're independent concerns and should be (unit) tested independently.  With the appropriate mocking, I don't see why a charm layer couldn't include an integration test, as well, though I would expect that to get unwieldy rather quickly.


@sindresorhus, worth keeping this?


`There we over` -> `There were over`


**require-jsdoc/require-jsdoc**: Missing @jsdoc for _customValidate


@tetron, strike this comment about the Docker pathmapper, you seem to have solved this but quite recently:

https://github.com/common-workflow-language/common-workflow-language/commit/1acf6ba987


yes will fix it on my next commit


Looking at the code for getRemoteRevisionIDs I think we can and should add in the interceptors to the HTTP connection, it should be easy for it to build it from the information in the `TestOptions` class


I'd somehow prefer to have it always return marker2camera or always camera2marker (I don't mind which of these) and document the output as such. Inverting a transformation is easy enough at the point where it is actually used (no need to add a property, IMO). Also, one code path less means one possible error less (although this one is trivial).


I don't understand this - I read it as:
> IF user does not have permission to view messages
> THEN show a link to the messages tab


that's not the case anymore since the original library has moved to guzzle


It is a real minor issue, but we should be consistent there.


this line not required. Meteor automatically takes care of including css & js.


The test suite finished running the test files. Below is a summary of the test results:

| Report name | Tests passed |
| ----------- | ------------ |
| _Broken | 15/15 |
| _DVB | 5/5 |
| _DVD | 3/3 |
| _DVR-MS | 2/2 |
| _General | **23/24** |
| _Hauppage | 3/3 |
| _MP4 | 2/2 |
| _NoCC | 10/10 |
| _Teletext | **20/21** |
| _WTV | 13/13 |
| _XDS | 34/34 |
| _Options | **83/87** |
| _DTVCC | **0/14** |
It seems that not all tests were passed completely. This is an indication that the output of some files is not as expected (but might be according to you). Please check the [results](http://ccextractor.canihavesome.coffee/reports/110) page, and verify those files. If you have a question about this report, please contact @canihavesomecoffee.


An addition that requires less changes:

```
func NewSwaggerBuilder(config Config) *SwaggerService {
   return newSwaggerService(config)
}

func (s SwaggerService) BuildListing() ResourceListing {
  return s.produceListing()
}
```

Or how about we create new SwaggerBuilder struct that delegates to the "old" SwaggerService and has public BuildListing and BuildApiDeclaration etc.


Or in `application.yml` (just translate existing env vars to `spring.datasource.*` equivalents). One benefit of doing that would be it would just work in Cloud Foundry out of the box by binding to mysql.

Are the mysql DDL scripts in this repository as well?


@Goober5000 this memset() call is backwards.
I think you intended to write `memset(background, 0, GR_NUM_RESOLUTIONS * MAX_FILENAME_LEN * sizeof(char));`.

As reported by GCC warnings:
```
./mission/missionbriefcommon.h: In constructor ‘briefing::briefing()’:
./mission/missionbriefcommon.h:193:77: warning: ‘memset’ used with constant zero length parameter; this could be due to transposed parameters [-Wmemset-transposed-args]
   memset(background, GR_NUM_RESOLUTIONS * MAX_FILENAME_LEN * sizeof(char), 0);
                                                                             ^
```


When using `cli.globs` there always a file at `program.files`, so we exit in the next (L49) if-branch anyway.


Instead of creating a traverser once, creating it every time (but later times based on one `cli.globs`) makes things a lot easier!


Yes, you are right. I notice that it is confusing too.
What about make it black text on white background and just a rounded corner?
![xkcd](https://cloud.githubusercontent.com/assets/4540682/10282692/231c0bfa-6b4a-11e5-9809-9cbde4c96229.png)
In this way it recalls all the text containers in the page, but the rounded borders make it clear that it is not part of the image.

But I'd like to fix what if too. What about this?
![what-if](https://cloud.githubusercontent.com/assets/4540682/10282858/fa68ffc8-6b4a-11e5-94ff-8b879c1213b4.png)
The text in smallcaps to recall the uppercase of the image, rounded corners and blue border as all the box containers. But I'm not sure about the background: transparent is ugly, white looks too different from the rest of the page, this semi-transparent light blue is the only one among my experiments that I consider nice.

Do you have any suggestions?


Renamed this to `filePath` because `path` is often used by the built-in node module `path`.


This is a bug in inttypes.h ; it should be inheriting from the compiler's definition at runtime if it is defined.  That said, we do use wchar_t in the code, just not by name. There's not a lot, but ucs2.h uses it - L'\0' is a wchar_t.


Nah


Maybe, it's more one-offs for different tests which becomes annoying and difficult to maintain. I tried to rewrite it to check each remote counterpart based on all of the local revs but gave up because really push and pull should be two different tests.


Looks great to me jhthorsen :+1:


@antonio015

can you add your profile in the issue `#31`?

Thanks! :smile:


@Tarabyte, this pull request is awesome. Perhaps you could comment some on the changes here and explain why and how they work. Thank you!


TeamCity pull requests :: netty-tcnative [Build 24](https://garage.netty.io/teamcity/viewLog.html?buildId=2049&buildTypeId=pulls_netty_tcnative&guest=1) is now running


The example includes a `sender` - this should mention it too.


beim zweiten commit habe ich auch keine Message hinterlassen.


TeamCity pull requests :: netty-tcnative [Build 24](https://garage.netty.io/teamcity/viewLog.html?buildId=2049&buildTypeId=pulls_netty_tcnative&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:01:01


@davidrapson Glad to have confirmed your sanity. Though I can still download stuff from s3 with aws cli without this config file. So maybe it doesn't matter for most operations.


Would be ideal to generate the Add Child option from `ContactSchema` instead of hard-coding special cases.


New loot system is almost perfect. However, my server sometimes crashes with new loot system. Here is the crash log:
000000014012A18A  0000000006ACD830  Pet::_SaveSpells+5A  e:\112server_1\trunk\src\game\pet.cpp line 1385
0000000140126CDD  0000000006ACDA10  Pet::SavePetToDB+12D  e:\112server_1\trunk\src\game\pet.cpp line 395
0000000140184C13  0000000006ACEC60  Player::SaveToDB+1163  e:\112server_1\trunk\src\game\player.cpp line 14537
0000000140191BC2  0000000006ACECF0  Player::Update+412  e:\112server_1\trunk\src\game\player.cpp line 1244
0000000140117806  0000000006ACFDE0  Map::Update+1B6  e:\112server_1\trunk\src\game\map.cpp line 519
000000014022F7CA  0000000006ACFE10  MapUpdateRequest::call+4A  e:\112server_1\trunk\src\game\mapupdater.cpp line 5

Do you think what probably caused these crashes.


Nice job, coming back to the main thread for UI stuff...


Also tests that deletion works - https://github.com/OpenDataServices/cove/issues/112


i see jfltexx there yeah!!!


yes, You right.
I was wrong.


動作確認して master に反映しました！
情報ありがとうございました！


What I mean is that using `present.map!(&:last)` is exactly the same as the line of code you’re using here while being more memory/time efficient.

I don’t think there’s a function for "all of these are included".


It's worth noting that the setting of `cpuid_feature_mask_key` is removed here as well.


Not needed to set -1 here, as 0 is invalid as well, you can just leave out the `Tasker` altogether.


Reason for an extra r that I don't know about?


Should that `<` be there after `./client`?


I think the `isAdmin` attr should belong to the `FormView` instead. When instantiating a `job_admin` form, it should be rendered automatically by the form theme with the proper fields, without doing this.


Debug level, or at least not info.


Hello @amenk It has been just moved to https://github.com/magento/magento2/tree/develop/app/code/Magento/Deploy/Console/Command
Still available as `setup:static-content:deploy`


I think it was caused by this kind of weird spacing. I don't know what it was. It's not even a tab.


Business logic in Program.cs is a bad idea. Please (re)move.


Yes, it is piping the script file into the client utility.
Is that what you are asking?


Thanks. I did consider both leading and trailing underscores, but ultimately settled on leading. I decided the user is more likely to believe they have a valid reason to use a trailing underscore in an enumerand, than a leading one – similar to the linked style guide. So, I preferred to avoid conflicting with that. There may have been other reasons too, but I can't remember them right now.

As for conflicts with the implementation, I don't think any are possible (up to implementation flaws). All my underscored names are scoped, so I have control over the full set. As long as the implementation doesn't inject incorrect names into the scopes, the worst that can happen is shadowing of an implementation name from another scope by my name. However, that should be discovered in testing, since Better Enums contains the code that would be affected by shadowing.


Might as well do the same thing as on line 183, instead of creating a new function.


@taconaut Java's lack of multiple inheritance has always annoyed me, the class/interface dichotomy is a really poor substitute.

It occured to me that you could bring back ```net.pms.dlna.RootFolder``` as a subclass of your own RootFolder and import that everywhere instead of ```net.pms.medialibrary.dlna.RootFolder``` (i.e. 'unpatch' that part of ums), so I tried it and the class loaders are now happy afaict and jumpy's rootfolder refresh doesn't throw any error (though I still don't end up with an absolute path but that's no big deal). But of course I haven't made a deep study as to other possible side-effects for mlx.  Diff is here: [reroot-mlx.diff](https://gist.github.com/skeptical/d0774a450ca488d9ed77)


This is the equivalent of `pip install -r requirements.txt` which can be written as `pip install -rrequirements.txt` and if you use `-r requirements.txt` tox tries to use ` requirements.txt` as the file which doesn't exist so you need to trim that extra whitespace.


Done bbb5fa9


I think you can get at this with Services.cpmm.


The problem with this one is that not all websites (many of them, now, but not all, by any means) support `https`. We'll get this error if we type in a URL with no scheme where the server doesn't support https:

<img width="578" alt="nohttpserror" src="https://cloud.githubusercontent.com/assets/5395845/10283348/140bac3a-6b34-11e5-8d48-7f62f3eb0699.png">

So you might want to either *try* https and then handle the failure and retry with plain Jane http or default to http (which is more insecure, sure, but also more prevalent).


TeamCity pull requests :: netty [Build 368](https://garage.netty.io/teamcity/viewLog.html?buildId=2048&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5618, ignored: 21 Build time: 00:37:53


@tpietzsch Awesome! Thanks. ;)


Oh! Carry on :smile:


Is this feature complete? If I am getting it correct, this is just a template of how custom pre-processors might be implemented. If someone wants to implement their own , he can take these files as a starting point and insert own details.


develop (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1370 of 1370 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1373 of 1373 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


`m.room.member`


This is a bad idea to use strings. At least use enum


if you're just doing a `find_each`, a `count` would have done the same.


`errno` should be modified in the following 2 `goto exit;`


Unless the display name is going to have HTML in it (I don't think it will), you should use textContent instead of innerHTML.


This should be checked during compilation! Especially the else part.


I really used oauth functionality for my stuff as i don't provide openldap. Some open configurable oauth middleware should be awesome.


you can initialize it here


I don't like scripts that change my configuration. Maybe we should log that we use the --local option and which ip we use for the local option. Before we start using the local option we should check if we need it at all. I guess we don't need it.

Perhaps there is a shell function in /lib/functions or something that extracts "the" ip of the interface.


textContent


Muss nicht aufgerufen werden (nur in der add.php festgelegt werden).


Shouldn't we depend on `media-libs/mesa[opencl]` for that?


...to not write this


"== false" ?


This looks wrong to me, you should probably just use the VSVer variable that is defined at the beginning of the script


.textContent, or getAttribute("value"


In `scripts/` we now have two shell script and one SQL script for Travis. Could we at least combine the Travis script and `create_user_db_test.sh`?


btn.button(data-dismiss): Muss nicht aufgerufen werden, nur festgelegt (in add.php)


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


Can we use static constructor instead?


So this won't work with the command event anymore? https://developer.mozilla.org/en-US/docs/Web/Events/command says that this should work for both mouse and keyboard accessibility.


<?php echo $profile["vorname"]; ?> <?php echo $profile["nachname"]; ?>
: Umstruckturiert damit es nicht doppelt angezeigt wird wenn nur die edit.php geladen wird (das ist bei einem neuem Profile der Fall).


It still might be a bad idea to pass error messages as the 'result' element if somebody decides to rely on the fact that the result is empty in some error checking code in future.
In 9b44dc89b17717f81798eda33b48d4c42d3879ac I change this to pass the error response body [as an element in the](https://github.com/matecat/MateCat/blob/9b44dc89b17717f81798eda33b48d4c42d3879ac/lib/Utils/Engines/AbstractEngine.php#L143) `error` array.


You can just do event.target.id. Also, no need for a switch if there's just one item we're looking for.


Upstream code will catch this (or not). The application should crash!


Variable name is obscure


@skeptical haha that's a hacky solution which should work, hadn't thought of that :) (I'm not used to keep backward compatibility at all cost)


Also, this type of hack might be danger in case of multiple emerges running in parallel. Maybe it's worth to show big red warning unless gentoo build system provides some better method for that?


You can use an arrow function in here instead of a function(){}.bind(this), like this:

    this._chooseClientApp((aResult) => {
       if (!aResult) {
         // In this scope, "this" matches the outer context's notion of "this"
       }
    });


textContent or setAttribute("value"


Me neither, but it would be zero. We wouldn't want to see "-1.5000" printed.

How about this? Add a 'sign' string. If t < 0 then sign = '-' and t = -1 * t

The way it prints the temperature is dodgy. I could change it to construct a string first, e.g. "-0.5000", then print that as a unit.


selectedItem.id


@vilchik-elena we need a way to visit a tree (not a full file). E.g. to implement JumpStatementFinder (inner class of NonEmptyCaseWithoutBreakCheck). Is there a better way?


Good, thank you


Yikes, I am going to get major merge conficts!


Should this and getContentHandlers throw Cr.NS_ERROR_NOT_IMPLEMENTED?


这里将数据存入时手动进行查询是否已经存在，如果直接用set存放的话就可以避免这个问题


@taconaut "hacky" is my middle name :-).


Did you just forget to remove the above two lines?


ok


@vilchik-elena Because the new grammar allows `__halt_compiler()` only as a top level statement.


Do you need these assertions?


Why can't we use the standard mail properties from Spring Boot? Is it because it doesn't offer some properties?


lets change this to something that is immediately understandable regardless of the folder its pulling from. i.e `wp_assets` or `wp_media`, "f" is confusing if you have no prior knowledge of our bucket structure


still hardcoded...


.wrapp :)


Because they will have their own version using the "mail." namespace -> we will migrate to it as soon as it's ready, but this will be in Spring Boot 1.3. So at the moment I'm trying not to clash with it.


> I think it would be better to introduce a new `qualified_lookup` instance method

Ok, I can work on changing to this

`qualified_lookup` is already a method for Backend with incompatible parameters.  It's also used by the `Hiera::Interpolate` class.  Should we pick a new name for the new qualified lookup function, or rename the existing `qualified_lookup` to something like `qualified_extract_subkey` or just `extract_subkey` since it isn't actually looking anything up?


Remember to change this line accordingly


Unfortunatly this modification impact international names like:
```
{ "City" : "Neuchâtel" }
```
I expect Stringify( ) to return "Neuchâtel" and not "Neuch\u00E2tel"

Using ANSI (iso-8859-1) or any character set doesnt matter.


at least make it a constant or a part of Configuration


Ok! https://github.com/lis-epfl/MAVRIC_Library/commit/173f08e62fcab96d40d16d6ffd39a08cfc020cdb