Checkmarx scan results 2015-08-05T13:45:38Z

High
Medium
Low
Info

New
0
0
0
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

2df19aa46451ad26f4dc72164d23cf438903565b	@cxehud

ad58b128e2fede39820ae668970bbfab38b56195	@cxehud

6b05867d4fe80a4c3b55cb36e0b6f76abc0a59f9	@cxehud

accf7e51faac6fffc26f426e5afeeaf3ed561731	@cxehud

7917c1e1165bbb3f8b67a7c2943b68d3780e2a79	@cxehud

74262a0e47571dc6ed4aa4c3a03f759fee5bd42c	@cxehud

5b96953244ae55ea53f4983dc1a9fa4d4eb872a6	@cxehud

2166d1ea7164e801468b1b1f20ff0a3ecf4c67dd	@cxehud


Weird.
Yes it is definitely easier to leave as is :)
But I am not opposed to making this better.
I think more tests showing when it works and when it breaks would be a first step.

Another option is to create you own protoc-gen-vanity using the vanity library:
```
func main() {
	req := command.Read()
	gogoPluginNames := []string{"unmarshal", "unsafeunmarshaler", "union", "stringer", "size", "populate", "marshalto", "unsafemarshaler", "gostring", "face", "equal", "enumstringer", "embedcheck", "description", "defaultcheck"}
	param := "plugins=" + strings.Join(gogoPluginNames, "+")
	if req.Parameter != nil && len(*req.Parameter) > 0 {
		param = *req.Parameter + "," + param
	}
	req.Parameter = &param
	resp := command.Generate(req)
	command.Write(resp)
}
```
Simply remove the embedcheck string from the pluginlist.


Checkmarx scan results 2015-08-05T13:36:54Z

High
Medium
Low
Info

New
0
33
22
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

c72690eba7e11492d97f0c7ab1239411e51d1488	@dortal

f4545b47b9f24835acdb9ec38420d31bea849110	@dortal

abb4c31a1e965ba5fa14857ae8935a2269a7a0b6	@dortal

565af35d4610101d7f0d7a9df6153063c12d0d3f	@dortal

bb37aa4384e3495bddd16447868529612cc14d22	@dortal

0467f9dc03aa49abf3742cdea0e547b1c9cf8179	@dortal


Does this need to be final? I don't see it being used anywhere besides calling `.start()` later in this method.


Why? Isn't that the point of the listview?


@akostadinov@brasmusson looking at this more closely the Hash::try_convert does not work as expected here.

Specifically `Hash(nil) != Hash.try_convert(nil)` and as @akostadinov has pointed out we risk having nil's. I wonder if we should backport the Hash method for 1.9.3.

@mattwynne when are we planning to drop support for 1.9.3?


Did you get it to work when opening the viewer after the session? If the session name is `None` the loop is prevented by the first condition in `slotSessionChanged`.


s/The//


kconfig will select sol_bus only if its dependencies are resolved, enforcing platform_systemd to depend on sd-bus will result on configuration breakage for scenarios with systemd present but without sd-bus. See, distributors may want to split the packages so, this scenario is completely feasible.

Besides that, the patch makes sense to me. To be clear, please just drop this systemd -> sd-bus dependency and we're good to go.feasible


Ah, now I get it.


> @Davorak: Who pushed it doesn't reflect who authored or committed it anyway. Note that I can push a repo with commits done by anybody else.

@hobarrera - I was/am aware, my main point was unlike the the author of the commit, to my knowledge, it is not possible to spoof the login of the pusher given by the events api.  So when using github and without additional signing a logical step is to hold those who push code to a repo responsible for any negative consequences regardless what the author information says, barring a some exceptions.


See above comment.


Works for me :-)


constraint


In commit note: "sentece" -> "sentence".


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.891.33e9bdb3491e06e1192c6f3a622fca15bc9b84f6](http://build.palaso.org/viewLog.html?buildId=60610&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 397, ignored: 4 Build time: 00:03:34


I'm not sure this is the right way to fix the issue because I don't understand why we need this special case.


Checkmarx scan results 2015-08-05T13:50:15Z

High
Medium
Low
Info

New
0
0
0
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

2df19aa46451ad26f4dc72164d23cf438903565b	@cxehud

ad58b128e2fede39820ae668970bbfab38b56195	@cxehud

6b05867d4fe80a4c3b55cb36e0b6f76abc0a59f9	@cxehud

accf7e51faac6fffc26f426e5afeeaf3ed561731	@cxehud

7917c1e1165bbb3f8b67a7c2943b68d3780e2a79	@cxehud

74262a0e47571dc6ed4aa4c3a03f759fee5bd42c	@cxehud

5b96953244ae55ea53f4983dc1a9fa4d4eb872a6	@cxehud

2166d1ea7164e801468b1b1f20ff0a3ecf4c67dd	@cxehud


Okay.


this should be in the anonymous class

and the class should not be anonymous (SonarTech will complain)


fixed.
https://github.com/OpenPTT/OpenPTT/commit/672c0eec5fbbb32f747d4a8b76e5cd1c7faf8be5


Is there really a need for a logout route?. Nope, please implement that as a function in your AuthController.


#define isn't better, we agree on that ^^

This function may even work on all plateforms : spec_ordre's fields are generic enough to work even without xdg. Maybe `get_std_path(spec_order)` ?


> "opam pin cohttp --dev" just works

it will just work with the version in the opam file as well.. In that case it will be set to the latest released version of cohttp, which is usually what you want. If you want to pin it to a specific version to test constraint bounds, you can do it manullay (`pin cohttp.<version>`). Both solutions are ok, but I really think that letting the version in the opam file leads to more annoying issues for the default case (ie. I see commits to fix out-of-sync versions every day or so :p)


By default, it's type is set to `text/css` so you don't need to explicitly define that.


Other than the typo in the commit note, this LGTM :+1:


Your indentation isn't consistent. Please Fix.


Use the `lambda` method for multi-line lambdas.


`addColRightTemplate` is exactly the same thing as `addColLeftTemplate`, so there's no need for extra variable. Also, there's no need to call `A.Lang.sub` here, it's only used for substituting variables inside the template with data, which you're not doing, so you can get rid of both variables by passing `this.TPL_RESIZE_ADD_COL` directly to both `A.Node.create` calls.


Do you want to build house on the space by the left?. :open_mouth:  Please Fix this wrong indentation.


Nitpick: this should be `%s#%s` to follow the github PR shorthand convention.


@yamii I don't see why it makes a difference. The these all look for the `options.rabbit`, and if that's not there then it just uses the entire `options`.
* https://github.com/School-Improvement-Network/lapin/blob/master/lib/req-res.js#L16
* https://github.com/School-Improvement-Network/lapin/blob/master/lib/pub-sub.js#L14
* https://github.com/School-Improvement-Network/lapin/blob/master/lib/send-receive.js#L14

The only thing with this is that if you are going to pass in `options` they need to include your own version of `wascally` to go with it.


PHPCS: Line exceeds 120 characters; contains 128 characters


Avoid unused local variables such as '$sitemap'.


Looks good.


I think the issue is that the metadata are stored in Python lists and ```all()``` operates on Numpy arrays. To keep the code consistent, let's modify as below:

```python
if not (np.array(self.mdRCConst)>0).all():
					self.rejectEvent('eInvalidRCConstant')
if not (np.array(self.mdStateResTime)>0).all():
					self.rejectEvent('eNegativeEventDelay')
```

The above code passes all tests.


Yes.


`Set.new` here isn't the greatest user experience.  The only reason the user would need to do that is because the diffing library we are using right now doesn't handle arrays of objects very well.

I would like to make this process automatic by inspecting the `expected` value to see if it can be cast to a set, and if so, doing that.

It would also be great if we could figure out how to get sdiff to correctly diff lists of objects.


Ah, good idea.


I would, but I don't know what version I actually depend on, if you do let me know.


When updating a recipe with Haikuporter, one gets constantly:
recipe xyz is still broken:
Error: package xyz cannot be built for architecture blablah

It would be a good idea to evacuate such messages when an architecture can't be supported. Not mentioning the architecture seemed rather straightforward for me.


Looks good. Though I wonder if it is worth keeping the strv_split_quoted() alias around instead of just patch all the invocations.


This is STORE_LINK which is used on Imin.AO_LINK, not on AO.
The difference with STORE it's that add function has one more argument (the key of the root).
I changed this signature because the code is more simple when we return the key that we given :
```
 (fun x ->
 let _ = PI.add t.pi index x in
 Lwt.return index)
```
by
```
 (fun x ->
 PI.add t.pi index x)
```


Well I appreciate everything you're doing, its making my job easier and others' as well. Don't work too hard.


@ganchclub why you change exception type?


No reason.


Lines 149-155 are almost the same as lines 156-162. It's be best to refactor this logic into a separate function that can be called twice here with different params instead of copy/pasting like this. This function will be cleaner as well, right now it's very big.


You might want to replace `2.1.0` by `latest` in that link.


I'd assume most places would want to a human to decide when it's time to deploy a new release to prod. Tagging prod-ready is fine, but I question if you'd want an ICT on it.


You mean libltdl? I don't know, by it's nature ctypes.sh requires very specific knowledge of how dlopen works. I don't think libltdl is a good match - it's a deliberately minimal interface.


If we do that, then we'll need a way to annotate when we /do/ want ordered comparisons.  Unless we can think of a way to not have to annotate in either case, I think I'm preferential to annotating on the Set side since then array works as it normally does in other ruby uses.


You'll have to be less cryptic, what's unsafe about it?


Agreed.


@ganchclub and should be in lowercase


I'm using AC_CONFIG_FILES in an uncommitted version, but I still need AC_OUTPUT to build the wrapper script.


Okay, just wondering if I had missed anything :)


There's no need to create a variable just to hold `this.get('addButtonLeftClass')` since it's only used once below. There are too many variables at the top of this function, you could remove `addColLeftButton` and `addColRightButton` by passing the call to the attribute directly where it's needed.


@ganchclub remove default value


Is the following yacc code still valid?
```
%%
A : 'a'
```

This line suggests that it needs to see "%%" to finish, but it seems like the `while i < self.src.len()` takes care of this case.


:) Great point! I wasn't aware that existed.


This is the best commit of the project!


Thanks for your recommendations!!


Plasticity in the GK version is generally working now, I tested it with my benchmark tpv12/13.
In the GK version I got slightly different results as for the classic version when I use plasticity:
This is not the case for the plastic simulation (see below).
Any ideas/comments why this is the case?

Plots: CL = classic, GK/github=generated kernel
1. off-fault receiver, normal stress (y-axis in MPa not in m)
2. on-fault receiver, horizontal displacement
3. same as 1. and 4. same as 2. for the elastic case
![plasticcl-gkofffault](https://cloud.githubusercontent.com/assets/9365957/9085971/97a3781c-3b81-11e5-8f62-bac93b553657.png)
![plasticcl-gk](https://cloud.githubusercontent.com/assets/9365957/9085972/97b9a696-3b81-11e5-953b-fa0412934541.png)
![elasticcl-gkofffault](https://cloud.githubusercontent.com/assets/9365957/9086050/234e4342-3b82-11e5-80f6-ff97d71515f7.png)
![elasticcl-gk](https://cloud.githubusercontent.com/assets/9365957/9086051/23542488-3b82-11e5-8d13-ec776f838b13.png)


Consider rearranging the entries to be in order of low->high.
Copyright date at the top of the file can be updated to include 2015.
Thanks,


It isn't -- but that's because it's missing a ';'. But modulo that, yes, you don't need to have a %% for the programs (e.g. many of the tests don't have this).


If you need an account on our jenkins server please ask.


I'm sorry, I had an outdated version of the cabnavigator.php file :-)


Haha this is a good one! I added it here for namespacing purposes, to reflect that this method was (currently) only needed as part of a larger "logical entity", that is `ensureOnlyOneOverlayOpen()`. I think either way works great, and a separate method like you're referring to would have the added benefit of being a little bit shorter :smile:


参照楼主这样修改完后，分页还是不正常啊


참 잘했어요!


Если ты читал мой issue про дружественные функции. то мог увидеть что я об этом писала. И в качестве пробы написала в поле другой области видимости, как просто пример для себя.


参照作者这样改完之后，分页还是不正常啊


Checkmarx scan results 2015-08-05T13:52:44Z

High
Medium
Low
Info

New
0
0
0
0

Recurrent
0
33
22
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

41be6851f50986ab2d4e3db987c3e42f469b1ada	@dortal

c72690eba7e11492d97f0c7ab1239411e51d1488	@dortal

f4545b47b9f24835acdb9ec38420d31bea849110	@dortal

abb4c31a1e965ba5fa14857ae8935a2269a7a0b6	@dortal

565af35d4610101d7f0d7a9df6153063c12d0d3f	@dortal

bb37aa4384e3495bddd16447868529612cc14d22	@dortal

0467f9dc03aa49abf3742cdea0e547b1c9cf8179	@dortal


There doesn't seem like you changed anything?


I believe it ends with an 's', so `./app/cache/dev/translations`


:+1: :)


How do you think about to support an autoconf software version which is a bit older than "2.69"?


Hmm, not sure I like using strv_split_escaped() here like this, and the strdup()ing the elemtns...

I think we should just extract the parts directly, if it's a fixed number of elements... Or at least steal the strings from the strv instead of dupping them, and then free just the outer array instead of the whole strv...


We're a bit conservative on tagging new releases, but this should be released with 0.10 in the next few weeks.


Looks good.


This has undone your previous fix...


Читал) Не ругайся. Хорошо.


access rights should work better than this, you have or not access
no sudo


Lambda_MemberArray_Reduce (mbadams5) - x86_64-MacOS-10.9-clang: OK (1766 of 1771 tests passed)

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


`SystemMemoryUpdater` is a better name


Thanks for the feedback on this @christian-scheible .  I'll revert this commit from the dspace-4_x branch. That sounds like all that needs to occur.


That is the intent. There will be a similar spec for each platform. On OSX, XDG_CONFIG_DIR will fall back to ~/Library/Preferences and so on...

I feel like in this case, a global variable is more natural than adding an extra function layer.
What is the benefit of `get_std_path(CONFIG_HOME)` instead of `std_paths.config_home`?


@SgtOddball thanks for taking a look. There's still a bit to go, but getting closer. Been manually testing along the way. It's been made much easier with elements prepended with $ where they are jQuery objects.


fugly, make a new module


Should carry a comment in the code of the function that clarifies that this changes the strv in-line, and does not roll back on error, but leaves the "half-processed" array in place...


Hmm. Not sure what happened there. Pushing updated version.


> Nice work with the rapid prototype of hypothesis integration

My pleasure... my present learning model is:
**Heutagogy** + a pinch of **Paragogy** served on **Peeragogy style**  :tophat:


Wrap the arguments


> Would expect this to use SubjectAccessReview

Can't, that would fail when you have a new registry image talking to an old openshift.


Ok, no problem.


ha right, ok. As long as it doesnt' break the existing API I'm happy :-)


Looks good!


讨论即将开始


Removed password from the logs.


No need for this constructor. The only unchanged usages are in tests. Add the dummy listener there.


There was also some cleaning of code with this commit along with the OS detection.


I don't think this is meant to be here either - has it got pasted in by mistake?


好，讨论结束!


let's tied this in with the rest service and drive the template based on what's configured in the rest response. So if a system has only 1 input and 1 output with no aligner, we will hide the unnecessary tabs. In other words, we should show tabs only based on the rest response.


Es gibt auch assertNotNull. Das macht im Prinzip das gleiche nur ohne das man 'null' angeben muss.


@rhc54 Just curious: is there a reason your're adding -g back to the general CFLAGS here?


This class was just added in the previous commit. Update the other commit to have the class in the final location.


Oops. ;)


Maybe call this DONT_COALESCE_SEPARATORS, to make this more understandable?

(i don't like "negative" naming but in this case it is really more explanatory...)


Name this `SystemMemoryUpdater`


There is no "two" implementations, but an effort to have a best design, and if we want to succeed we need to test it too, until the 2.7 release.


Would you like to exit from this configuration script for example if an appropriate shell command interpreter was not found?


I didn't understand why this was necessary here, can you explain it to me @fernandosouza?


maybe better like this:

```
BOOST_ROOT/bjam toolset=gcc cxxflags=-fPIC cflags=-fPIC location=bin/linux/x86_64
TARGET=bin/linux/x86_64/libjlibtorrent.so
mv bin/linux/x86_64/libjlibtorrent.so.1.1.0 $TARGET
...
```


good catch


What is this supposed to return? Add Javadoc for it


It is an habit to write the ; at the end of lines


Looks good.


It seems like you didn't include all changes in this commit? :smile_cat:


`dragNode` is not a `A.LayoutRow` instance, it's a `A.Node` instance from what I see. Can you update this?


`RecordSink` needs this as well


fast-forwarding 2.0 to borshop-integration-532-handling/zl/better-handle-of-wm-errors-yz_wm_search = 9c3bfba5


Looks OK.


Okay, I messed up that commit message. I actually meant "I'll just use fixed positioning."


Sure, it will still working, but it is more common in many languages to usage of parenthesis for functions


This API does not work for connectors that allocate lots of memory in `commit()`. Though as discussed with @dain yesterday, we might need to break that up anyway allowing it to be iterative.


I agree, but the testing must happen in a **separate** repository. It will be hard enough to fix one implementation. If you want to maintain a second one (even only until we make a release), that should be your responsibility alone.


Yes, but potentially with variance in the versions of nested dependencies, etc. Checking `node_modules` in is the easiest way to ensure that nothing changes unexpectedly. (I always find `npm shrinkwrap` too hard to maintain in practice.) I'm only committing the client-side modules, which one would hope are pretty lightweight anyway.


Checkmarx scan results 2015-08-05T14:10:12Z

High
Medium
Low
Info

New
0
0
0
0

Recurrent
1
1
2
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

2df19aa46451ad26f4dc72164d23cf438903565b	@cxehud


Checkmarx scan results 2015-08-05T14:04:06Z

High
Medium
Low
Info

New
0
0
0
0

Recurrent
0
33
22
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

dabbec16f3f2e33ce610861fbbc290f3e48de249	@amitchen22

41be6851f50986ab2d4e3db987c3e42f469b1ada	@dortal

c72690eba7e11492d97f0c7ab1239411e51d1488	@dortal

f4545b47b9f24835acdb9ec38420d31bea849110	@dortal

abb4c31a1e965ba5fa14857ae8935a2269a7a0b6	@dortal

565af35d4610101d7f0d7a9df6153063c12d0d3f	@dortal

bb37aa4384e3495bddd16447868529612cc14d22	@dortal

0467f9dc03aa49abf3742cdea0e547b1c9cf8179	@dortal


> one step removed from anarchy and nihilism - there should be no law, religion or government, it's every man for himself.

That's not what anarchism is. Anarchism simply makes membership of an 'association' (eg. some form of government) fully optional and voluntary. It's not a synonym for "chaos", however much some media outlets would like to suggest otherwise. Quite the contrary, in fact - in a typical anarchist system, there's no 'falling through the cracks'.


Nice Job!..However rename `TestChopsPostCept.php` to `CreateChopsCept.php`. That's more readable and understandable.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.892.05a0406dd42a8c4f85f5ce0cb493fd7ddb63e760](http://build.palaso.org/viewLog.html?buildId=60612&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 397, ignored: 4 Build time: 00:03:49


The optional separate destination path makes things more interesting.

I think a call to `extract_many_words(&s, ":", EXTRACT_SEPARATOR_SPLIT, &source, &destination, NULL)`, followed by a `destination = strdup(source)` if it returned 1 rather than 2 would probably be the neatest way to do this.


Why would there be variance (out of curiousity; I'm not doubting you)? I thought that if you installed v x.y.z of a module, it came with its own package.json that listed all of the versions that it had as well.


This variable is only used once, so you could just call `row.get('cols').length` directly below, so this big list of variables can be smaller. Sometimes creating a variable like this helps make the code become more readable even when it's only used once, but in this case `row.get('cols').length` is already quite clear, so the variable is just making the function bigger, which makes it harder to read in the end.


sounds good to me...


This is a comment


Symbol has a new method, e.g.

Symbol::new(sym, SymbolType::Terminal)


"a" or "the"?
I'm not sure.


should should ;)


ply:IsAdmin() always return true if player has SuperAdmin rights


I don't understand, let's have a talk


Yes, basically. But I have a project specific need to remove this as a dropdown dependency with heavy custom CSS that is causing issues with DOM recycling. I don't want expect anyone to need this fork, It's just for that single project.


So that this
![mtjpd88](https://cloud.githubusercontent.com/assets/6500800/9087864/8ea462ce-3b8d-11e5-82e5-6f90eade5885.png)

is now this

![screenshot000](https://cloud.githubusercontent.com/assets/6500800/9087871/9667b6d2-3b8d-11e5-9a1a-ecad4bc5abc2.png)

in all views.


Good point!


ok


indeed


The principle of implementation really successful i.e. to use information on the breaker size.

But unfortunately in cabnavigator it still isn't evident, and in CDU tooltip it isn't realized yet and 3ph CDU is displayed without division of phases there...

For comparing I send two screenshots as it is realized at me (one 3ph CDU is division into three 1ph CDUs):
![splitphase_my](https://cloud.githubusercontent.com/assets/10425179/9087895/146ad408-3b96-11e5-8b24-addfe7ae0174.png)
![splitphase_new](https://cloud.githubusercontent.com/assets/10425179/9087909/201b1d8a-3b96-11e5-9269-a759431f449c.png)

Very thanks for the support.


Just convert the acceptance to functional tests. Delete all the acceptance tests.


Call it "correct-password"?


Do you really need any functionality which is not covered by the Libtool interface so far?


Please convert to functional tests.


Call it "wrong password"?


review acl on projects: leave internal and followers only, review customer rights: need to be in follower_ids, public must go


Thanks


This comment originated from dashboard_controller.rb in commit 3a28ea79 from early 2011.  Is it still needed?


I don't think you need all these parentheses here.


Don't ignore case? The goal is to have a strict checker here so we know the code can pass info with integrity.


@digitalextremist Here are the changes so far. What are the remaining parts?


merge portal_project and project


The backticks look good here: https://github.com/gaborcsardi/maxygen#links but probably you are right and the other form is better.

As for the alternative notation, you mean the equivalent of `\link[=dest]{name}` ? Then the whole `[text](::function)` thing would be within backticks? Or not? This is a not entirely consistent with the `` `pkg::foo` `` that I implemented. It would be nice to have a consistent form, either always with brackets/parens, because that's used for links in md, or without them entirely, like: `` `::foo=dest` ``, although I am not happy with this last one either.

Maybe it is best to just use brackets and parens. I'll reopen that issue.


There should be no space between `size` and `--`: `size--`. Same for line 269.


#include <iostream>
#include <fstream>
#include <cstring>
#include <stdio.h>
#include <string.h>
#include <array>
#include <vector>

using namespace std;
struct TRegLLeg
	{
		string PlanetaDesignado ;
		int IDindividuo;
		string Nombre ;
		string Apellido ;
		string Nacionalidad ;
		string InformeExperimentos ;
		int DanioPsicologico;
	};
	struct TRegSal
	{
		string PlanetaDesignado ;
		int IDindividuo;
		string Nombre ;
		string Apellido ;
		string Nacionalidad ;
		int Edad;
	};
	struct TRegVector
		{
			int IDindividuo;
			int Edad;
			int DanioPsicologico;
		};
int main() {

	TRegSal TRegSalida;
	TRegLLeg TRegLLegada;
	int TamanioRegSalida;
	TamanioRegSalida= sizeof(TRegSalida);
	TRegVector Vector[TamanioRegSalida];

	bool Esta;
	int Pos;
	int LLegan, Viajan, Participan;
	FILE* fllegada = fopen ("C:\\Users\\Daniela\\Documents\\Algoritmo\\tp algoritmo1\\ArchivoLLegada.dat","w+b");
	FILE* fsalida = fopen ("C:\\Users\\Daniela\\Documents\\Algoritmo\\TpAlgoritmo2\\ArchivoSalida.dat","r+b");
	Esta=0;
	int CantVect=0;

	fread(&TRegSalida, sizeof(TRegSalida),1,fsalida);
	while (!feof(fsalida))
	{

		for (int j=1;j< TamanioRegSalida ;j++)
		{
			if (TRegSalida.IDindividuo == Vector[j].IDindividuo)
				{
					Esta=1;
					Pos=j;
				}
		}
		if(Esta==1)
		{

			if (TRegSalida.Edad <= 50 && Vector[Pos].DanioPsicologico < 3)
			{
				//Pregunto si cumple con los requisitos de mision
				// HACER ESTE PASO EN UNA FUNCION VOID
					TRegLLegada.IDindividuo = TRegSalida.IDindividuo;
					TRegLLegada.PlanetaDesignado = TRegSalida.PlanetaDesignado;
					TRegLLegada.Nombre = TRegSalida.Nombre;
					TRegLLegada.Apellido = TRegSalida.Apellido;
					TRegLLegada.Nacionalidad = TRegSalida.Nacionalidad;
					TRegLLegada.InformeExperimentos = " ";
					TRegLLegada.DanioPsicologico = Vector [Pos].DanioPsicologico;

					fwrite(&TRegLLegada,TamanioRegSalida,1,fllegada);

					Vector[Pos].DanioPsicologico++;
					Vector[Pos].Edad=Vector[Pos].Edad+6;
					//Contadores de individuos que llegan y viajan
					LLegan++;
					Viajan++;
			}

			else

			{
				//Si no cumple con los requisitos de mision, leo otro registro
				fread(&TRegSalida, sizeof(TRegSalida),1,fsalida);
			}
		}
		else
		{
			//Guardo el registro en archivo
			TRegLLegada.IDindividuo=TRegSalida.IDindividuo;
			TRegLLegada.PlanetaDesignado=TRegSalida.PlanetaDesignado;
			TRegLLegada.Nombre=TRegSalida.Nombre;
			TRegLLegada.Apellido=TRegSalida.Apellido;
			TRegLLegada.Nacionalidad=TRegSalida.Nacionalidad;
			TRegLLegada.InformeExperimentos= " ";
			TRegLLegada.DanioPsicologico=Vector[CantVect].DanioPsicologico;
			fwrite(&TRegLLegada,TamanioRegSalida,1,fllegada);
			//Sumo Daño psicologico +1 por mision y edad +6 por el duracion de la mision
			Vector[CantVect].DanioPsicologico++;
			Vector[CantVect].Edad=Vector[CantVect].Edad+6;
			fread(&TRegSalida, sizeof(TRegSalida),1,fsalida);

			//Agrego al vector los campos que necesito

			Vector[CantVect].IDindividuo=TRegSalida.IDindividuo;
			Vector[CantVect].DanioPsicologico= 0;
			Vector[CantVect].Edad=TRegSalida.Edad;
		}

	}
	//Contador para la cantidad total de viajes
	for ( int j=1; j<TamanioRegSalida ;j++)
	 {
		if (Vector[j].DanioPsicologico < 3)
		{
			Participan++;
		}
	 }

	cout<<"La cantidad de individuos que participan en misiones son "<<Participan<<endl;
	cout<<"La cantidad de individuos que viajan en misiones son "<<Viajan<<endl;
	cout<<"La cantidad de individuos que llegan de misiones son "<<LLegan<<endl;

}


Would specialized methods for different ConstraintRef types work?


Let there be a space in between `{!!` and `Form` at the beginnning and the end.


There should be a space between `+` and `1`: `col2.set('size', col2.get('size') + 1);`


Where is $upload variable coming from?


ok


ここ挿入したんやな


You may not be doing anything wrong. If the class doesn't have any fields this shortcut might not work, however, you can still create constructors manually.


そや


removing underscore demo stuff as it is not used currently


Couldn't this also be achieved with requesting `twitch.tv/membership` after joining the channel?


Nested the ternary-if inside the 'disableTab'-operator to prevent showing tooltips on match pages, which are located before the assumed replay availability time but DO have a parsed replay. (is this even possible? who knows..)


I updated it at https://github.com/mdarefin/Convert-For-Each-Loop-to-Lambda-Expression-Eclipse-Plugin/commit/b11a5dc2d6c9254222cb3af1d207c93dd41c2b1f also use List with or just pass the default test


He means GoCD


The given date (only year 2014) has to be specified furthermore. I am not sure when replay availability has started.


Updating


TeamCity YaccConstructor :: YC.Core General Build [Build 482](http://teamcity.codebetter.com/viewLog.html?buildId=196634&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) is now running


TeamCity YaccConstructor :: YC.Core General Build [Build 483](http://teamcity.codebetter.com/viewLog.html?buildId=196635&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) is now running


good


TeamCity YaccConstructor :: YC.Core General Build [Build 483](http://teamcity.codebetter.com/viewLog.html?buildId=196635&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) outcome was **FAILURE**
Summary: System.Exception: Error during Nuget update. ..\tools\Build.Tools\NuGet\NuGet.exe install "Z:\Builds\work\4138ce7c6332a78c\FST\FST\FSA.Tests\packages.config" -OutputDirectory "..\FST\FST\packages" -ConfigFile "..\src\NuGet.config"   at FSI_0002.Packa... Build time: 00:01:47


Shouldn't this be in the database? (#89)


Oh, this is bound to go wrong.


I don't think you want to remove the `MANIFEST.MF` file. Why was that removed?


A small point, but the convention is to terminate question keys with a question mark.


@macumber assuming the PVT is on the supply side of AirLoopHVAC you don't need to do anything to the translateAirLoopHVAC function. https://github.com/NREL/OpenStudio/blob/1e16bb8a1232cbce9e34fd940b0c19d3fe1cf206/openstudiocore/src/energyplus/ForwardTranslator/ForwardTranslateAirLoopHVAC.cpp#L432 Just make sure you have a translatePVT function.


If I understand you well, you need more data displayed.
I summarize, just to be sure to add all in one push:
-display power on each phase on cabnavigator
-same for tooltips (in dc and zone views)

Tell me if I forgot something.


I think `\link[=dest]{name}` is mostly commonly used in text, so `[text](::foo)` wouldn't be in backticks. You'd also want `[text](pkg::foo)` to work.

And have you thought at all about S4 classes?  (Links in Rd are hard!)


Checkmarx scan results 2015-08-05T14:19:27Z

High
Medium
Low
Info

New
0
0
0
10

Recurrent
0
1
1
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

5d62a3db10c6cd547782a592687f76602847efb8	@dortal

9876d71c42467754a5e79804d9ae64adabcf8e7c	@dortal

2b3459224d2fc11188d41e5b6f4e39616d66c001	@dortal

7e8b4083ae0a09dfc2efaf431dc69bd637c33c38	@amitchen22

dabbec16f3f2e33ce610861fbbc290f3e48de249	@amitchen22

41be6851f50986ab2d4e3db987c3e42f469b1ada	@dortal

c72690eba7e11492d97f0c7ab1239411e51d1488	@dortal

f4545b47b9f24835acdb9ec38420d31bea849110	@dortal

abb4c31a1e965ba5fa14857ae8935a2269a7a0b6	@dortal

565af35d4610101d7f0d7a9df6153063c12d0d3f	@dortal

bb37aa4384e3495bddd16447868529612cc14d22	@dortal

0467f9dc03aa49abf3742cdea0e547b1c9cf8179	@dortal


Hmm, that means all the changes to refreshRendering() methods in https://github.com/wkeese/delite/commit/14b1d57a22c976f65f600cab4b73c413c79fed47 are invalid now?

Also, what happens if the widget is rerendered because the template is changed?  In that case, is refreshRendering() called again with the widget instance as the argument?


Hm... Just checked with a Ubuntu user colleague. He's using a LTS version 14. His git version is 1.9.1, which seems to be the version at the ["official" repository](http://packages.ubuntu.com/trusty/git) (checked today). That version already has the option "--git-dir" for rev-parse.


After reading this function I've noticed that it's actually just adding half of size to the first column and half to the second one. This can be done in O(1), but since you're adding one by one this function is O(size) instead. Size is at most 12, but still it's harder to understand what this does like this. You can instead just give half to the first column (if defined), and the rest to the second (if defined). That will be much clearer and faster.


You should be able to get the method from `enhancedForStatement`.


TeamCity YaccConstructor :: YC.Core General Build [Build 484](http://teamcity.codebetter.com/viewLog.html?buildId=196636&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) is now running


I would think that we should update the img.src to make it reload the image stream.

There is a lot of complexity here...


I fixed it :-) (together will all @ncdc comments (thanks!))


is the just a formatting?


I was thinking a check would be good if larger than 256 (as it then goes out of bound nvmrq->nr_pages (uint8_t) as well


not a big fan since that would be indicating we are breaking backwards compat.
Any reason for this? Can't you get the tests to pass on 0.8?


sorry for the confusion, as you mention above comment
 "not sure what a menist is, but you should remove the .gitignore file in this directory"
I was trying to mean is MANIFEST.MF file not menist


TeamCity YaccConstructor :: YC.Core General Build [Build 484](http://teamcity.codebetter.com/viewLog.html?buildId=196636&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) outcome was **FAILURE**
Summary: System.Exception: Error during Nuget update. ..\tools\Build.Tools\NuGet\NuGet.exe install "Z:\Builds\work\4138ce7c6332a78c\FST\FST\FSA.Tests\packages.config" -OutputDirectory "..\FST\FST\packages" -ConfigFile "..\src\NuGet.config"   at FSI_0002.Packa... Build time: 00:01:39


Do we need the flags for now?


:ant: if `!(project instanceof ParameterizedJobMixIn.ParameterizedJob)` then you will not be able to trigger that job so this should be treated as an error (`BuildTrigger.NotBuildable` as above).


TeamCity YaccConstructor :: YC.Core General Build [Build 485](http://teamcity.codebetter.com/viewLog.html?buildId=196637&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) is now running


I guess this comment was copied from `hudson.tasks.BuildTrigger` where it made sense.


Yo people! Stop arguing, @jython234 should be a part of the organization but he haven't accept the request yet. xD


Why not combine both regex into one? e.g.

`Regex::new("^'[a-zA-Z_][a-zA-Z0-9_]*'|\"[a-zA-Z_][a-zA-Z0-9_]*\"").unwrap()`


I hope this question should commit to release version,avoid other developer meet this problem


OK. I don't see any such error on travis.


Uh, why the new lines? Else it looks fine


@puredanger does this need to be rolled back as part of ASYNC-103?


Should probably return v, rather than void.


Similarly to `_expandNeighborColsWidth`, this can be made a lot clearer and faster without the loop. This is just removing from the first column `Math.min(col1.get('size') - 1, Math.ceil(size / 2))`, and then trying to remove the remains of size from the second column, while keeping it with size of at least 1.


I recommend that you should make sure it works, we have tested the driver on multiple bayer contourn devices and multiple OS including Windows, Linux and mac. With this change the driver doesn't work properly in our tests.


Also do not have conversations in such a long commit which let make browser lag. xD


These methods will be added to the server again soon.


TeamCity YaccConstructor :: YC.Core General Build [Build 485](http://teamcity.codebetter.com/viewLog.html?buildId=196637&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) outcome was **FAILURE**
Summary: System.Exception: Error during Nuget update. ..\tools\Build.Tools\NuGet\NuGet.exe install "Z:\Builds\work\4138ce7c6332a78c\FST\FST\FSA.Tests\packages.config" -OutputDirectory "..\FST\FST\packages" -ConfigFile "..\src\NuGet.config"   at FSI_0002.Packa... Build time: 00:02:20


AddCalcCoilSurfTempSource (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1262 of 1262 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-69.96%25-green.svg)


Looks good.


Why toggle `enableAddCols` inside this test? If this is testing another scenario for coverage then it should be in a separate test case.


The default looks better than `TextInputLayout`, so I will not merge this branch.


I would like to have an option on the `/autocomplete` request that could provide override snippets.


Maps are even tougher because they can cross gen units.  Also, I think this would break when you have a type that is an alias on a slice or map (it would generate two different conversions) although that's not the end of the world.


maybe rename this to parse_terminal?


You can actually speed up things if you add implementations for IterableIntervals with same `IterationOrder`. You can get rid of the `RandomAccesses` in these cases. Just implement against `RandomAccessibleInterval` and use `Views.iterable(...)`. In case the both `Iterable`s have same `IterationOrder` you can use them, else you can use one of them and set the other one with a `RandomAccess`.


What does this line accomplish?


css选择器优先级金字塔
![image](https://cloud.githubusercontent.com/assets/4270467/9088434/a6721498-3bc2-11e5-9457-b97db305ae84.png)


Then maybe we should just forget the backticks. Stuff within backticks is supposed to be verbatim, anyway. So how about just

* `[](::function)`
* `[](pkg::function)`
* `[name](::function)`
* `[name](pkg::function)`

This will mess up hyperlinks that contain `::`, though.

I wouldn't say I care much about S4 classes, the sooner they die the better. :) But you can always write `\linkS4class{}` by hand. Or we can have a form for S4 classes as well.

Btw. I probably want to `escape()` the function name, I did not do that....


Sounds good. Is it done on the server side?
5.8.2015 5.37 ip. "Jason Imison" <notifications@github.com> kirjoitti:

> I would like to have an option on the /autocomplete request that could
> provide override snippets.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/OmniSharp/omnisharp-emacs/commit/b41f1b7c55191ab4c09197d3d320b960b3ca4b45#commitcomment-12542702>
> .
>


Why do we need to `new` this call? The function name has a capital first letter which is common for classes, which needs to be newed, but the actual function isn't implemented like a constructor?


Agreed.


I would be best to change this test to check if the drag handlers for adding new columns are being correctly added to new rows, instead of just removing it.


Deletion is being done in the wrong way.

Why?
1. How do you know the ID of the ToDo in table is always going to be position + 1?

Imagine this case:
You have 10 items in the table  ID ranging from 1 - 10. You fetch them into a list and they have index ranging from 0 - 9. You delete 8th element i.e. A ToDo Item with ID 9. You close the app and come back. Now you have 9 items in the table with IDs 1,2,3,4,5,6,7,8,10. You fetch them into the list and the list has index ranging from 0-8. Now you again delete the 8th element i.e. A ToDo item with ID 9, voila ? It doesn't exist in the table right.

Instead of passing the position of the ToDo in the list, you must pass the actual ID of the ToDo item, because it extends SugarRecord there should be a `long` field Id available for you.

So what you should do now is
`long id = toDoObject.getId(); \\ Find out the exact method if this doesn't work.
ToDo toDo  = ToDo.findById(ToDo.class, id);
toDo.delete();`


You also need to change the variable DragonetServer class to true and it will send usage info to the new statistic system. ;)


un nouveau mime type ?


Yes, everything is correct if it is possible.

Very thanks for the support.


I feel this command should either send the response to `respond_to` or not do anything when used on the channel.


And make a `release` in GitHub, the format should be like previous ones.


ce check netait pas bon ?


get a spring project


I'm wondering if we are overseeing something. Can't we use the standard `Map` implementations (also for the above mentioned case)? I mean we have a map from `Iterable<Pair<I, Neighborhood>>` to `O` right? Wouldn't it be easier to implement the  `Iterable<Pair<I, Neighborhood>>` instead of reimplementing all the specialized `Map`s?


TeamCity YaccConstructor :: YC.Core General Build [Build 482](http://teamcity.codebetter.com/viewLog.html?buildId=196634&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) outcome was **FAILURE**
Summary: Building ..\src\YC.SDK.sln failed with exitcode 1. Build time: 00:13:34


FixLinksInDocs (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)


:ok: :1234:


I've only used astyle http://astyle.sourceforge.net/


ah je vois que tu la deplacer maintenant...


Shouldn't this be inside the callback function of the call to `_simulateDragToBreakpoint`? It's probably not being asserted at all for this test, since anything after a wait() call is ignored.


You're combining use case (the above paragraph) and the example (below).  There might be many examples.

I would recommend describing the use case with a title (an h3) like "Automatic promotion based on tests", then describing the use case, then list the below as the first example.  You should be writing the examples up in a more descriptive style (including how a user would go about doing it) if you want this to be something that could be used as documentation or something a user could look at.


This will not be done in this class right?


@kfogel Thank you!  I wasn't sure how much information we needed about PapaParse.  Thanks for pointing me to your commit.


We tried to make stop and reload work on the server but couldn't because of bugs with the Kestrel web server. Since switching to stdio, I don't think anybody has tried.

Atom just sends the kill signal to stop it. Maybe this is good enough.


How could I get IMethod from the enhancedForStatement


typo` s/bu/be`


Agreed.


Not arguing just agreeing that we are awesome


Looking much better!

There are a series of Python Enhancement Proposals (PEPs) that encourage certain positive coding practices. One is PEP8, it requires a number of things. One is a maximum line length of 79 characters.

To enforce this line length with PyCharm go to Preferences -> Editor -> Code Style and set "Right Margin (columns):" to 79. This should place a gray line in your editor. Anything over it needs to be wrapped. I'll make some line notes about particular lines I'm concerned about.


After all, intval() is necessary. Otherwise in case of change of a language other then en_US of GUI there will be a problem:
![bar_ok_en_us](https://cloud.githubusercontent.com/assets/10425179/9088537/30e0941c-3b99-11e5-92c8-80cae8490867.png)
![bar_err_uk_ua](https://cloud.githubusercontent.com/assets/10425179/9088545/3786151c-3b99-11e5-835a-d8a9a0740a55.png)


In that scenario, how do you do the equivalent of `\code{\link{foo}}` ?  Or does `[]` imply it?


@TekkitCommando Everyone is awesome here. xD


Does passing `undefined` not work anymore?


Checkmarx scan results 2015-08-05T14:41:53Z

High
Medium
Low
Info

New
0
0
0
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

2df19aa46451ad26f4dc72164d23cf438903565b	@cxehud

ad58b128e2fede39820ae668970bbfab38b56195	@cxehud

6b05867d4fe80a4c3b55cb36e0b6f76abc0a59f9	@cxehud

accf7e51faac6fffc26f426e5afeeaf3ed561731	@cxehud

7917c1e1165bbb3f8b67a7c2943b68d3780e2a79	@cxehud

74262a0e47571dc6ed4aa4c3a03f759fee5bd42c	@cxehud

5b96953244ae55ea53f4983dc1a9fa4d4eb872a6	@cxehud

2166d1ea7164e801468b1b1f20ff0a3ecf4c67dd	@cxehud


I presume that when I migrated from GTK2 it broke. Thanks for the patch.


hi I am really happy for lizmap, I will make a donation. everything works, this is my demo:

http://46.101.131.245/lzmp/lizmap/www/index.php/view/map/?repository=fr&project=fr

if in online editing I can also enter a photo in attribute table field (from smartphones or tablet) would be perfect. it's possible? thanks ciao valerio roma


yeah. It's not marked as an `Op`. Maybe the naming is misleading.


Done


Test comment:)


How is `self.current_connection` different from `self.connection`?


Thanks John!
      I am currently working on the readme now
Bailey

From: jakirkham <notifications@github.com>
Reply-To: bnhwa/mouse_record <reply@reply.github.com>
Date: Wednesday, August 5, 2015 at 10:41 AM
To: bnhwa/mouse_record <mouse_record@noreply.github.com>
Cc: Hwa Bailey <hwab@janelia.hhmi.org>
Subject: Re: [mouse_record] mouse/record.py: Reformated spacing. Added author, creation dates, and comments. Fixed consistency of quotations(e.g., Double quotes for multiple characters and Single quotes for single characters).
 Removed unused lines of code. Ensu...

Looking much better!
There are a series of Python Enhancement Proposals (PEPs) that encourage certain positive coding practices. One is PEP8, it requires a number of things. One is a maximum line length of 79 characters.
To enforce this line length with PyCharm go to Preferences -> Editor -> Code Style and set "Right Margin (columns):" to 79. This should place a gray line in your editor. Anything over it needs to be wrapped. I'll make some line notes about particular lines
 I'm concerned about.
�
Reply to this email directly or
view it on GitHub.


Did you also test them against other existing implementations (e.g. the one in KNIME or IJ1?)


Other devices do it too.


`greater` instead of `up`


Yes, if the brackets are empty, and the link starts with `::` (or is in the `pkg::function` form), then we assume that it is a function and make it `\code{}`.


Может, этому более место в extensions?


Обнови requirements.txt


This class should also handle [usage limit `NOTICES`](https://discuss.dev.twitch.tv/t/whisper-rate-limiting/2836).


`newCol` should go before `row` in alphabetical order.


probably also want to in_progress_proxy_reads.erase(p) ?


`cols` should go before `layoutRow` in alphabetical order.


I think "a" because here the parser has just found a ctor-initializer on the input stream.
The parser has not found a mem-initializer-list yet though the parser expects finding it.


These variables should be listed in alphabetical order.


can we use the work dir insteadof this dir?  this is the same dir where we put the temp console logs before moving it.


Break out the different flows that could exist into separate sub paragraphs or examples


It's better to fix the custom fields to always be either a single item or an array.


Follow nvme-lightnvm.c


Где используется этот relationship?


Great! Feel free to come back to these after you finish up with the README.


он с Flask-ом ставится и уже есть там


The issue comes from US system that uses ',' instead of '.' for float values. I open an issue to fix that later.


should be:

    s->track = sd_bus_track_unref(s->track);

otherwise s->track is not reset if sd_bus_track_new() fails...


ah, i must have missed that option, thank you for the input :+1: - i'll withdraw the pull request.
just an idea: maybe it would be helpfull if shown offline items would be marked somehow, as "preview" to make this more clear..


I understand your point, so let's do like this: I will try to build `spatialite` for RaspberryPi/Linux/MacOS/Android/iOS.
When I succeed, I will start looking into the files which have the install dir into their contents to make it cache friendly. The `*.la` files will be included in this part of the process. If removing them doesn't break anything, then I will have to come up with a way to remove them for all autotools projects, and not only in iOS specific build.

Does that sound like a plan?


@WooD1k  very hard to review code with pod commits, so plz add Pods folder to gitignore


Don't know - the cljs implementation is different and may not suffer from the same issue.


Temos um loading! :+1:


FYI, https://github.com/dblock/slack-ruby-client which is a rewrite of slack-ruby-bot, deals with this in a cleaner, documented and tested way.


I had an older version of selectize.js that didn't include that option. So I updated and tried it. Looks like onChange() still gets triggered. Maybe clearOptions() or addOption() is responsible?  http://plnkr.co/edit/eHALnB?p=preview


This isn't 100% correct - since the synchronized methods lock the entire class, separate locks with explicit sync blocks would be better.


![thumbs up gif](http://static.giantbomb.com/uploads/original/17/179016/2570640-5353427362-thumb.gif)


But for real, this proposal is pretty great.


May not have passed now that you are collecting statuses.


I would love first class celluloid support in https://github.com/dblock/slack-ruby-client (which is a rewrite of slack-ruby-gem) with tests and docs and what not. I opened https://github.com/dblock/slack-ruby-client/issues/5 to track.


Should be:

    s->track = sd_bus_track_unref(s->track);

otherwise s->track will not be reset properly should sd_bus_track_new() fail...


I think you can get it from the binding. I don't remember off the top of my head.


I tried returning just `result` and the tests pass. If this was protecting us from something, can we add a test for it?


Awesome, thanks for the review! :smile:

And very good catch, the technique you're referring to definitely seems to be used more and more lately, especially on news sites, so I also think that's something we should pay attention to. Like you suggested, I think the best way to approach this is to simply compare the URL the current overlay was open from with the current URL, and open a fresh overlay if they're different.

Regarding storing that URL, using `iframe.src` would probably work great, but I'm always wary of using data for something it's not explicitly here for, which could lead to surprises in the future (e.g. if someone was to change the query param's name without knowing other features relied on it). My hunch here would be to store the URL the overlay was open from in a data attribute on the iframe element, without any transformation. If you think that's a good idea, I'll add this to the other [very small tweaks](https://trello.com/c/kwB3sgDP/) we want to add to this feature and ship it at the same time :smile:

(As a side note, and to make sure we're on the same page, I don't think monitoring URL changes while the overlay is open and visible would make sense: reloading a visible overlay with new pre-filled data if a user scrolls down enough on such a site would be unexpected and disruptive. Rather, comparing URLs and loading a fresh overlay would only happen for hidden overlays when users try to open them again.)

Thanks for surfacing this interesting scenario, that'll only make that feature more robust as we use it in more situations! :smile:


FYI this is dealt cleanly in https://github.com/dblock/slack-ruby-client, which is a rewrite of slack-ruby-gem. You can set tokens in the initializer of a client or globally.


I would like to be able to do this via the `/autocomplete` endpoint at some point.


Lambda_MemberArray_Reduce (mbadams5) - Win64-Windows-7-VisualStudio-12: OK (1772 of 1777 tests passed)

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


What means 0L? 0 in a long int format?


"how many of the local weave router's connections"


That would be a change in HaikuPorter then.


@jedesah I wondered if you could help me out a bit. I have this code that is based on 1.3.0:
```scala
  implicit class ProtocolOps(inner: Protocol) {
    def ++(other: Protocol): Protocol = Protocol(
      inner.codecs ++ other.codecs,
      Signatures(inner.signatures.signatures ++ other.signatures.signatures)
    )
  }
```
I wonder how I can adapt that to 1.4.0 now when type params are needed.


Checkmarx scan results 2015-08-05T14:50:13Z

High
Medium
Low
Info

New
0
0
0
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

2df19aa46451ad26f4dc72164d23cf438903565b	@cxehud

ad58b128e2fede39820ae668970bbfab38b56195	@cxehud

6b05867d4fe80a4c3b55cb36e0b6f76abc0a59f9	@cxehud

accf7e51faac6fffc26f426e5afeeaf3ed561731	@cxehud

7917c1e1165bbb3f8b67a7c2943b68d3780e2a79	@cxehud

74262a0e47571dc6ed4aa4c3a03f759fee5bd42c	@cxehud

5b96953244ae55ea53f4983dc1a9fa4d4eb872a6	@cxehud

2166d1ea7164e801468b1b1f20ff0a3ecf4c67dd	@cxehud


What is this?


Is this working for you?


Same here


FYI this is dealt with properly in https://github.com/dblock/slack-ruby-client, which is a rewrite of slack-ruby-gem.


That "is only present if ..." bit is wrong. Needs fixing in latest_release_doc_updates.


Not yet. But I want it to.

I think when this happens, the existing code should just work providing that the completion provider has snippets support.


Looks good.


It will be possible in lizmap 3, please see: https://github.com/3liz/lizmap-web-client/issues/175


Hi all, just wanted to let you guys know I'm happy to join the project. Do
we have some place to communicate? IRC, Gitter, etc.?

On Wednesday, August 5, 2015, The Evil Man <notifications@github.com> wrote:

> @TekkitCommando <https://github.com/TekkitCommando> Everyone is awesome
> here. xD
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/DragonetMC/Dragonet/commit/2cbacb59399cb34959dc9f48f402d5dc04173b0c#commitcomment-12542852>
> .
>

--


It is possible. Values are already there, I just have to print them. It should be done tomorrow.


Alas


Missing new line when you see red symbol. Please add a new one.


"consensus state" - There is no mention of consensus anywhere in the IPAM docs. Should be introduced in the "Initialisation" section.


ok


This is still being worked on server side https://github.com/OmniSharp/omnisharp-roslyn/pull/280


ok. but commit msg should be prefixed with "hostname:" or so...


Ok, in der nächsten verison geändert.
Fast so wie vorgeschlagen. Aber: Statdessene ine vereinheitlichte Farbe, die mit der farbe der MetaNav oben korrospondiert.


The key to getting the summary is to NOT use this:

```
[pull]
    rebase = true
```

but to instead use this.

```
[merge]
    ff = only
```

This will simply cause a pull to fail if a pull --rebase is required.    Good stuff.


No need to mention any of that.


ok.


Could you elaborate, I am not sure why line 217 in comment, in my local machine all the JUnit Test is passing except two for #8


"local weave router's connections"


I still disagree defaulting this to False. By default it should do the right thing for the locale. The fact that it didn't do this before is a bug. People shouldn't need to change their invocation to get that fixed.


or am I getting something wrong?


:+1: :kiss:


TeamCity YaccConstructor :: YC.Core General Build [Build 486](http://teamcity.codebetter.com/viewLog.html?buildId=196638&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) is now running


Unfortunately I don't have the test case anymore which triggered a case clause exception here. So let's stick to this until I encounter the problem again.


hello. here's a comment.


Come mai multiTask invece di task?


Not copyright shadowfacts, copyright the contributors.


Hi,

I can import public image with this script?

With these images I can create my own image server LXD?

Regards,


[Deployment available online](https://s.brightspace.com/apps/hello-world-fra-iframe/dev/8665e6af69d086b0f7427bca97b3c244dcbbac90/index.html)


.lk.k.


You have my permission to change it.


I don't think this will ever be coming back. It seems redundant to me.


make that a link to the relevant section in the docs.


Removing this causes install instructions [here](http://torch.ch/docs/getting-started.html#_) to fail on Mac OS X. Adding this back in before executing install seems to work.


No need to list the columns here.


Add/update "TheOneHe-Man", STEAM_0:0:83360893

C++ hacks, Ampris, huge skid


The changes look reasonable to me. Some property tests would be a good fit for the encoding/decoding I guess.


I uploaded a new gitattributes file.  It should fix the problem if they
update it in their branches.  Don't worry too much though - we can fix
it up during the merge.

On 8/4/2015 6:18 PM, L Melissa Guzman wrote:
>
> I told them to keep working normal though...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SrivastavaLab/bwg_names/commit/6881ce28e0a2c78f90b3e8ebfdb1d6939ac18aa3#commitcomment-12530590>.
>

--
Alathea D. L.
PhD Candidate, Srivastava Lab
www.zoology.ubc.ca/~srivast


這邊就不用 then 唷

let findProduct = await db.Product.findById(product.id);

這樣就可以了，下面部分就往前縮排移出 .then(function(targetProduct){


一樣不用 then 跟上面同樣處理


TBD, this update will not make it to the corresponding workflows until we figure this out.

I'm adding this because Kyle asked for it.  I'm not too concerned about adding this, we'll likely turn this container into a general transfer/utility container that includes some bioinformatics tools, the S3 upload/download tool, and the GNOS tools.   I might move this to a new container named pancancer-tools but that is Tbd


APITestCase.java looks for the getters like this now.

https://github.com/lance/api-gen/commit/75014b52839a0f4be3b1de8639ff39714de0e9b6

This line in EntityAdapter.java is unnecessary now, as the getter and setters are overloaded.

https://github.com/lance/api-gen/blob/master/generator/src/main/java/org/wildfly/apigen/invocation/EntityAdapter.java#L136


Hmm, making set-default's argument optional seems very wrong to me. Can we just support empty string instead?


@ryanoasis yep, iterm.

And yep, i have this font installed. See it, i have some glyphs and my nerdtree and vim tabs but just have some errors using g:airline_left_sep and g:airline_right_sep config

![screen shot 2015-08-05 at 11 56 48 am](https://cloud.githubusercontent.com/assets/154603/9088954/1faa7f8a-3b69-11e5-97ef-4c00e08bb012.png)


testtt


We should allow all file formats


This will need adding to the server at some point. It's still needed for msbuild based projects.


thanks for letting me know, added it back

https://github.com/torch/distro/commit/f0b911c1a61c52b56f0a59c8588dfa160bfb807a


Sure, it can be inverted.  My motivations for not changing actual behaviour were:

1. Lazyness, more tests need to be changed when defaulting to force currency decimal digits
2. Fear of rejection, since I had no idea at all about how bad was _breaking_ backwards compatibility

I'm on it.


but weird as well. looking into it.


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


Add another "Kawaii Hafnium/Aria" alt, STEAM_0:1:124309038


You mistakenly removed this!


Requested feature:
- create an array for each file format


I think we need the code to create new components in the init, but once that has been performed, we should not need to call it again.


Unify this block and the following one to be unique


Add another "Function" alt, STEAM_0:0:5991293


I have a custom version of this in my config that works for DNX projects - https://github.com/nosami/omnisharp-demo/blob/master/config/omnisharp.el#L77-L90


the instruction follow up to this part, after I am getting the superInterface should I use String[] getSuperInterfaceNames() (http://help.eclipse.org/luna/index.jsp?topic=%2Forg.eclipse.jdt.doc.isv%2Freference%2Fapi%2Forg%2Feclipse%2Fjdt%2Fcore%2FIType.html ) and check if it's instance of Collections


Someone tell me how to unsubscribe from notifications every time someone makes a comment on this thread. My email is blowing up with notifications since I first commented here a week ago.


tks jaitor


I get this to compile, suppose that is ok?
```scala
  implicit class ProtocolOps[I <: HList](inner: Protocol[I]) {
    def ++[O <: HList](other: Protocol[O])(implicit p : Prepend[I, O]): Protocol[p.Out] = {
      val c: Codecs[p.Out] = inner.codecs ++ other.codecs
      val s: Signatures    = Signatures(inner.signatures.signatures ++ other.signatures.signatures)
      Protocol(c, s)
    }
  }
```


Can the staging branch be changed to production branch?


Looks good, we also need the RetryJob part. Maybe you can add it using the current logic and then we can open a ticket for refactoring the Retryjob as @belforte suggested


Add/update "Xavier", STEAM_0:1:70694849

C++ hacks, TriggerCheats, BigHack, shitposting, VAC ban


It's not really a compatability break but a bugfix. At least IMO. @erickwilder can you please approve this decision before unneeded work is done here?


Alright! It's not finished yet, I'll try and finish it soon. Need to work
out the concepts too (I pretty much wrote that without going through pseudo
code haha)

On Wed, Aug 5, 2015, 2:40 AM stagas <notifications@github.com> wrote:

> Perhaps use the oscillators from https://github.com/opendsp/osc using import
> { sin, tri, sqr } from 'opendsp/osc/1.1.0';
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/opendsp/auto-panner/commit/a5edfa7ed57fa1c4434f83882d4858b457d13dc5#commitcomment-12536467>
> .
>


Requested feature:
introduce a grunt plugin that allows us to automatically generate a Change log history


A bit more info: CMake was complaining that it couldn't find some file. Notably, why you removed these lines, git still complains about being able to find the correct path for extra/sdl2 in .gitmodules. So my guess is that there's some hidden reference somewhere.


s/bugfix/bugfix IMO/


Cool, man! Glad you're contributing.

We should stick to a lower-cased naming convention for the challenges folders, and all files/folders other than the ones that are required to be UpperCased by our tools (e.g. JavaFiles.java).

Could you rename Age-Message to age-message? Muchas gracias!


There is still a bis_fnc_param here:
https://github.com/A3Wasteland/ArmA3_Wasteland.Altis/blob/master/server/functions/refillPrimaryAmmo.sqf#L22


looks good for me

:+1:


You can't. You need to create a filter and skip inbox with your provider's web ui or api. That or delete your account.

		 ---- On Wed, 05 Aug 2015 08:05:00 -0700  notifications@github.com  wrote ----Someone tell me how to unsubscribe from notifications every time someone makes a comment on this thread. My email is blowing up with notifications since I first commented here a week ago.
—Reply to this email directly or view it on GitHub.


FixLinksInDocs (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (746 of 746 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-7.14%25-green.svg)


I just moved it from localed.c. We can add the return value when there are users for it...


Add/update "Leedle/Ele0t", STEAM_0:1:117651240

C++ hacks, 2 VAC bans, cdriza friends


no


In Java, brackets generally go on the same line as the statement that they follow. So, this opening bracket should be at the end of line 4. Same thing goes for if statements, etc.


Can you please add the related files in the test directory as well to this PR


`it's` cases are consistent within this file.
`'signature valid'` is consistent with how I'm using quotes in marriage abroad [here](https://github.com/alphagov/smart-answers/blob/181e93b34e7be921f49fa0dc9eced3232565ede4/lib/smart_answer_flows/locales/en/marriage-abroad.yml#L184) and at least [one other smart answer](https://github.com/alphagov/smart-answers/blob/181e93b34e7be921f49fa0dc9eced3232565ede4/lib/smart_answer_flows/check-uk-visa/outcome_joining_family_m.govspeak.erb#L16).

Moreover, It looks like Chrome (at least) is rendering both smart and non-smart quotes the same: https://www.dropbox.com/s/5vmf0fjx0d8m4l9/Screenshot%202015-08-05%2016.07.24.png?dl=0


> I will start looking into the files which have the install dir into their contents to make it cache friendly

It is not possible if paths can't be relative. Otherwise the only approach is to add extra patching code to the unpack cached archive step. I don't think this is a nice idea, we should do it only if we definitely have a problems and there are no other choices left :)

> Does that sound like a plan?

Sounds good to me.


:goat:

depending on your ci we may need to tweak things like this
https://github.com/samccone/drool/blob/master/test/tests.js#L5-L17


closes #43


good. improved alot


Indentation should be consistent. So, this closing bracket should be at the same indentation level as the keyword public at the beginning of the file.

```
public class HelloWorld {
    // Stuff goes here
}
```


just compile it and see the errors... you can remove the errors... if find any difficulty then ping me


Also this...

https://github.com/samccone/drool/blob/master/.travis.yml#L5-L11


Make sure you use spaces instead of tabs. Right now, this file has mixed tabs and spaces. That's a no-no! Most people seem to use spaces instead of tabs. You can configure your editor to do this automatically for you whenever you hit the tab key.


Issue #12468 : I found this commit to be associated to a SIGSEGV error in some condition.


Teredo signature should expect at least one teredo style IPv6 address that contains prefix 2001:0: Otherwise packets that come on different port and accidently contain data starting with \x60 would be considered to teredo ipv6 packets - not necessary true.


We should use pull-requests for adding challenges/solutions to the master branch. That way I or someone can review the changes before they go live. Also, ask me about the git rebase command next time we talk. It allows you to take a bunch of local commits and turn them into a single commit so that the repo doesn't get cluttered with lots of little commits.


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 263](https://teamcity.labkey.org/viewLog.html?buildId=264707&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:11:50
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSMeasureStoreTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSSecurityTest: <no details avaliable>

```


wjh++

Tabs for indentation, spaces for alignment is indeed #1 proper style. Anything else is stupid, unless code is uglified/minifed.


sadasd


Do we need "basename" here?


I have had a chat with the guys, and I am not sure we are that happy with this implementation.  We would prefer that we use named arguments for all the options, so that you can do any of the following:
```
AblyRest(key="api:key")
AblyRest(key="api:key", host="custom.host", port=8080)
AblyRest(token="token.string")
AblyRest("api:key")
AblyRest("tokenstring")
```

I do not understand why options is a separate argument as opposed to just providing the options as arguments like all other client libraries.

Also, we could detect if a token string is passed in instead of an API key for the key argument, and treat it as a token, see https://github.com/ably/ably-ruby/blob/master/lib/ably/rest/client.rb#L96-L102

WDYT?


Yep, sounds better to me.


Do we need basename here


hello


no it is just a pointer to basename. Code cuts '/basename' to get path


I agree. Need execute single time.


why not in seperate commits?
I cannot check it like this....
please fix it asap
:cactus:

:P :P :P


Added the ones with VAC bans or that are already in the database, couldn't add a few, no proof.


TeamCity YaccConstructor :: YC.Core General Build [Build 486](http://teamcity.codebetter.com/viewLog.html?buildId=196638&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 236 Build time: 00:17:18


WTF are u doing??


Could try Denis' suggestions: https://github.com/ICGC-TCGA-PanCancer/dkfz_dockered_workflows/blob/develop/Dockerfile#L113

Need to try this out and "-u" next time I release the DKFZ/EMBL workflow


This doesn't compile, you're redeclaring CC_*


I would like to run from any place as ~/prj/pmix/pmix/build/test/pmix_test for example (not from test folder)


Is this needed?


@krukow

This is not very well behaved.

Pinch to zoom out is causing the view to grow rather quickly.  This is the growth rate of a square box.

* 44,44
* 172,172
* 697,697

Pinch to zoom in collapses the box completely, no matter what the starting dimensions are.

I think the API needs to change to allow a finer degree of control.

Maybe a percent could be passed in?

```
pinchOffset(direction, starting_from, percent_to_pinch, duration)
```


def login(self, login_url=None, login_data=None):
Default parameters of None are not recommended if they are required. Better to make function require those args. The dictionary in 'settings.py' will raise a KeyError Exception if they do not exist.


What's intended to be handled here? Would it be possible that the item were not a Job/AbstractProject?


Had to use fs.readFileSync() because require() would cache the file and it would never be reloaded into memory.  This reads the assets file automatically, when Assets.json is changed.  It is not triggered when any Less or JS files are updated.


This pass is destructive; it should not work correctly inside of this loop.


I think we should not in_progress_proxy_reads.erase(p) , because we also need in_progress_proxy_reads in finish_proxy_read if proxy read return first.


New readme file for version 2.0


testing


Nice update. Thanks!


testing 2


- How yo install thrift
-- install boost library
http://www.boost.org/users/download/
-- install thrift and build
https://thrift.apache.org/docs/install/os_x


Let's clear out all comments.


Did you test this? doesn't work for me... :(


very nice!


The UIA pinch can also take a `rect` for the to and from - not just points.


These don't actually return a router in Finatra.


nice


Please verify translation to Swedish


Should this have been `$?` I haven't seen `$!` before but I'm not in this particular case that often.


It'd be good to notify for this and I agree.


5058-fix-readvars-cli (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1789 of 1789 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)


Hi,
I'm using c3 directive angular in my project and gave a try to upgrade c3.js lib from 0.4.10 to 0.4.11-rc1. Well, I've noticed something pretty weird: from now on I can always see a subchart axis X displayed on my chart even if no subchart-show attribute was enabled.
Comparing 0.4.11-rc1 to the prevoiuos one I've noticed this 'pull request' and I guess would be linked to the above issue.

Wouldnt be better to set newest subchart_axis_x_show to false by default?

kind regards


do not. Remember that removed one. Will resubmit.


@arthurkay @Faiz7412 Or just click the "Unsubscribe" button below the comment box


In fact, I don't see how to implement equals and hashcode -> paar type and fileAttribute are not unique


FixLinksInDocs (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1783 of 1784 tests passed)

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


You are missing va_end() here. You have to call that before leaving the function.


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


Oh, thanks for your note! I didn't know this before. Will fix it in a short time.


> Wait, aren't you removing support for @before / @after here ? (not that I'll be the one to complain...)

It's just using an alternative API that I had added for `registerWebInterface` to be able to support member functions as attribute modifiers: `computeAttributedParameterCtx` and `evaluateOutputModifiers` (the latter is indeed missing from this commit, but included in the current one)

The latest version now also puts back everything back to CT that can be CT (AFAICS).


Looks OK otherwise.


Most of the `realms` section in the krb5.conf looks unnecessary, since we're not going to be contacting or interacting with them in any way.  The `domain_realm` and `login` sections aren't things we're exercising, either, so they can probably be dropped.
Can we depend on the KDC being okay with listening on the localhost address?  Older versions at least used to take pains to avoid doing that.
Are the changes to docker_utils.go needed for the test, or are they cleanups that upstream might ding for?


Hmm, what's the precise rationale here?

(looks ok generally...)


this will force `HazelcastParallelClassRunner` to use just a single thread.


And all the other i/o pins?


When I run the script I get the following error :

root@imagelxc:/home/usersio# ./scriptlxc.py import ubuntu --alias ubuntu
Downloading the GPG key for https://cloud-images.ubuntu.com
error: Failed to retrieve the GPG key
usage: scriptlxc.py [-h] {import} ...

Regards,


dunno xD i can just speak over my own observations ;-)
btw: funny thing: yesterday i was playing with Mali_L2_max_reads values.. and l2_register_size...
I've increased the register from 48 bytes @ 128 ... and the mali_L2_max_reads from 28 bytes @ 64 ... and the UI was lagging like hell ^^ It was lagging all the time like using APE_OPP50 @ 200Mhz GPU and CPU 200Mhz... horrible :D i thought it should run better ... but ...


this is for tests in the `ParallelTest` category


Ok, cool. Let me know what you find out about that.


Spurious whitepsace.


this is for the other (quick) tests


Missing OOM check.


Ugh yeah that's a typo. Fixing now.


its working!


var othersname = jQuery("#postID_<?php echo $last_post_id;?> option:selected").text();
この書き方だと、$last_post_idはひとつしか設定できないです。
なので、一つだけしか上手く動かなくなってます。

$last_post_id=くださいされた商品ID
なので、くださいされた商品ごとに<?php echo $last_post_id;?>の部分を変えてください。
具体的には、関数の引数として$last_post_idをとってきてあげれば良いと思います。


this is perhaps a bit controversial. There is an overlap with https://github.com/hazelcast/hazelcast/pull/5859 so I can revert it if needed. I shouldn't have a significant impact on tests duration.


I'm still figuring out how to use GitHub. It's been awhile. I'm not sure how to re-name that, you'd have to walk me through that. We can talk about it later if you like.


I could fortunately restrain myself from modifying anything more ;) (I didn't want to cause merge conflicts for the open PRs and have in fact reverted some of my changes). But it will be interesting to see how clean we can get `RestInterfaceClient` with some removal of string mixins and by using `RestInterface`...


This is really an argument error, not an Ably error code, so I have done the following in Ruby - https://github.com/ably/ably-ruby/blob/master/lib/ably/auth.rb#L65.


This can be a problem, if stack delete fails, due to some reason and service vm gets deleted then deleting afterward those resources will be a issue. I have seen this sometime with firewall. We will have stale resources in db and will need manual cleanup. Can we avoid this by raising exception?


@RevanProdigalKnight Hahaha Why didn't I see that button before! XD


Can you use `zip` over here?

That will help you get rid of `p_count` in `display` function.


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


ボタンの場所はここでは、うまく動かないです。
ソースを良く見て、ループがどのようになっているのかを調べてみてください。
ヒントは
<?php
echo "</select>";
echo_map_section($last_post_id);
echo_message_section($last_post_id);
?>
の部分です。これが２回ほど出てきている意味を考えてみてください。


I think the last line should be `options.key` or `options.token`.  We don't use `auth_token` anymore and a key value without a key secret is not going to work.


Is that endpoint implemented?


Naming is `key_name`, `key_secret`


the folks with split /usr, probably want /lib/systemd here too. But it should be #ifdeffed away unless we have split /usr.


Can you get rid of this if statement?
May Be,Some tricy logic!!!!
hint: (list or tupple)


should be `key_name` and `key_secret`, but should also support simply `key` which is the documented way to use this method.


So you have a feedback loop type thing going on here.

Try an event driven approach so you don't keep running pieces of code that don't do anything.


Thanks


We need to check both /usr/lib and /lib here i guess, to make the split usr people happy... the latter should be ifdeffed out however, unless split usr is on.


Reviewer note - this was a bug in the existing all-tests.sh, the return code of the run_quality command was not taken into consideration for the exit code of the script.


yeah. But right now it only works for DNX projects and the dnx xunit runner command. It's what I used to test omnisharp-roslyn with :)


Nice :+1:


I'll stop adding comments about this naming, should be `key_name` and `key_secret` everywhere, however we expect people to use `key` 99% of the time.


NIce !!!!
Good Progress.
DRY successful.


@jython234 I will add you to the organisation.

My twitter: @TheDiamomdYT
Not sure about anyone else

We could do with a gitter chat @DefinitlyEvil


going back to old json, hopefully it works


Yes, if dokuwiki.org is updated then the fix is shown as well.


@DefinitlyEvil Will do


gah! Already fixed, but I forgot to commit!  committed and pushed.


Humpf. I don't like this I must say, we should return this as proper error code, not as "false". Make check_whitelist_for_symlink() return an int, where errors are returned negative, false as 0, and true as > 0... This is how we do that elsewhere.

It might make sense to degrade errors to bools but that should do the caller, never the callee...


Looks good. Should the older pull request be abandoned/closed?


See the issues should be for bugs, not things like this. (Btw I'm already
part of the project).

On Wednesday, August 5, 2015, RandomAltThing <notifications@github.com>
wrote:

> @jython234 <https://github.com/jython234> I will add you to the
> organisation.
>
> My twitter: @TheDiamomdYT
> Not sure about anyone else
>
> We could do with a gitter chat @DefinitlyEvil
> <https://github.com/DefinitlyEvil>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/DragonetMC/Dragonet/commit/2cbacb59399cb34959dc9f48f402d5dc04173b0c#commitcomment-12544452>
> .
>

--


Vikash, Raising an exception will have a stale VM(or ports) in addition to
stale stack. The service chain instance is already cleaned up from db in
precommit hook.
On 05-Aug-2015 8:56 pm, "Vikash082" <notifications@github.com> wrote:

> This can be a problem, if stack delete fails, due to some reason and
> service vm gets deleted then deleting afterward those resources will be a
> issue. I have seen this sometime with firewall. We will have stale
> resources in db and will need manual cleanup. Can we avoid this by raising
> exception?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/oneconvergence/group-based-policy/commit/d9d0c0337590843c55e5f659a6613520e0ede8c7#commitcomment-12544318>
> .
>


Нормально.


Same here...

Also, this needs to be a search path loop I figure... see above...


I agree the two are distinct. However, AssemblyReferenceResolver has a single MetadataFileReferenceResolver, so if we introduce two different resolvers, GacFileResolver and NuGetPackageResolver, then we'll need a third CompositeResolver to combine the two instances for AssemblyReferenceResolver. Also AssemblyReferenceResolver.PathResolver.BaseDirectory and AssemblyReferenceResolver.PathResolver.SearchPaths are exposed as BaseDirectory and SearchPath properties from ScriptOptions. That means CompositeResolver needs to expose those two properties. Which individual resolver does CompositeResolver use for those property values? We'd need to assume BaseDirectory and SearchPaths are the same for the two resolvers. Furthermore there was only one use of GacFileResolver. In short, it seemed simpler to have one combined class (perhaps renamed GacFileAndNuGetPackageResolver?) that handled both even though the resolver behaviors are distinct.


To parse out the instance, consider using unit_name_to_instance() and related calls.


please use FOREACH_DIRENT...


Please use fstatat(fileno(d), &st) here...


No this is the same logic we use in the export payments build processor, logic is correct.


Sorry for this massive commit. It was necessary on my end to remove the space at the beginning of a folder. OneDrive doesn't want to sync a folder with a space... :/


It could be added, but is unnecessary as it's used by the primary exporter base class...


Yeah, but if we can use the way, we did for heat create ie overriding complete def., think we can resolve this, if time permits. Your call. :)


Agreed. Will modify.


`/*/_vti_cnf/` should match all subdirectories of the name `_vti_cnf`.


Creo que hizo falta el .show() en el AppMsg.


Thank you, I forgot you can have multiple values selected.


I think you're right. The private options are opaque to us, so we _should_ be using a void* to store them. Let me make the change.


@slarson Alerting you to [this](https://github.com/openworm/PyOpenWorm/issues/218). Should we deal with it in PyOpenWorm or this repo?


This method was checking the target file exists in all cases except for the last case. And all callers were checking the return value.


@jedesah It is a bit cumbersome that one needs to do all that typing wrt: `Field.strict[Foo]("in"), Field.strict[Foo]("out")`, especially if one is always going to type "in" and "out". Could the boilerplate perhaps be reduced by a dsl or some default standard values for `in` and `out`?


@arw180 , it seems that some endpoints return the full url for an image (e.g. /storefront) while others return the image id (/listing). Is there a reason? Maybe it would be better to just pick one or the other and stick with it.


Good point.  Hopefully this will split will be be short-lived either way.  In similar situations I've also done:

```python
return filter(None, [
    item1,
    item2,
    item3 if special_flag else None,
    item4 if special_flag2 else None,
])
```
That way you don't have to repeat.


・ 最後の値を超えてしまう
・ 最後の値を超えると何も表示できない


Describe this as "areas for improvement"


Renamed.


I can not write non ascii characters when commiting to SVN because of this "hard coded" --encoding utf8 thing. Please make it customizable.

Commit message was: "Arruma essa bagaça!"

svn: E000022: Commit failed (details follow):
svn: E000022: Error normalizing log message to internal format
svn: E000022: Non-ASCII character (code 195) detected, and unable to convert to/from UTF-8
The command "C:\dev\bin\svn.exe" terminated with exit code 1.


@kholk
This breaks hotplug on non-hmp systems.
Will try to find a fix.


Just FYI
We found that this change, calling `removeChild` and `appendChild` in this context on the elements resulted in an `Illegal Invocation` script error on IE 9/10/11 and Android Browser.
Watch out! :8ball:


Hi @lweasel --- just FYI, the explicit bias correction flag in Salmon is *experimental* and likely soon to be deprecated (as soon as the last of the remaining biases it addresses --- sequence-specific effects --- is incorporated directly into the quantification model).  As such, I often recommend to users to *not* use bias correction with Salmon by default, but only if they have reason to believe significant biases exist that are not accounted for by the default model.  All this is to say that it may be best to *not* enable bias correction by default, or, at least, to report both results separately (*note*: the tiny marginal cost of Salmon's bias correction is designed so that it's trivial to produce & analyze both results).


According to the specification they all should end with a trailing slash.


Lambda_MemberArray_Reduce (mbadams5) - i386-Windows-7-VisualStudio-12: OK (1772 of 1777 tests passed)

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


Wouldn't `params=[3]` be better as the number of whispers before Twitch `NOTICE`s us is limited?


Hi @jkotas I have done the suggested changes and updated the PR with a squashed commit


@173210 cand see X is incremented in drawcharacter and halw-width characters workaround. Also you replaced pointer access with 3 dimentional array access, which may be slowly. Pointer addressing was optimized so each iteration it only have one increment. You used array addressing with loop iterator used, that may not be optimized by compiler.


Just a tiny thing: I think it would be good to clear() directly in createGraph(), I don't think we ever want to create without clearing before.


Yes, `settings.displayField` will never be undefined, but if I'm not understanding the code wrong, `result` is the search result. So, if I set `settings.displayField = 'some_non_existent_field'` and the `some_non_existent_field` field is not in the result, there will be an error. And not only that, if I set `settings.displayField` to something else than a string (array, object), more errors. So you have to check that `settings.displayField` is a string **and** that `result[settings.displayField]` exists.

Sorry if I'm not right, just made a quick look at the code.


não prefere marcar com `@unittest.skip("messagepack not implemented")` ao invés de comentar não?


Sorry that was the linter confusing me after the actual cap.  Fixing now


Why not cache the gvrContext at creation time; then you don't have to pass the extra parameter..


"Prevent an event-handler function **from** being called via private message"


Forgot to do this `$this->path = null;` ? more details in https://github.com/symfony/symfony-docs/issues/5448


Also, will this.setParameterValue "unselect" previously selected values? Will selecting each new value unselect the ones i previously set inside of that loop


I just realized there are more valid values than that.  There are like 5:  See https://jira.swisstopo.ch/browse/GEOCAT_SB-207 for the list


Looks good just the one comment


Well, if the name changes, then we can also change the entry... I think I would prefer that.


Specifying a dependency as optional does not literally make it so. This way, the npm install routine won't fail if a particular module does not install correctly.


can't we do the is_symlink() and readlink_and_canonicalize() in one? i mean, the readlink_and_canonicalize will fail with a clean error code anyway if the thing is not a symlink.


These should be something like kXDGSpecConfigHome. (`k` then capital, then the rest.)

Also when enum is bound to the struct I use something like

```C
typedef enum {
  kMPNil,
  kMPBoolean,
  kMPInteger,
  kMPFloat,
  kMPString,
  kMPBinary,
  kMPArray,
  kMPMap,
  kMPExt,
} MessagePackType;
static const char *const msgpack_type_names[] = {
  [kMPNil] = "nil",
  [kMPBoolean] = "boolean",
  [kMPInteger] = "integer",
  [kMPFloat] = "float",
  [kMPString] = "string",
  [kMPBinary] = "binary",
  [kMPArray] = "array",
  [kMPMap] = "map",
  [kMPExt] = "ext",
};
```
(Note: you don’t have to keep the order in the struct synchronized with enum, only order in enum matters.)


Heu, in fact, because it need the current metric


why einval here?


Python has [enums](https://docs.python.org/3.4/library/enum.html).


+	      if (in() < 8 || in() < (8 + (in[5] & 0x0F)))  // Telegram complete

The first clause of this if() is redundant.


Please parse/join unit names always wiith the helpers from unit-name.h


TeamCity xBIM :: XbimExchange :: Build [Build 377](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=2205&buildTypeId=XBIM_XbimExchange_Build) is now running


Looks ok otherwise.


It could be a future optimization: create a different, lighter and thinner, future in case the call is made on the GL thread. After all if something is on the GL thread then every effort needs to be exercised to be efficient.


We need to separate GAC resolution logic from NuGet resolution. GAC is only available on Desktop FX. There is no GAC on DNX. We should have a DesktopMetadataReferenceResolver that combines relative path resolution (which needs SearchPaths and BaseDirectory), GAC resolution, and NuGet resolution. The latter two don't need SearchPaths and BaseDirectory.


+ Add the actual constants for the pins
+ Setup the pins in the `setup()`
+ make functions for actuating the steering, brake, and accel that you just need to pass values too. Something like
```
if (brakeFlag) {
    moveBrake(brakeGoal);
}
```


Ok.


nice job! :+1:


Who got Tumblr all in my Github?!


>Throwing equipment doesn't get rid of it, you can keep throwing it after it's down

Got a semi-fix for this, stops you from duping but trying to throw after you've already thrown will still destroy the first thrown object. Kind of a neat feature since doing this with tripmines explodes the tripmine you threw, working as a sort of detonator for the mine, but we should probably try and get it to work like H3.

>Walking over equipment while you're holding a different type of equipment will crash
Crashes at 0x5399BC, seems to be something related to the HUD, crashes in a func related to c_hud_camera_view. Maybe trying to print the "press [] to swap [] for []" text but it fails.

Also this crash happens if you throw equipment and then try picking up any other equipment.

>Dying while holding equipment will crash
Unsure of the cause of this, probably something related to dropping your items when you die.


Please add a comment that the method returns null if the file doesn't exist.


TeamCity xBIM :: XbimExchange :: Build [Build 377](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=2205&buildTypeId=XBIM_XbimExchange_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:02:51


This doesn't look right.


probably the refered parameter should be "--encoding UTF-8", not "--encoding utf8" (missing dash)


probably the referred parameter should be "--encoding UTF-8", not "--encoding utf8" (missing dash)


Why was this removed? The cask works, I don't see there being a problem.


86139842-AddAutoSizing (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1764 of 1789 tests passed)

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


@jedesah Would it be an idea to allow for this?
```scala
def specify1[A, O](name: String)(implicit SA: Selector[H, A], SO: Selector[H, O]): Protocol[H] =
  specify1(name, Field.strict[A]("in"), Type[O]("out"))
```
or similar


It has now been removed: https://github.com/django/django/commit/61ad1ea92b1f4df992b0ef1dcc7c781da2413815


u seem to have a typo
صدقت بدل صدقة


x discription
o description


### Such a nice serendipity day Joe :)

The [The Autodidacts blog](https://via.hypothes.is/http://www.autodidacts.io/openbci-brain-basics-neurons-structure-and-biology/) It is developed by using a **Responsive, Minimal, Free, and Open Source**. I think it will looks awesome as "The peeragogy Handbook" theme.

Resources we need:
*- [mnml, a ghost blog theme](https://via.hypothes.is/http://mnml.herokuapp.com).
*- [gitHub repository](https://via.hypothes.is/https://github.com/curiositry/mnml-ghost-theme)

Then I also find out that is support a powerful publishing platform [Ghost](https://via.hypothes.is/https://ghost.org/)  with hipothes.is  easy integration. It's allow team members to get involved writing about the areas they specialise in.
*-[Ghost source code github](https://github.com/tryghost/Ghost)

The project is maintained by a non-profit organisation called the Ghost Foundation, along with an amazing group of independent contributors.

So a new peeragogigal task for the **peeragogy design team** it could be *rebuild a demo version* of peeragogy.io as to present whit:

**mnml theme + hypothes.is + ghost editor integration**
I think it could suit very well the community this combination and help go a step foreword. Can we learn together on it?

**BTW** Today my candidacy for a job position into a peer-education Wikimedia Italy project has been rejected mainly because I do not have any school degree. The funny thing is that when I told here that I'm learning how to manage project work-flow usiung repository, she asked me what a repository actually is.
Fortunately, really great organizations & companies don’t hire skilled people and motivate them, they hire already motivated people and they training & inspire them.


التعلم بدل الالتعلم


Yes, `settings.displayField` will never be undefined, but if I'm not understanding the code wrong, `result` is the search result. So, if I set `settings.displayField = 'some_non_existent_field'` and the `some_non_existent_field` field is not in the result, there will be an error. And not only that, if I set `settings.displayField` to something else than a string (array, object), more errors. So you have to check that `settings.displayField` is a string **and** that `result[settings.displayField]` exists.

Sorry if I'm not right, just made a quick look at the code.


Also on error it should print a error message. `os_mkdir_recurse` is intentionally not doing this.


Thats not actually true - the established and unestablished counts are for all peers. The original individual 'Unestablished' line (before you suggested coalescing everything into one property) did this as a way to fix #908 - if you make it local connections only, then you'd have to run `weave status` on all peers to notice if anything was wrong...


tetdgf


very nice


The `duration` defines how long the session will live, unless you logout – thought the idea was to make this as long as the persona session - but that might not be the best idea.

`activeDuration` allows the session to be lengthened, by the `activeDuration` if they are active within that period of session expiry.

So, to build on your example - you probably would not use a hard coded `duration` but probably use the number of milliseconds to say 0:00 on Monday. That way the user would have to login the first time they used the system each week. This though would cause a problem for anybody using the system overnight. To resolve that issue we would set an `activeDuration`, say of 3 hours so that if the application was accessed after 21:00 on Sunday the session would get lengthened by 3 hours. If then the application was accessed between midnight and 3:00 the session would get lengthened again, and so on...

The original `duration` and `activeDuration`, of 1 day and 5 minutes, are both the defaults.


How? Where?


Note: the package version is updated every time js or scss is updated. This will bust cache. Do not touch the `version.php` file - if you need to manually do this, you can do so by running `gulp version-update`


Shouldn't we mention this change in the release notes? nixos/doc/manual/release-notes/rl-unstable.xml


thanks for fixing this; much appreciated


@tymulholland


Sounds like you're either behind a strict firewall or http proxy.


Done

I've added a test where the index exists to ensure its caught.


Done.


One additional question. Am I not supposed to use `Signatures` directly? In 1.3 I do things like this:
```scala
object contracts {

  val commandsProtocol = Protocol(codecs, signatures)

  def signatures = Signatures.empty
    .specify1[s.SteadyEarningCmd, Result]("steadyEarnings")
    .specify1[e.EarningCmd, Result]("earnings")
    .specify1[c.ConfirmationCmd, Result]("confirmation")
    .specify1[g.GenericCmd, Result]("generic")
    .specify1[cc.ClaimControlCmd, Result]("claimControl")
    .specify1[pp.PlanProgressionCmd, Result]("planProgression")

  def codecs =
    Codecs.empty.
      codec[s.SteadyEarningCmd].
      codec[s.Register].
      codec[s.ChangeInterval].
      codec[s.ChangeSalary].
      codec[s.Revoke].
      codec[e.EarningCmd].
      codec[e.Register].
      codec[e.Replace].
      codec[e.Revoke].
      codec[c.ConfirmationCmd].
      codec[c.Confirm].
      codec[c.OverruleConfirmation].
      codec[g.GenericCmd].
      codec[g.Register].
      codec[g.Revoke].
      codec[cc.ClaimControlCmd].
      codec[cc.OpenSession].
      codec[cc.ReopenSession].
      codec[cc.CloseSession].
      codec[pp.PlanProgressionCmd].
      codec[pp.OpenPlanProgression].
      codec[pp.ChangeScope].
      codec[pp.ClosePlanProgression]

}
```
That falls a bit in the water with 1.4 :)


The cask *does not work* if we cannot download the most recent version. If you wish to use cask to install an older version of Audacity, please use the `Caskroom/versions` tap.


s/verifcationDeadline/verificationDeadline


:cookie: :bread: :doughnut: Tu paies le petit-déjeuner ?


Drop `= `.


you're right, never mind


Drop `z`.


Great!


This statement is necessary for second time too (in case when you set only the file field). This is the reason of the issue https://github.com/symfony/symfony-docs/issues/5448


@danbernier  I want to combine both Contribute buttons into a single button, with clickable dollar and bitcoin icons.  Sadly, my Gumby-fu is lacking. Also, RIP Gumby (apparently).


Toca hacer entonces una lista de los posibles tamaños de los campos


Lines 22-25 here aren't actually arguments. Maybe move them to a `Note:`  or `Notes:` section with some preface like "Contents of `argv` in order (excluding command name)". Also, drop `z` as it is computed from the arguments. Maybe that should be a comment in the code. Also, drop all the ` = ...` cruft.


FixLinksInDocs (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1266 of 1266 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-69.98%25-green.svg)


Hopefully it did! :)


Yeah, I tried using an in-line IF in the ng-class, but couldn't get it to work, so went with controller.
Thanks for fixing ;-)


melhor msm.


Drop paren to next line.