import java.util.ArrayList;

public class PathFinder {

	private ArrayList<GridMember> neighbors, visited;

	private int xStart, yStart;
	private int xTarget, yTarget;
	private GridMember previousNode;
	private boolean goalReached;
	private boolean isVisited = false;

	public static final int GRID_MEMBER_SIZE = 32;

	public PathFinder(int someX, int someY, int targetX, int targetY) {
		goalReached = false;
		xStart = someX / GRID_MEMBER_SIZE;
		yStart = someY / GRID_MEMBER_SIZE;

		xTarget = targetX / GRID_MEMBER_SIZE;
		yTarget = targetY / GRID_MEMBER_SIZE;

		previousNode = null;
		neighbors = new ArrayList<GridMember>();
		visited = new ArrayList<GridMember>();

		// Add the Spawned node
		neighbors.add(new GridMember(xStart, yStart, xTarget, yTarget,
				previousNode));

		findPath();

	}

	public void findPath() {

		ArrayList<GridMember> neighborsOfCurrentNode;

		while (!neighbors.isEmpty() && !goalReached) {

			GridMember currentNode = neighbors.remove(0);
			visited.add(currentNode);
			neighborsOfCurrentNode = currentNode.findNeighbors();

			while (!neighborsOfCurrentNode.isEmpty()) {

				// Head of the list contains the lowest cost Grid Member
				GridMember newNeighborOfGridMember = neighborsOfCurrentNode
						.get(0);

				// If we have reached goal we indicate that we have reached
				// destination
				// and exit
				if (newNeighborOfGridMember.getCostToGoal() == 0) {
					System.out.println("Goal Reached");

					goalReached = true;
				}

				for (int i = 0; i < visited.size(); i++) {
					int x, y;

					GridMember visitedObject = visited.get(i);
					x = visitedObject.getxIndex();
					y = visitedObject.getyIndex();

					if (x == newNeighborOfGridMember.getxIndex()
							&& y == newNeighborOfGridMember.getyIndex()) {
						// do not visit again and do not check neighbors
						isVisited = true;

					}

				}// for

				if (!isVisited) {
					// check neighbors

					for (int i = 0; i < neighbors.size(); i++) {

						int x, y;
						int costFromStart;

						GridMember neighborObject = neighbors.get(i);

						x = neighborObject.getxIndex();
						y = neighborObject.getyIndex();
						costFromStart = neighborObject.getCostFromStart();

						if (x == newNeighborOfGridMember.getxIndex()
								&& y == newNeighborOfGridMember.getyIndex()) {

							if(newNeighborOfGridMember.getCostFromStart()< costFromStart){
								neighborObject.setCostFromStart(newNeighborOfGridMember.getCostFromStart());
							}

						}

					}
				}

			}

		}// while neighbor

	}

}


Great, thanks! You´ll have to PR to master. This one might even work from helix to master but you better create a master-based branch and apply your change. If you´re running into trouble just ping me here or on IRC.


Choalover Universe is in development hell.


А префикс вообще нужен? В хроме естьв ерсия без него?


Taking a look at this now. Will be pulling locally and running tests. If you have updated tests to cover the fixed use cases, that would be helpful.


了解


please @drgarcia1986, enable travis-ci


あと、
「そのユーザーは存在するが、プロジェクトが1つもない」のか
「そもそもそんなユーザー名が存在しない」のか、
エラーハンドリングが不十分


The change on this line isn't mentioned in this commit's commit log message.

Revert the change or explain it in the log?


I don't think I understand the change on this line - why make it?


El ultimo commit deberia fixear esto que decis. Probalo y contame si funciona


@tobscure Meh, I was missing that `ember` alias when trying to get it working on Homestead...


82420068_GridViewMultiEdit (evanweaver) - Win64-Windows-7-VisualStudio-12: Build Failed

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


Let's pull this commit completely out of pull request #64 and into a pull request of its own. This commit is about wanting to use 2.7-level features of unittest in a 2.6-compatible project. It's only related to Python 3 by the weakest of causal chains.


@rcpeters

Please do not merge until I let you know


can this be static readonly?


@rhyolight why does the record need to be a list? Records of type tuple work as well. Not allowing tuples here is the current break in NAB.


OK, that seems like it's a relatively unlikely case in practice. But still worth putting out another release?


This still seems wordy - I would remove the first two sentences in this para, and just introduce the term "Application TURN Server".


Deleted


[![Double Gourd-shaped Ewer](http://api.artsmia.org/images/10020/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=10020)


[![Page from the Koran](http://api.artsmia.org/images/18096/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=18096)


[![The Bull Frog](http://api.artsmia.org/images/9809/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=9809)


[![Black Squirrel](http://api.artsmia.org/images/9807/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=9807)


[![Snowy Owl](http://api.artsmia.org/images/75136/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=75136)


[![Snowy Owl.  Plate XXIII.](http://api.artsmia.org/images/76406/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=76406)


[![Great-footed Hawk (Peregrine Falcon), Plate 16  from The Birds of America](http://api.artsmia.org/images/119937/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=119937)


[![Winter Hawk, Plate 71 from The Birds of America](http://api.artsmia.org/images/119938/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=119938)


[![Mercury and Minerva Preparing Perseus for Battle](http://api.artsmia.org/images/118952/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=118952)


This might need a little more explanation.


Can you please update the version date too?


very good -- yes, i'm a bit familiar with "case" and will start to use these.


Should be: `Backend.parse_string(data, scope, [extra_data], context)` I think.


No, im compile this with normal linaro


This line seems to behave differently on Ruby 2.0 than on more recent Rubies.  :(  It's causing the Travis failure.

Looking at it more.


yes, I added more
```
$('#form-add-message-schedule').yiiActiveForm('add', {id: 'messageschedule-event', input: '#which-event', container: 'field-messageschedule-event', error: '.help-block.help-block-error'});
```
but it seems it's ok when we remove but not ok when we add it back.. I always {... status: 1 } in attribute object. Although I set it is 0 when I use add method


Because it's a small(ish) prime. I'll have a look at how hash values are computed elsewhere and perhaps change it. There's probably a utils.hash function or something I could use.


Late! Tsk, tsk. :P


same question,

i tried to request a new password, it says "status:fail, reason:blocked"

does that mean i cant use WA with this number anymore?


Создание и пересоздание Ai - это некоторая ответственность, которую можно вытащить в отдельную сущность. Например, AiPool(aiPath) у которого будет метод TryGetAi().
Если Ai упал или еще не создан - надо создать, а иначе просто вернуть.


Look at the two loops you have implemented. Where are they the same? Where are they different?

Try to find commonalities and do some set up and have one loop.


I'll remove it. The reason I wrote it like that was because logically it's an input parameter and _should_ be const, but reading from a soci blob is not const.


this is ok


I think it would be better practice to have the 'Learning' header tag within the learn-cont div. (similar or Arts and art-cont).

Alternatively, you could have a second div similar to your 'panel' class (with id="learning") that would contain the learning header and learn-cont.

My reasoning behind this is that we should ideally be trying to encapsulate things with related content.


removed default


Looks like your commits did make it!  good job!


ok


You may want to rethink this image, the size and spacing seems to be all out of whack.


If you decide to give for loops a shot, you should be able to have one loop and _maybe_ one conditional. All you need is a for loop and a string builder. It's all in how you set up the for loop. Give it a shot and if you get stuck, I'll see if I can give you some hints to point you in the right direction.

This is probably one of the most difficult parts of writing programs. Loops are hard.


leave in


Maybe should be `var` (can you use that keyword?) because `Var` is a public accessor for `ConfigVar`'s


79453838-AutomateSomeDocumentation (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1103 of 1103 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.04%25-green.svg)


Description is the accurate nomenclature over "usage"


Thank you! Practice makes perfect!


For some of the other convenience factories that have extra data like delimiters and options, maybe make a map[string]interface{} field called Ext


For some reason this sets the right name only every second hit? The odd one it sets the name to the "original" name the mob spawned with?

My guess: Minecraft doesn't like Snowballs existing at the same time with the same name? Because if I output the mobname it is right...


No Prefix ... I decided to just have Name


Missing a > at the end of html


bump


rangeCheckedCast is only called when casting from to a narrower type. I wrote it because ripple reads 64-bit values from the db and puts them into 32 bit values and I wanted a range check. I think boost has a `is_subranged` type trait that I could use with `enable_if` to only do the check for types that need it. However, I don't think that's worth it (but will do so if you do think it's worth it - easy to code, just worried about extra complexity).


This is cool!


86605784-RefrigNoNodeCrash (jasondegraw) - 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)


IDFXT (refaqtor) - Win64-Windows-7-VisualStudio-12: Build Failed

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


The only change I think I'd made in this is that you could handle multiple commas/periods in your number regex. If you use groups, you can make that happen: `/^-?[\d,.]$/` This doesn't really check for the order of the punctuation, but it will at least allow multiple items in the field.


There is an option to name `use` parameters and match by name, but we never do that. The `use` are matched in order (as are the `into` functions).


Because of:
1) inline-block will set the height equal to its parent and
2) inline-block with overflow other than visible will change the baseline to the bottom margin edge and default value for vertical-align is baseline

So the height and margin will chagne after this hack apply. Acutally we should set the bdi height back to original height and set vertical-align to other than baseline. I know you discourage the vertical-align styling in our css so I didn't use this fix first, but just wondering it it's possible to use vertical-align fix here?


if you're going to keep taking my code can you at least link the original commit?


Will the withIndex work on orderBy as well?  If not, how would we specify ordering on an index (for either a table or a slice using between)?


@mciruzzi Esto significa que la busqueda si la hago por varios tags va a devolver aquellos suppliers que contengan AL MENOS UNO de ellos. Otro criterio podria ser que devuelva aquellos que poseen TODOS los tags que incluis como query. Si preferis esto otro avisame y lo cambio


rax retry tests


you removed the gemfile and gemfile.lock from the repo here.


I agree. However, code written with `soci::row` required much less translation from existing code (and would have been easier to review as well). However, as you know there are several problems with `soci::row` and it should not be used in ripple.


这个 show_repo_download_button 很早就去掉了，没有意义。


这里的 share 肯定是私人共享


Yeah, I think I corrected that right after.

Marty

Sent from my iPad

> On Mar 4, 2015, at 9:42 PM, Donald F Coffin <notifications@github.com> wrote:
>
> Marty,
>
>
>
> The link you posted to the Greenbuttondata.org developer website obtains a 404. Were you trying to reference the /library or /build pages?
>
>
>
> Best regards,
>
> Don
>
> Donald F. Coffin
>
> Founder/CTO
>
>
>
> REMI Networks
>
> 2335 Dunwoody Crossing Suite E
>
> Dunwoody, GA 30338-8221
>
>
>
> Phone: (949) 636-8571
>
> Email: <mailto:donald.coffin@reminetworks.com> donald.coffin@reminetworks.com
>
>
>
> From: Marty Burns [mailto:notifications@github.com]
> Sent: Tuesday, March 3, 2015 9:52 AM
> To: energyos/OpenESPI-GreenButton-API-Documentation
> Subject: Re: [OpenESPI-GreenButton-API-Documentation] updated the text and place a green button on the screen (f152545)
>
>
>
> See www.greenbuttondata.org/developer <http://www.greenbuttondata.org/developer> for technical data, apis, and test
> tools.
>
> HTH,
> Marty
>
> On Tue, Mar 3, 2015 at 4:12 AM, charaninduri <notifications@github.com <mailto:notifications@github.com> >
> wrote:
>
> > Is there any documentation for download my data for greenbutton.
> > Are there are any REST api to download green button data.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <https://github.com/energyos/OpenESPI-GreenButton-API-Documentation/commit/f152545b9763b686c77d1e2cfb1b814ee9059c0f#commitcomment-10001793>
> > .
> >
>
>
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/energyos/OpenESPI-GreenButton-API-Documentation/commit/f152545b9763b686c77d1e2cfb1b814ee9059c0f#commitcomment-10008106> . <https://github.com/notifications/beacon/ABvrQ9QFkxuUGlkVEk_scHprM_D8Ga4Oks5nxcIZgaJpZM4DolbH.gif>
>
> —
> Reply to this email directly or view it on GitHub.
>


This regex technically only looks for "in" - the "g" is optional because of the question mark. To actually test for ending with "ing," you should make everything required, and use the `$` metacharacter to place it at the end of the string: `/ing$/`.


On this line, you should probably put the disabled attribute back for it to work.


why an empty main?


you shouldn't ignore the .gitignore file itself and you shouldn't be ignoring the Gemfile either. Create a local gemfile if you want to have a custom one for local development or a group within the shared gemfile


Reading the sqlite documentation, I don't believe we need setupCheckpointing (see: https://www.sqlite.org/wal.html#ckpt). However, please do double check my understanding of that. I am happy to add the checkpointing back if you think it is too risky to take it out (or it should be taken out in a separate PR).


also don't add your local gemfile to the .gitignore, add it to your .git/info/exclude


<!DOCTYPE html>

<meta http-equiv="Content-Type" content="text/html; charset=utf-8"/>
<meta http-equiv="x-ua-compatible" content="IE=9" >

<title>SPSSINC ANON Extension Command</title>

<link rel="stylesheet" type="text/css" href="extsyntax.css" />

SPSSINC ANON Extension Command

Anonymize a set of variables.

SPSSINC ANON  VARIABLES=varnames*

/OPTIONS SVALUEROOT=“prefix” NAMEROOT=nprefix

METHOD=SEQUENTIAL** or TRANSFORM or RANDOM

SEED=number OFFSET=number SCALE=number

MAXRVALUE=positive integer or list of integers

ONETOONE=varnames

MAPPING=“input filespec”

/SAVE NAMEMAPPING=“filespec” VALUEMAPPING=“filespec”“

/HELP]

* Required

** Default

SPSSINC ANON /HELP displays this help and does nothing else.

SPSSINC ANON x y z
/OPTIONS METHOD=RANDOM
/SAVE VALUEMAPPING="c:/temp/values.txt".

This command replaces the values of the specified variables with new values
according to one of three methods.  It is intended for situations where the
true values must be concealed for privacy or other reasons.  It can also
replace the variables names with anonymous equivalents.

VARIABLES specifies the list of variables to anonymize.

OPTIONS

METHOD specifies how to generate the new values. The choices are

SEQUENTIAL replaces each value with a sequential number.  If a value
recurs for a variable, it gets the same number.

RANDOM replaces the values with a random integer in the range 0 to MAXRVALUES inclusive.

TRANSFORM, which applies only to numeric variables, calculates a new value
according to the formula OFFSET + SCALE * value.  If TRANSFORM is
specified and the variable list includes strings, SEQUENTIAL will be used
for those variables.

System-missing values are treated as the same value across cases, and for
TRANSFORM, the new value will also be system missing.

MAXRVALUES, which applies only to the RANDOM method, can be a single integer
applied to all variables or a list of as many integers as there are variables being
anonymized.  For string variables, the range is further limited by the declared
string width.

TRANSFORM and SEQUENTIAL will produce unique values as long as, in the case
of SEQUENTIAL with string variables, the field is wide enough.  For RANDOM,
you can specify a list of one or more variables as ONETOONE, which will
generate unique values if that is possible.  If unique values cannot be generated, the
procedure will stop with an error message.  Using ONETOONE increases the time and
memory requirements.

For repeatability, you can specify a numerical value for SEED that will produce the
same results if the syntax is rerun at a later time with the same inputs.

If the variable is a string, SVALUEROOT can specify a string to be prefixed
to the new integer values if the string is wide enough.

If NAMEROOT is specified, the variables will be renamed to the form
xxxnnn
where xxx is the NAMEROOT value and nnn is an integer.

SAVE

The SAVE parameters create text files containing the mapping of variable names,
if any, and old and new values.  The new-values file is in csv format.
The new values are listed alphabetically by the new
value within variable for ease of lookup.  As explained above, the inverse mapping
is sometimes not unique.

A new-values file can be used as input in a later run by specifying it in
MAPPING if the method is SEQUENTIAL or RANDOM.  If given, any variable mappings defined in
that file are applied to the input values before applying the chosen method.
This means that previously encountered values are mapped the same way
as they were previously.  However, if MAXRVALUE is different or the string
width is different, the results are undefined.

There are side effects to this command.  Value labels and missing value definitions
are cleared, since they no longer apply.  You can anonymize date-format variables, but the
resulting values will usually not display as the values are not in the valid range for dates.
Changing the variable type to numeric will allow the values to display.

/HELP displays this help and does nothing else.


This looks odd to me.  We can now pass `null` through node execution?


Yes, this is needed to make unsigned ints work correctly with boost::optional. My goal was not to change any soci code at all, but I couldn't not think of a way to make it work without removing the check from soci. In fact, there are probably better fixes for this once we decide it's OK to change soci, but I'd like to make that another PR.


SharedRepo(APIView):

       if share_type == 'public':
       ....

这里需要做权限检查


Appveyor win64 has timed out while building inference on every single build since this was merged. Haven't reproduced locally yet, but I think it's probably not a coincidence.


 rogersb11 authored 23 days ago
 pimpmaneaton committed 22 days ago

Authorship retained from pick. Blame someone else.


> Unfortunately, I can't get the feature to work. I've checked the branch out locally, and created a file like `file.c` from [company-mode#286 (comment)] (https://github.com/company-mode/company-mode/issues/286#issuecomment-71353325).

I've fixed it in `company-clang.el` by changing:
`(re-search-forward "TextComment.*Text=\" \\(.*\\) \"$" nil t)`
to:
`(re-search-forward "TextComment.*Text=\"\\(.*\\)\"$" nil t)`

The former regexp requires the presence of both leading and trailing spaces in the comment, instead the latter captures indiscriminately all the text enclosed in double quotes.

> Further, I can't seem to get any comments even in the AST output for the standard functions.

The standard functions in `stdio.h` do not have **Doxygen style comments**, only Doxygen style comments are parsed, setting `company-clang-parse-system-headers-comments` to `t` wont help in this scenario.
Try this:
* use the same `file.c` as before, but instead of `#include <stdio.h>` use `#include <foo.h>`
* then, make a copy of /usr/include/stdio.h: `cp /usr/include/stdio.h /usr/include/foo.h`
* leave `foo.h` as it is, but make this little modification:
```
/**
  * Write formatted output to stdout.
  *
  * This function is a possible cancellation point and therefore not
  * marked with __THROW.  **/
extern int printf (const char *__restrict __format, ...);
```
Now, in the buffer `*clang-ast*` you should have something like this:
```
Dumping printf:
FunctionDecl 0x2b8f820 prev 0x2b8f6d0 </usr/include/foo.h:363:1, col:56> col:12 used printf 'int (const char *, ...)' extern
|-ParmVarDecl 0x2b8f5d0 <col:20, col:43> col:43 __format 'const char *restrict'
|-FormatAttr 0x2b8f8f0 <col:12> Inherited printf 1 2
`-FullComment 0x2b9f100 <line:359:4, line:362:27>
  |-ParagraphComment 0x2b9f060 <line:359:4, col:37>
  | `-TextComment 0x2b9f030 <col:4, col:37> Text=" Write formatted output to stdout."
  `-ParagraphComment 0x2b9f0d0 <line:361:4, line:362:27>
    |-TextComment 0x2b9f080 <line:361:4, col:68> Text=" This function is a possible cancellation point and therefore not"
    `-TextComment 0x2b9f0a0 <line:362:4, col:27> Text=" marked with __THROW.  *"
```
I've modified `company-clang--parse-AST` to handle **multiline comments**, as the one you see above.
``` lisp
          (when doc
            (setq doc (concat doc "\n")))
          (setq doc (concat doc (match-string-no-properties 1))))))
```
Also, note that in the AST empty comment lines are absent, we can only infer them guessing that between `ParagraphComment 0x2b9f060 <line:359:4, col:37>` and `ParagraphComment 0x2b9f0d0 <line:361:4, line:362:27>` there is something missing...
Do you suggest to parse empty lines too?

At this point there is another problem.
> The declaration you see in `stdio.h` (and so in `foo.h`)
`extern int printf (const char *__restrict __format, ...);`
is read (in `*clang-output*`) as
`COMPLETION: printf : [#int#]printf(<#const char *restrict __format, ...#>)`
in this case the keyword `extern` is completely missing, and its meta is
`int printf(const char *restrict __format, ...)`
which once stripped (by `company-clang--strip-meta`) is
`int (const char *restrict, ...)`
preserving the keyword `restrict`

> On the other hand, the declaration in the AST is
`printf 'int (const char *, ...)' extern`
and its meta (when `extern` is filtered out) is
`'int (const char *, ...)'`
the keyword `restrict` is present in the AST on the "ParmVarDecl" line

> In natural cases, the candidate's stripped meta and the AST's meta will never match:
`int (const char *restrict, ...)`
`'int (const char *, ...)'`

We can **filter out all the keywords and have a "rough" match**, or **analyze the AST dump reversing the way Clang "splits" the function's declaration**.
What do you suggest?


Yes you can do `seq.orderByIndex(index:String|Order)` or `seq.orderBy(...).withIndex(....)`.


Thanks @ximinez. One major worry of mine adequate testing. I've seen it pass tests and appear to run well and then I've found problems with the code. One thing I would like to request is for you to run it at least overnight and see anything suspicious happens in the log. I've done that myself, but I am not at all confident of spotting possible errors from the log.


We didn't steal from you, we CPd from someone. If it is your code, then jump down their throat not ours.


Worthless noobs. Oh well.


@the person at fault


Now exception messages are:
Signature validation failed. SAMLRequest rejected  or  Signature validation failed. SAMLResponse rejected
instead of
Signature validation failed. Logout Request rejected  or  Signature validation failed. Logout Response rejected

Can we call any aux method in the messages to show the right value? When debugging I prefer to have those messages, so you can't mix betwen a SAML Response (with Assertion) and a Logout Response


It's probably a good idea to move the ? until after the decimal point group, so that I don't have to include a decimal place for the number to validate. This is one way to do it: `/^-?\d+(\.\d+)?$/`


OK terrific, thanks Keyston


I noticed that you splited the XML logic in a different file.  Since this not affect the main methods that are core of the API I think it is ok.


Ouch. Sorry - no access to windows. I'll improve as I get access to the windows build bots through iojs/build. If someone else could test/verify I'd appreciate it greatly.


i'm going to revert this. I can't build without gemfile..


If possible, I recommand you to reduce the external dependencies in the future.


Your regex works, but it has a couple of problems. A) it allows other things, because it doesn't lock the match to the beginning and end of the string, and B) it checks on keypress, which means it actually tests the previous value (values don't change until after keyup, which is annoying and weird).


zz


LGTM


Понимаю, что теперь бы эту штуку вообще выпилить, но пора на пары)


Thing I would add: In your last group, don't forget to add a repetition metacharacter for numbers after the decimal place.


@mikesherov I can't go deeper in history so I think you are the author of this change. If I'm wrong please tell me where you've got this lines.
Ref: #1137

Anyway, can we make this more concrete (let's say something like `/^([@#]import|#include) ['"<][^\n]+['">];$/`) because it affect unsafe md in comments?


Change these colours


I understand we are an easy target for this since we are a prominent rom but seriously this isn't the first instance of this and frankly it is quite annoying. Please do more research before you accuse us of stealing code. Thank you.


Can you also change the [name](https://github.com/bgaifullin/python3-saml/blob/master/setup.py#L11)? and the version for
name='python3-saml',
version='2.1.0',


Need to add the social menu for the theme (social media icons from Font awesome and enqueue the scripts for link the FB website etc)


Rename this


*Renumber


有人告诉我 IE 是支持这样的 CSS 的, 我测试了一下. IE9 还的确支持. 那就怪了, 错误究竟在哪儿呢?


@joehuchette, this change was never reflected in ``getters.jl``.


Add custom font type here for search box form (field piece)


Remember to add footer section


Yay for error handling!


Yes, JavaScriptCore does require a `--`, like v8. I assume v8 copied it from them actually? Anyhow, spidermonkey's js doesn't need it.

But I'd be ok with adding a test for jsc here as well, but i don't think our bots nor any of my machines have jsc built on them, I'm afraid.


changed to green


Refactor is important but try no change much the logic of the app in order to keep the compatibility of app that currently uses python-saml.  I plan to add a document that explains all the changes that you did and how the method changed but less minimize those changes if possible ;)

P.S Great job.


Игра может крэшнуться только один раз.


Имя метода должно быть RunGame или Run (глагол), а имя класса GameRunner вместо AiTester


This is the spam that never ends...


Tên class không nên viết tắt như này


@davewasmer - 👍 Looks good for submission to me


publish


What's going on here?


I've been messing around with the log and have it looking like uMatrix with the filterings.

![ublocklogfilters](https://cloud.githubusercontent.com/assets/6408935/6498913/f40922e0-c2a6-11e4-8729-6b84f2926cac.png)

Heres the zip file with just the modified files that I've been unzipping after pulling latest commits.  It doesn't have the text filtering or request # limit though.
https://github.com/my-password-is-password/stuff/blob/master/uBlock.Chromium_LogChanges.zip


86605784-RefrigNoNodeCrash (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (633 of 633 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-4.06%25-green.svg)


You can make NoEnemyException an instance of RuntimeException to not declare it if you don't want to


Thanks for catching this. We did an in-person code review for this one.


Yes it goes on and on my friends...


Something personal? :P


Where is the tooltip text for cards stored at? In testing I noticed several cards with inconsistencies between their written text and the tooltip, but the card file and mtg-cards-data seemed fine.

Cards were:
Spawn of Thraxes
Nessian Wilds Ravager
Glissa, the Traitor
Allosaurus Rider
Abhorrent Overlord
Foundry Champion
Fathom Mage
Ember Swallower


Can you write tests of just this function and see what it does?


I dig the reverse regex--I haven't seen anyone else do that--but the reason it's not working is because when you use `attr()` you need to provide a value in order to set it. You also (and I didn't cover this very well) need to clear it with `removeAttr()`. So it should probably look a little more like this:

```js
if (valid) {
  submit.removeAttr("disabled");
} else {
  submit.attr("disabled", "true");
}
```


This function = yPositionAhead


What's the purpose of this change?


http://plakattrophystainlis.blogspot.com
http://plakattrophysouvenir.blogspot.com
http://plakattrophysouvenirku.blogspot.com
http://buatplakattrophyku.blogspot.com
http://buatplakattrophyok.blogspot.com
http://plakattrophyterimakasih.blogspot.com
http://plakattrophyterimakasihku.blogspot.com
http://plakattrophystainlisku.blogspot.com
http://tokoplakatfiber.blogspot.com
http://www.medalijakarta.com
http://membuatplakatakrilik.blogspot.com
http://hargaplakat-blokm.blogspot.com
http://tempatgambarplakat.blogspot.com
http://gambarplakat3.blogspot.com
http://trophypiala.blogspot.com
http://plakatsouvenir2.blogspot.com
http://pialatrophyjakarta.blogspot.com
http://jualplakatakrilik.blogspot.com
http://plakat2.blogspot.com
http://plakatcor1.blogspot.com
http://jualplakatkuningan-blokm.blogspot.com
http://tokoplakatkuningan.blogspot.com
http://trophyaward1.blogspot.com
http://pialatrophy.blogspot.com
http://trophypiala.blogspot.com
http://trophypenghargaan.blogspot.com
http://patungtimah1.blogspot.com
http://jakartatrophy.blogspot.com
http://trophyjakarta3.blogspot.com
http://jualplakat4.blogspot.com
http://jualpiala2.blogspot.com
http://plakat2.blogspot.com
http://plakatfiber-blokm1.blogspot.com
http://trophyfiber.blogspot.com
http://trophyfiber.blogspot.com
http://trophyplakatjakarta.blogspot.com
http://plakattrophyjakarta.blogspot.com
http://jualplakatjakarta-blokm.blogspot.com
http://jualakrilikdijakarta.blogspot.com
http://pusatplakatakrilik-blokm.blogspot.com
http://buatplakatexpress.blogspot.com
http://plakatakrilikexpress.blogspot.com
http://produksiplakatexpress.blogspot.com
http://plakatexpress.blogspot.com
http://bikinplakatkilat.blogspot.com
http://pusatplakatkilat.blogspot.com
http://produksiplakatkilat.blogspot.com
http://plakatkilat.blogspot.com


88734010-AFN-horizontal-openings (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1106 of 1106 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.08%25-green.svg)


Used c++


오타가 있는데... 설마 확인 안하고 커밋 풀 한건가????


http://pembuattrophy.blogspot.com
http://produsentrophyku.blogspot.com
http://trophytimahaja.blogspot.com
http://trophytimahok.blogspot.com
http://trophytimahku.blogspot.com
http://buatplakatsouvenir.blogspot.com
http://souvenirplakat2.blogspot.com
http://produksiplakatcepat.blogspot.com
http://plakatcepat1.blogspot.com
http://plakatdariakrilik.blogspot.com
http://plakatplakatakrilik.blogspot.com
http://jualsouvenirplakat.blogspot.com
http://desainplakatsouvenir.blogspot.com
http://tempatplakatsouvenir.blogspot.com
http://plakatjakarta-blokm1.blogspot.com
http://plakatjakarta-blokm.blogspot.com
http://plakatjakarta-blokm2.blogspot.com
http://plakatdijakarta-blokm.blogspot.com
http://plakatdijakarta1.blogspot.com
http://plakatlogam2.blogspot.com
http://pusatplakatakrilik.blogspot.com
http://produksiplakatlogam.blogspot.com
http://membuatplakatlogam.blogspot.com
http://plakatlogamjakarta.blogspot.com
http://plakatplakatkayu.blogspot.com
http://pusatplakatlogam.blogspot.com
http://plakatplaaktkayu.blogspot.com
http://plakatdariakrilik1.blogspot.com
http://membuataplakatdariakrilik.blogspot.com
http://pusatjualplakat.blogspot.com
http://pusatplakatkayu1.blogspot.com
http://jualplakatkristal-blokm1.blogspot.com
http://jualplakatdijakarta.blogspot.com
http://stempelfiber.blogspot.com
http://stempeltimbul.blogspot.com
http://stempelembos.blogspot.com
http://stempelwarna1.blogspot.com
http://bikinstempel1.blogspot.com
http://gudangstempel1.blogspot.com
http://pusatstempel1.blogspot.com
http://buatstempel1.blogspot.com
http://produksistempel.blogspot.com
http://stempel1.blogspot.com


http://plakatinstan.blogspot.com
http://plakatjakarta-ok.blogspot.com
http://pusatjualplakatjakarta.blogspot.com
http://jualplakatfiber.blogspot.com
http://jualplakatfiberku.blogspot.com
http://trophytimah7.blogspot.com
http://plakatbening2.blogspot.com
http://produksitrophytimah1.blogspot.com
http://trophytimah6.blogspot.com
http://produksitrophyjakarta.blogspot.com
http://desainplakatdanplakattrophy.blogspot.com
http://plakatkayudanplakatsouvenir.blogspot.com
http://plakatfiberdanjualplakat.blogspot.com
http://pusatjualplakatok.blogspot.com
http://jenisplakatok.blogspot.com
http://jualplakatku.blogspot.com
http://pusattrophytimah.blogspot.com
http://pusatpialatopscore.blogspot.com
http://jualpialatopscore.blogspot.com
http://pusatsepatuemas.blogspot.com
http://tempatpialatopscore.blogspot.com
http://produksipialatopscore.blogspot.com
http://pembuatanpialatopscore.blogspot.com
http://pengrajinpialasepatuemas.blogspot.com
http://pialatopscore.blogspot.com
http://pialasepatuemas.blogspot.com
http://jualsepatuemas.blogspot.com
http://bikinsepatuemas.blogspot.com
http://tempatpembuatanplakatsepatu.blogspot.com
http://produksiplaaktsepatu.blogspot.com
http://plakatsepatu.blogspot.com
http://patungtimah4.blogspot.com
http://awardtrophy1.blogspot.com
http://trophyaward2.blogspot.com
http://pialatimahjakarta.blogspot.com
http://pialatimah.blogspot.com
http://jualtrophytimah1.blogspot.com
http://jualtrophy3.blogspot.com
http://patungtimah3.blogspot.com
http://plakattimah.blogspot.com
http://produksiplakatbening1.blogspot.com
http://membuatplakatbening.blogspot.com
http://jualplakatbening.blogspot.com
http://pusatplakatbening1.blogspot.com
http://membuatplakatresin.blogspot.com
http://pusatplakatresin.blogspot.com
http://jualplakatresin1.blogspot.com
http://plakatresin3.blogspot.com
http://jualtrophytimah.blogspot.com
http://tempattrophytimah.blogspot.com
http://trophytimah5.blogspot.com
http://jualtrophyjakarta.blogspot.com
http://pusattrophyjakarta.blogspot.com
http://trophyjakarta6.blogspot.com
http://trophy2.blogspot.com
http://plakataward1.blogspot.com
http://plakatwedding2.blogspot.com
http://pialaoscar1.blogspot.com
http://pusatpialaoscar.blogspot.com
http://pembuatanpiala1.blogspot.com
http://produksipiala.blogspot.com
http://pusatpiala.blogspot.com
http://pusatplakattrophy.blogspot.com
http://produksiplakataward.blogspot.com
http://bikinplakataward.blogspot.com
http://pembuatanplakataward.blogspot.com
http://pusatplakataward.blogspot.com
http://pusatplakatsilver1.blogspot.com
http://jualplakatsilver.blogspot.com
http://tempatplakatsilver.blogspot.com
http://tempatplakatoscar.blogspot.com
http://trophyoscar.blogspot.com
http://membuatpialaoscar2.blogspot.com
http://pangkalpinang1.blogspot.com
http://plakataward2.blogspot.com


Yep it is.
go watch this video https://www.youtube.com/watch?v=OHbSx2iVujE
Yah I think it doesn't matter sir didn't add any restrictions to it right?


@jbergstroem Don't worry, I will fix this after installing curl. Just moment.


그러네.. 니가 리뷰 잘 해줬을거라 믿었음 ^^


Did you run it does it work in you laptop?(just want to confirm)


Sniping at me on my own repo...


86605784-RefrigNoNodeCrash (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1647 of 1647 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)


이건 왜 넣었다 뺀거야?


http://www.plakatlogam.com
http://www.plakatunik.com
http://www.pusatplakatajakarta.com
http://www.plakatsilver.com
http://www.trophyplakat.com
http://www.pusatpercetakanjakarta.com
http://www.membuatplakat.com
http://www.tempatplakat.com


`join(' ')`과 `join(" ")`의 차이가 뭐지?


Ya
Windows Builder work charm.
If it possible I'm ok with do GUI in that way.


스타일 체크에 걸리고 안 걸리고의 차이


这一句可以直接放在主函数开始的地方，不需要重复调用，而且可以代替`readAll()`函数中的那句`setlocale`


boolean attribute는 presence validation 을 하는 게 큰 의미가 없어서..


Yes let's do it this way.
Watch the video so you can learn how to make it.
It is pretty easy.


happens locally, but not with cross-compile from cygwin. appears to be a function of which compiler is being used. will experiment with things and tweak appveyor if necessary


89224666-FixNANProblems-Issue4557 (mjwitte) - 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)


QwtUpdate (macumber) - Win64-Windows-7-VisualStudio-12: OK (2185 of 2201 tests passed)

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


Nice kang bro
https://github.com/SaberMod/GCC_SaberMod/commit/1a3ff53b7eaaa0c2cbe8b8671647ccc8ca56253c


Oh, I added this because I wanted the game to pause for just a bit after the player hits something so they can see what caused the game-over.  I found this way to implement that online and figured since it was just a small snippet it would be ok to add it here, but if this is bad I can take it out.


Ok, I was under the impression that RuntimeExceptions were pretty bad and should be avoided whenever possible.  Would it be significantly better to use a RuntimeException here, or is that just a matter of personal choice?


LGTM


The most beautiful way is to create another state where the tick() function is just counting down time and not doing simulation. Or where it is doing nothing and the keyEvent will reset the game (so you have to press a button for the game to be done.)


Good
So what are the codes that have you upload as a 7th commit?
Before today lunch time I'll upload the Full GUI (RealState Class)


Personal choice. It is bureaucratic to keep track of them. If you find it frustrating, use the escape hatch


#include <iostream>

using namespace std;

void display_board();
void player_turn();
bool gameover();

char turn;
bool draw = false;
char board[3][3] = {{'1', '2', '3'}, {'4', '5', '6'}, {'7', '8', '9'}};

int main()
{
    cout << "Tic-Tac-Toe"<<endl<<endl;
    cout << "Player 1 = [X]   Player 2 = [O]"<<endl<<endl;

    turn = 'X';

    while (!gameover())
    {
        display_board();
        player_turn();
        gameover();
    }
 if (turn == 'O' && !draw)
    {
        display_board();
        cout << endl << endl << "Player 1 [X] Wins! Game Over!"<<endl;
    }
    else if (turn == 'X' && !draw)
    {
        display_board();
        cout << endl << endl << "Player 2 [O] Wins! Game Over!"<<endl;
    }
    else
    {
        display_board();
        cout << endl << endl << "It's a draw! Game Over!"<<endl;
    }
}

void display_board()
{

    cout<<endl;
    cout<<" "<<board[0][0]<<" "<<board[0][1]<<" "<< board[0][2]<<endl;
    cout<<" "<<board[1][0]<<" "<<board[1][1]<<" "<< board[1][2]<<endl;
    cout<<" "<<board[2][0]<<" "<<board[2][1]<<" "<< board[2][2]<<endl;
    cout<<endl;

}

void player_turn()
{
    int choice;
    int row = 0, column = 0;

    if (turn == 'X')
    {
        cout << "Player 1 turn [X]: ";
    }
    else if (turn == 'O')
    {
        cout << "Player 2 turn [O]: ";
    }
    cin >> choice;

    switch (choice)
    {
        case 1: row = 0; column = 0; break;
        case 2: row = 0; column = 1; break;
        case 3: row = 0; column = 2; break;
        case 4: row = 1; column = 0; break;
        case 5: row = 1; column = 1; break;
        case 6: row = 1; column = 2; break;
        case 7: row = 2; column = 0; break;
        case 8: row = 2; column = 1; break;
        case 9: row = 2; column = 2; break;
        default:
            cout << "You didn't enter a correct number! Try again"<<endl;
            player_turn();
    }

 if (turn == 'X' && board[row][column] != 'X' && board[row][column] != 'O')
    {
        board[row][column] = 'X';
        turn = 'O';
    }
    else if (turn == 'O' && board[row][column] != 'X' && board[row][column] != 'O')
    {
        board[row][column] = 'O';
        turn = 'X';
    }
    else
    {
        cout << "The cell you chose is used! Try again"<<endl;
        player_turn();
    }

}

bool gameover()
{
    for (int i = 0; i < 3; i++)//Check for a win
    {
        if ((board[i][0] == board[i][1] && board[i][1] == board[i][2]) || (board[0][i] == board[1][i] && board[1][i] == board[2][i]) || (board[0][0] == board[1][1] && board[1][1] == board[2][2]) || (board[0][2] == board[1][1] && board[1][1] == board[2][0]))
        {
            return true;
        }
    }

    for (int i = 0; i < 3; i++)//Check for draw
    {
        for (int j = 0; j < 3; j++)
        {
            if (board[i][j] != 'X' && board[i][j] != 'O')
            {
                return false;
            }
        }
    }
    draw = true;
    return true;
}


我不太看得懂这里两句有什么用……
如果不是有效的参数，也不存在该文件，应该报`No such file or directory`吧……？


An easy target? I could care less if you were the only "target". That's entirely beside my point. If you're having repeated problems with plagiarism maybe _you_ should do your research when a commit says creds someotherrom. Also, I'm pretty sure i never accused you of stealing because I don't really care that much. all I asked for was a link to the original commit next time someone incompetent in git/gerrit workflow pushes code to your review with the wrong authorship.


sdfgsdfgsdfg


实际上这里不用判断……重复提供一个选项一般来说是可以接受的


Then perhaps this might work :
Using the dependency injection, the jQuery object (ou bien "service") can be injected into the $dl object, allowing for the generation of jQuery scripts in Phalcon controllers, respecting the MVC design pattern.

or

Using the dependency injection, the jQuery object (ou bien "service") can be injected into the $dl object, allowing for the generation of jQuery scripts in Phalcon controllers that respect the MVC design pattern.

It depends what is respecting what. Is it the scripts that respect the MVC design pattern?


額外補充 : 測試將首頁logo與內頁logo同化


baka是哪个=. =？


I think one thing we want to avoid is having to install the memoizer on the class prototype in the first place. I can think of doing this in two ways:

1. Generating code that checks to see if a method is linked, and if not links it:

```
(o.foo || o.link("foo"))(1, 2 , 3);
```

This has the drawback of generating extra code for each virtual method call.

2. Inject a proxy object on the prototype chain that traps and links the method.:

```
o.foo(1, 2, 3);
```

This has the drawback that it's potentially slow. I have not benchmarked this.


I think I expected something more like `/ing$/`, but word boundary (`\b`) is a pretty decent way to handle it too.


OK, I was planning on adding another state anyway so the game could be restarted, so I'll change this to be state-based as well when I add that in. Thanks!


Can you generate at runtime instead of hardcode?


All you're lacking is the ability to clear the disabled property here. Otherwise, looks good.


we have a tab view widget in analysis/; can we please use that?


Don't take my approach the wrong way. I'm not trying to make waves or be a dick. But seriously I'm getting kinda sick of finding my code scattered all the fuck over with no change except to authorship and this is by far not the only commit I wrote that I've found this way in your source.


Then you need to look at who AUTHORED the commit, not who COMMITTED it. All you are doing right now is showing your lack of knowledge in how github and gerrit work.


还有几个问题

1. 如果提供多个目录参数的话程序会崩溃，可以用valgrind检查一下
2. 主函数太长……一般情况下，我们需要控制单个函数的长度来维持代码的可读性和可维护性，常用的标准是函数的长度不能比你的脸长。当然这个标准太严格，一般可以松动，但也不能太夸张……


I understand it sucks, and people suck. But we do our BEST to keep our picks clean. If someone else is stealing your code we have no way of knowing it, we are picking it from another source and you should be going there and putting their heads on pikes, not ours. Sorry if I am coming off as a dick but I get tired of explaining this after a while.


왜 스페이스바를...?


@mppf: My compile is breaking with -Wall -Werror because a size_t value will always be <= SSIZE_MAX.  I propose eliminating this part of the conditional (and may remove it tonight if I can get a second pair of eyes on it, just in case it breaks some configuration that we don't smoke test).


"if you're going to keep taking my code can you at least link the original commit?"
"Also, I'm pretty sure i never accused you of stealing because I don't really care that much."
"all I asked for was a link to the original commit next time someone incompetent in git/gerrit workflow pushes code to your review with the wrong authorship."
"this is by far not the only commit I wrote that I've found this way in your source."
.........
"Don't take my approach the wrong way. I'm not trying to make waves or be a dick."
.........
Really?


Thanks!


LMAO, if the dude didn't care as he claims, then he would of NEVER commented. He needs to go bash the guy that actually "stole" the code. If we stole the code then it would say author was one of us not commited by one of us.


whats this?


89224666-FixNANProblems-Issue4557 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1645 of 1647 tests passed)

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


> Also add ipkg-fw that uses /lib/ld-uClibc.so.0 for bootstraping.

As for me, I made Entware package manager as [static binary](https://github.com/Entware/entware/blob/master/packages/opkg/fix-path.patch#L66). No matter how f\w toolchain will be changed, `ipkg` will remains stable and working.


Test Repo


89438532_RadianceBugFix (rpg777) - x86_64-MacOS-10.9-clang: Tests Failed

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


@dbaio was this confirmed as fixing the problem in pfSense 2.2? I installed the update today, restarted apcupsd, unplugged my UPS from the wall outlet, but don't get an email.

Test emails work fine for me on Advanced > Notifications.
Emails also worked for me with Apcupsd+mailreport packages on pfSense 2.1

Thanks


QwtUpdate (macumber) - i386-Windows-7-VisualStudio-12: OK (2188 of 2201 tests passed)

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


Lol oh I will.. But the fact that this passed your review with that cute little "credz" tag makes you just as guilty whether you want to own up to it or not.


Please update usage string (line 22) to include max.


nice!
what was the difference in number of errors cause of this?


还有，其实程序中有很多的字符串复制是不必要的。如果对字符串是只读的操作，而且后续的其他操作保证不会覆盖这个字符串或者覆盖时已经不再需要这个字符串，一般只需要复制指向字符串的指针即可。此外，申请空间后，`memset`并不总是必须的，只需要把第一个字节赋值为0即可视为空字符串。


Jah, tenkte på det, så trodde jeg bare forka dev-branchen, men selvfølgelig forker man hele repoen. Vil fikse før PR :-).


`var` wont work. How about `newVar`?


This also appears to make pykek generated tickets work properly.


Sounds fine, the main concern was having Var for accessing

On Wed, Mar 4, 2015 at 10:30 PM, Matt Aitchison <notifications@github.com>
wrote:

> var wont work. How about newVar?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/MattAitchison/envconfig/commit/409fa4cd9d22ab999d9f7d336bffa9e8aa9b7b08#commitcomment-10044577>
> .
>

--
Jeff Lindsay
http://progrium.com


Nice catch. Can you submit a pull request (so that you can get added to the list of contributors?)


Que piensas programar?


Didn't we just get _rid_ of `Object.defineProperties`? 95fb798980d7e404c413e29e20016db9052e2bf2


jquery file for validating input fields on Add Peer Reviews.htm page


@rwaldron: what is the value of separate execution paths here?


I think this is completely unnecessary, given that it's already covered by the preceding `this.cache( owner )`.


yeah, use strpprintf should be fine


r+


Извини, в следующий раз отправлю только то, что нужно.


Why does this method throw InvalidQueryException?


saw approval from jdm
at https://github.com/gilles-leblanc/servo/commit/268a140e25437658aa6a5ea17fe073bc5d56ec89


merging gilles-leblanc/servo/issue-5063 = 268a140e into auto


gilles-leblanc/servo/issue-5063 = 268a140e merged ok, testing candidate = a9b74830


89224666-FixNANProblems-Issue4557 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (633 of 633 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-4.1%25-green.svg)


膜拜


Ack


Really dude? Really? This is why I don't want to have children....


Yeah, I forgot about it when I added the code for Var.


You may want to push to gerrit.


@dylanwenzlau @jebentier

Gulp's bump script didn't automate nearly as much, so I went with Grunt instead (and anyway this matches what we're using on the two javascript repos).

First-time setup:

```
cd DB/
npm install -g grunt-cli
npm install
```

Commit to the repo like normal, but the new commits won't be live (won't ever be installed by `composer update`) until they're tagged with a release version. To do that:

```
grunt bump
```

Which will automatically do the following:

1. update the **patch** number in `composer.json` and `package.json`
2. create a new commit titled "Release v0.1.X"
3. create a new git tag
4. push all of that to GitHub.

After that, the patch will be installed the next time we run `composer update`. If we need to do minor, major, or custom version bumps:

```
grunt bump:minor
grunt bump:major
grunt bump --setversion=1.0.0
```

There's also a `--dry-run` flag if you want to test something without actually pushing to GitHub.

Does this sound like a decent workflow? It's worked well for me on NPM projects but i've never tried it with a PHP repo before.


IDFXT (refaqtor) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Build Failed

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


Redundant. Should append String in previous line.


Should specify limit to improve the performance.


@mciruzzi Tincho, podras jugar con esto un rato para ver si no se rompio nada? Esta version es la que esta deployada actualmente en Heroku.


some tests failed:
failure: http://build.servo.org/builders/gonk/builds/116


This really doesn't seem like an improvement. More code instead of less? Nah.


IDFXT (refaqtor) - 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)


IDFXT (refaqtor) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-NA-red.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-0.0%25-red.svg)


What is the real question you're asking? Would you prefer that this was `Object.defineProperty`?


IDFXT (refaqtor) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-NA-red.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-0.0%25-red.svg)


@changokun: more stylistic changes...

there are two camps when it comes to block indicators.  the overwhelming majority prefers to use `{..}` for single line blocks, and `do..end` for multiline.

there is another approach (which i don't regularly use, but i _do_ find appealing), which is to use `do..end` if the intention of the block is to cause side effects, and braces if the purpose for the block is to provide a return value.


但是有delay的时候，浮层不一定可见

2015-03-05 10:46 GMT+08:00 leon <notifications@github.com>:

> 这里建议直接使用this.hasState('show')来完成，不需要额外的_visible了
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ecomfe/moye/commit/07cd0f62a8becfc1ec257cc3954b1a4ce0a1e814#commitcomment-10043804>
> .
>


Resource*


Congratulations! You have implemented NoSQL in SQL!


Value? Probably none. I'm sure there was probably very good reason at some point—whether that reason holds up today or not is worth checking. I'll look at it tomorrow.


done


Ошибка. send_comMand_to_server. Судя по всему, тесты не запускались перед коммитом.


It's a good idea. I will add it.


LGTM


Maybe use `isTRUE(try(packageVersion("rstudio") > "0.99.324", silent = TRUE))` to avoid the dependency?


1. Код перед коммитом не запускался, очевидно. См. import send_comMand_to_server
2. FileNotFoundError: [Errno 2] No such file or directory: '/home/maria/prog/my_data' Почему это не в репозитории? Как тесты запускать?
3. см. другие вопросы.

Не принято.


Here is where I would add:
```
# This finds the peak of the spectrum, smoothing to reduce noise.
max_index = np.argmax(median_filter(data['DATA'].T,size=value))
# This sets the velocity of the guess to be the max.
guess[4] = v11[max_index]
```


Откуда такие аббревиатуры берутся? Почему бы не назвать например SshGetter. На 5 букв длиннее, зато все сразу понятно. Больше никаких непонятных аббревиатур.

Опять же, почему xxx_object? Ты что, все объекты объектами называешь? надо оставлять просто xxx, из нижнего регистра и так понятно, что это инстнс класса, а не класс.


The median is only used to reduce the noise in the peak finding.  This only needs to be done to the (1,1) data.  It doesn't need to be done to the `v11`, `v22` etc. or the `t22` etc.


I think the check is a kind of dead code which never happens as we are calling this function after just setting the org.


Плохое имя для переменной. Result, variable, data.
Как насчет того, чем на самом деле этот объект является?


QwtUpdate (macumber) - x86_64-MacOS-10.9-clang: Tests Failed

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


В документации к unittest есть совершенно королевские методы self.assertEqual. Прочитай про них и выброси self.message.


thanks for this @dougwilson


Не pass, a NotImplemented


informayion


run и run2 - сразу понятно что это такое? (Нет)


Updated description


@AFBarstow can you please check.


amiin vro


Earlier I did study this method with an intention to move it to the `ContactsExtractor`. Then it seemed odd to dump the responsibility of rendering onto a `ContactsExtractor`, so I abandoned the thought. Still feel it is better as is.


import re
is removed, but u need to put it near the top of the file


@lhmmry  :walking:
Check below
```cpp
//
// To implment the pseudocode in 16.3 CLRS, we need to define a min-priority-queue.
// STL provides such thing called std::priority_queue. Problem is that std::priority_queue
// is a max-priority-queue in default. To modify it as what we want, we need
// to provide a custom predicate that works similiar to operator ">". That's why we defined
// lambda expression "greator" and pass it to queue's constructor.
//
// Google terms like  c++ 11 lambda expression, std::priority_queue, etc for more detail.
//
// If needed, we can discuss it in depth next time in Lab.
//
SharedNode make_tree(FrequencyMap<Char, Freq> const& map) const
{
	//lambda expression greater that works like a ">"
	auto greater = [](SharedNode lhs, SharedNode rhs)
	{
		if (lhs->freq_ != rhs->freq_)
			return lhs->freq_ > rhs->freq_;
		else
			return lhs->character_ > rhs->character_;
	};

	// pass greater's type to form a new type MinPriorityQueue
	using MinPriorityQueue = std::priority_queue < SharedNode, std::vector<SharedNode>, decltype(greater) >;
	//                                                                                ^^^^^^^^^^^^^^^^^

	// pass greater to queue's constructor
	MinPriorityQueue queue(greater);
	//                    ^^^^^^^^^

	for (auto const& pair : map)
		queue.push(make_new_node(pair.first, pair.second));

	for (int count = 1; count != map.size(); ++count)
	{
		auto merge = make_new_node<Char, Freq>();
		merge->left_ = make_new_node(*queue.top());
		queue.pop();
		merge->right_ = make_new_node(*queue.top());
		queue.pop();
		merge->freq_ = merge->left_->freq_ + merge->right_->freq_;
		queue.push(merge);
	}

	return queue.top();
}
```


@ahmadnassri no problem :) I try to ensure that dep changes get rippled up in a timely manner, haha.


good call


There is still a small probability that makes this assertion wrong.


The `rstudio` package version no longer tracks RStudio's (the `manipulate` and `rstudio` packages are now on CRAN and can be installed independently of RStudio client version).

If the dependency on `rstudioapi` is a problem I could rewrite this so it doesn't do anything if that package isn't installed (IIRC there's a way to do this that doesn't flunk `R CMD check`).


全てのフィールドが必須でもないので、
デフォルトコンストラクタも定義してあげておいたほうがいいと思うよ。

Book book = new Book();
book.setBookName("スッキリわかるJAva入門");
　・
　・
　・
ってな感じで


v is optional now


> this is completely unnecessary

When `this.register(elem)` is called it creates an empty data object, which may be a _new_ data object, or replacing an _existing_ data object. If the `owner` _has_ data, then `this.cache` wouldn't even make it to `return this.register( owner, initial );`—which doesn't do what Data needs it to do at that point anyway.


2) go runtime do creates goroutine to send a value to a channel.
3) this cost is not negilible.
Also consider that timers operations (starting new timer, firing expired) are protected by global lock, and they have O(log n) cost, so if you do nor stop useless timers, every timers operation becomes more costly.
4) this *IS* profile guided optimization: i wrote a library which could send 900000rps, and this cost becomes valuable at the 100000rps.

But if you have no wish to rich that rps, you can decline any optimizations.


変数名をキャメル記法にしたのであれば、
セッター、ゲッターの名前もそれに合わせようよ

getBook_name → getBookName


Hell ya!


I think this is vestigial and nothing breaks when it's removed.


Looks good, but for ease of maintenance and reading, I would suggest putting the release notes into a single file, in reverse chronological order.


Most of time channel will be empty. Non-blocking send to single channel is very cheap (ie select with one channel and default) cause it is translates. Select with many channels are slower cause it needs to allocate memory and do some complex logic.


use “download.mokeedev.com”,thanks to mokee


I added inline answers.


Link is broken. Please do correct


Sets maxAge to 100000 ms


@erikbern The ``sqlalchemy`` import here is used for the SQLAlchemyTarget. Not sure if the comment regarding ``db_task_history`` is relavant.


Deu pau no log.

:octocat:


테스트 코멘트


Was the code changed to use -1?

I believe my implementation used 0.


so cool! thank you very much ! is there any  tools can generate my entity class for npoco?


`result` is the postcss instance right ?


No `text` ? Maybe we should create simple method like warn() for log(text, data)?


If we use a method on the node, can we make the plugin field optional (taken from a postcss instance prop, updated on the runtime for each plugin loop ? Not sure if it's doable)


Looks good !


These changes look fine to me.


This shouldn't be needed. See #3972


Happy with this :star:


Even if we use `MinInt32`, there's a still a possibility that `kill(2)` will kill something if the user is careless ( if there's a pgid with the same value ) right ? I dont know if we should handle that case.

But this is still much better than returning -1.


have a look at the jsons in dropbox along with this commit. crosses are strains used as reference sera. clicking on them shows log2 titer distance of measured strains. you can substract virus and serum effects by checking boxes (but need to  click on the serum again, doesn't update). In addition, you can check the use predicted values to see the predicted titer distance of every node to the serum you click on.


This change doesn't eliminate batching, cause there will be a lot of requests in a requestsChan when go runtime switched back to clientWriter goroutine (note that requestsChan is buffered with huge buffer).

Just add a logging of counter in default which leads to Flush, and you'll see that at the 40000 rps it incremented just with several dozens of requests, and possibly even hundreds of requests.

And with lower requests rate you will care more about latency then about cpu.


this is all a little hacky at the moment, but could very well be useful


For debugging the init part.


Should the link be the whole phrase "Freedom of Information Guidelines" or just "Guidelines"? I think adding the Freedom of Information bit is better as that's the name of the document, but I could be being pedantic!


n/p... Thanks for the speedy fix. :)


Don't need this route


Sorry, that was my confusion. But since we're here, I'll ask a related question: could we use `cache = owner[ this.expando ]` instead of `cache = this.cache( owner )` to avoid _creating_ data objects when we're trying to remove data?


GOAL: Write a program that repeatedly prompts a user for integer numbers until the user enters 'done'. Once 'done' is entered, print out the largest and smallest of the numbers. If the user enters anything other than a valid number catch it with a try/except and put out an appropriate message and ignore the number.


Thanks. how can I tell if this is a debug build? is there standard cmake for this or do we have a flag?


Can you provide a target of Makefile to automate as well?


Huh, I never knew GitHub creates these commits.


> use our dependency system to retrieve and build nestopia and then require it in your CMakeLists.txt for your game.nestopia project.

how can I use the dependency system to retrieve and build nestopia?


hello


still not working.


@shlevy it works locally, just pushed it.


... 46bcd770161845e516006f5987459d41e511671e ... wtf?


There's a link right after this that goes to the Freedom of Information Fact Sheets and so I'd guess the original intent was not to repeat ourselves. This reads better.


If you change `value` to `url` we'll create the link for you. However, the `label` becomes the anchor text, so you should update it to `Evolves into <name>`


ha ha. Django docs made me think this was only a Django 1.6+ thing :)


Who do you think will win the world cup?


It looks like these calls to Dispose will need to stay anyways regardless of whether the memory issues came up or not. is that right?


I assume this was jsut a line break


Make sure this means what you think it means. (R * S)


89438532_RadianceBugFix (rpg777) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2181 of 2213 tests passed)

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


I think there are some LLVM passes that get rearranged or disabled or something to work around some bugs in 3.3 when we have `USE_SYSTEM_LLVM` on, whereas the LLVM binary I'm downloading on AppVeyor was built by me including the Julia patches. Still confusing though.


86605784-RefrigNoNodeCrash (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1141 of 1141 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.62%25-green.svg)


It has been done on the previous commit.


Looks like you misspelled "canola"! Let me know when you've updated the change.


聊天服务端


I don't see the rstudio package on cran, only rstudioapi. On Rstudio 0.98 the rstudio package is still included right?


It was a bug in script or something else? Can you reproduce it?


Is the outcome here that because of this function, a `JsonField` accepts either a JSON-formatted string (in which case it decodes it) or just a regular string (in which case it returns the original string)? Is the _input_ to this function ever not a string (this code makes it seem so)?

The other question is whether this lenience is a good thing/intentional—is the intention that in the case of something being a string, you don't want the user to have to quote it? So that `"EDD"` and `EDD` are both valid ways of entering the string `'EDD'`?


89438532_RadianceBugFix (rpg777) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


wat


QT_MODULE_BRANCH ?= from "stable" to "5.3"


To resolve the 2nd issue, please add "mkdir ../res/src/cocos/" before copy files into "cocos" folder.


QwtUpdate (macumber) - x86_64-MacOS-10.9-clang: Tests Failed

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


Actually I just realized you can set `url`, `label` and `value` to achieve basically what you're doing here. It results in special styling/markup that we made for the Infobox


I have a kmp based index of method, but most of the fancy string search algorithms like that require building a lookup table.  Surprisingly, many of the languages I looked at (Java) had very simple implementations.


You're right about dependant always being an `Edition`. The class name emphasises that.

For now, a dependable is only a `Contact`. We'll be making the `Edition` dependable soon enough when we do the same for admin links. At that point, it should be easy to make a distinction between a dependant edition and a dependable edition.

Given this setup:
```
corporate_info_page = create(:corporate_information_page)
edition = create(:submitted_news_article, body: "Our main office: [Contact:#{corporate_info_page.id}]")
edition.publish!
```

```
EditionDependency.create!(dependant: edition, dependable: corporate_info_page)
```
reads better, more explicit than:
```
EditionDependency.create!(edition: edition, dependable: corporate_info_page)
```

I think it is a subtle point, but hopefully should help us think clearly, and give us a hint to come up with suitable variable names like: `dependant_edition`, `dependable_edition` etc. I'm sure there can be a better example than the one above.

This should also answer [your comment](https://github.com/alphagov/whitehall/commit/06ad4cb5233004e10eb2bb39c959e287b1c85cdc#commitcomment-10033419) on the model instance variable name.

If this still feels inconvenient, let me know.


Good catch!


actually this bit is a bit hacky... lets maybe do the pieces of this patch that are clear [the location etc glue] and make a new button tool in the mouse mode selector for making an annotation....


I prefer `numix-gtk-theme` (not a strong option).


also change complex depth equation to the shallower one used by earth impacts
and set bypassSpring default to true


I'll try it out in the am


Should be `numix-icon-theme` and `numix-icon-theme-circle`.


as mentioned, you don't need to restrict the length, we do it automatically :)


Please note that now that I've fixed the bitfield issues this is completely vestigial and irrelevant.  This code will be removed in future versions.


Platforms should be `stdenv.lib.platforms.all`.


This line and the two following are what've fixed the intro scans.


QwtUpdate (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


where is all this meta information used? I feel like we'll forget to update both places


Hint: using `.gitignore` can ease the issue. Please check this: http://git-scm.com/docs/gitignore


if there is only one author, might want to say FindTheBest Engineering. Not sure where this would actually show up though..


I'll try this out on my next commit!


@blackrim's parentwise idea for synthesis: since nodes can have at most one parent, look in that direction (rather than tipward, where an arbitrary number of children may be possible).
Seems conducive to draconian rank decisions (implemented), as well as less strict criteria where compatible (but lower ranked) relationships are considered.
Crazy fast, and not even remotely optimized.


review


Not sure. @czue wrote this method originally.


This was my idea:

> For <b> Federal Authorities </b>, have a look at the [Freedom of Information guidelines](http://www.oaic.gov.au/freedom-of-information/applying-the-foi-act/foi-guidelines/) and [fact sheets](http://www.oaic.gov.au/freedom-of-information/foi-resources/freedom-of-information-fact-sheets/) on the Information Commissioner's website.


Nice one, thanks!


This seems pedantic for a one liner, but I'd recommend having a separate property for the service

bieService.baseURL

this will just make deployments more flexible elsewhere.


descriptor_id should preferably reference descriptor_table - what was the reason for the change?


Is file 2 really needed?


Revisiting this, currently .footer__outer-wrap.SGPP_FixedFooter_outerWrap background is set to inherit the body background color. The footer growing in size when you scroll to the bottom is not that noticeable, however if you color it black http://i.imgur.com/U42gMM7.png you can see the 6px added when body.SGPP_FixedFooter is set to 45px.


I wouldn't take this in, because it is not a typical user case and against the UI specification. I don't wanna bring addtional complexity for this rare usage.
An workaround with current implementation is using a transparent image as icon


subscript error will here due to empty args


Hello everyone.
   I am sorry that I missed meeting all of you this evening. I read over your minutes and would really like to get involved with the Fort Collins Creator Hub. I think a conversation would be the best introduction. Is anyone available to meet and chat with me?
hunter@everybodylovessauce.com


Hi Bchavez,

I have followed your steps to integrate Coinbase into my .net application. But i getting error on creating Client method i.e "No connection could be made because the target machine actively refused it 127.0.0.1:8888" . Can please me guide me regarding this.


LGTM


This is handled by [GitHub Issues](https://github.com/WizardSpire/ccarps/issues) as well as the [WizardSpire Forum](http://forums.wizardspire.com/viewforum.php?id=5). Having a file like this excludes people who do not have write access and will just add unnecessary pull requests.


Thanks, fixed in the new release.


This file seems like dummy code since it only print `"device=%s\n NO LOG AVAILABLE\n"` but not log buffer. It should be removed or added some concrete functionality??


@trustin is the same also in epoll transport ?


I thinks this could be reasonable for some cases, but it would be a little wired to remove an item in a builder from the interface point of view.
Another thing worth to verify is that have you test this in fullscreen mode? I assume there might have potential defects in some cases.


Hi,

Did you compile the DLL yourself?

If so, this is probably because the compilation was set on DEBUG. You'll need to rebuild (if building from source) in `Configuration=Release`.

`127.0.0.1:8888` is a fiddler proxy for localhost to test API calls to Coinbase.


Can you make this private API?


it's not used, NPM just asks for it all when you run `npm init`. Guessing we could delete it without any issues, or just never update it.


@alexyoung Why are you checking for `element.className` first? Are there browsers out there that lack it?


@cheuschober

My code
----------------------------------
 #!/usr/bin/env python
# -*- coding: utf-8 -*-
""" Package and module creation"""

import task_01

BUTTER = True

OIL = False
---------------------------
create init file in task_01 package but any time i import peanut i get this error msg.

>>>import task_01
>>>if task_01.peanut.BUTTER: print 'I am truthy'

Traceback (most recent call last):
  File "<pyshell#18>", line 1, in <module>
    if task_01.peanut.BUTTER: print 'I am truthy'
AttributeError: 'module' object has no attribute 'peanut'

what m i doing wrong?


But the title of the second page is "Freedom of Information Fact Sheets" so being pedantic you'd need to repeat the title.

Anyway, we're not doing this. Being pedantic doesn't help the citizen in this case and that's the most important thing.


Can you please show me your user case which do need this interface?


:+1: yeah that's fine, shouldn't show up either way since we won't be publishing this to NPM


great! :+1:


This could be a great project if build, let me know if any work or thoughts have been given. Would like to contribute in building if possible


前のコミットで微妙だったマクロの名前が良い名前になった。


cái này là demo thử thôi fai ko anh


if package not used, then name -> .SomeMessage, after name[1:] ==> SomeMessage without dot


Also, try using `build.bat` in the root of the repo and it should automatically compile everything in release mode.


Thanks!


not a big deal: i grep'ed `assert` in the source tree. seems that we are putting the constant at the right of the comparison operator, like:
```
common/Mutex.cc:71:  assert(nlock == 0);
common/Mutex.cc:95:  assert(r == 0);
common/Mutex.cc:107:  assert(r == 0);
```

maybe we can follow this convention here also?


We'll soon be handling admin links, which will make editions depend on other editions. When that happens, the association `has_many :edition_dependencies` will be required to fetch dependants, which is the more likely query we'll be making: "I have been updated, so I need to republish my dependants."
```
has_many :edition_dependencies, as: :dependable
has_many :dependant_editions, through: :edition_dependencies, source: :dependant
```
that'll allow us to have:
```
# fetch dependant editions
edition.dependant_editions
contact.dependant_editions
```
The same association is configured in `Contact` already, and reads very well there. That will allow us to extract out a `Module` on the lines of `DependableByEdition` and use that in `Contact` as well as `Edition`.

To avoid the ambiguity, I've chosen to use a scope on the `EditionDependency` model: `contacts` for now. So that we can query as: `edition.dependencies.contacts.count`, or `edition.dependencies.editions.count`.
```
# if we need to fetch dependencies, mostly for testing
edition.dependencies.contacts
edition.dependencies.editions
```

A part of your suggestion will help simplify this for sure, but the last `has_many :contact_dependencies` is not necessary due to the above design.


No, it is real implementation. The _prng_index is initialized at prng.init() (https://github.com/m943040028/optee_os/blob/prng_hal/core/lib/libtomcrypt/src/tee_ltc_provider.c#L2250). ltc has its own prng_descriptor array that describes multiple PRNG implementations and _prng_index is used to choice among them. I should reconsider this if we want to support multiple PRNGs


It looks exactly like it. Thanks.


このコミットに限らず、基本的には一度に一つのことをやるようにしているのでコミットが小さく読みやすい。


引数をそのまま使わずに一旦名前を付けていることで、引数の意味が分かりやすい。


I think it's no need to define the macro `_KMALLOC` and `_KFREE`.  Using `devm_kzalloc` and `devm_kfree` directly is more clear.


Uhhhh good point I have no idea what I was thinking. Thanks!


this import is unused.  it causes plain-jane "git clone thisrepo; cd thisrepo; ant dist" to fail


各行を読み込む処理を、べた書きせずに関数(メソッド)としてまとめているので、メイン処理の流れを把握しやすい。


@normanmaurer I'm not sure. It can be a problem when FD is invalid, but I guess an unconnected FD is not invalid.  Looking from the JNI code, it seems to return -1 in case of failure. Please feel free to test it by yourself, though. Just enable LoggingHandler in the EchoClient example and use the epoll transport to see if `UNREGISTERED` message is logged.


Delete gHMSLContext and hmslColors here.


assign this in the controller


It's better to follow Go naming convention, which uses camelCase way of naming. So, it should be `getConfigFile`, not `get_config_file`.


@trustin will check.


I can change it ... I simply copied what was already there ;)


no need for parens (ruby is awesome!)


im totaly agree with you and your team that you are another fucking leecher of wowsource team who stolen work rename and open for public hahha so go fuck your self trash bad and penis sucker that you are , if you ask how i know is my core and db well penis sucker look at the milions damage from mobs who buff characters and modify haste with valor - it makes houndreats of milions damage http://s9.postimg.org/ot9wfja1b/Wo_WScrn_Shot_030415_095005.jpg check the damage from my characters , also the other thing that it showme is my old world and core is that the npc guids and objects and also duel gable npc spawn around the world , any way people we fix this huge issues in our core and a beter version then this crap leeched last year can be fiind here https://github.com/WoWSource/WoWSource434 so pee this fucking bastards between the eyes and let them suck hairy balls LOOOOL. btw if you wish to contact the real wowsource owner add skype geany_100 or hes forum http://wowsource.info/


Gotta love them comma's


assign params to @variables in controlller


请问 `hc5761/package/xt_salist` 在哪里下载呢？ google里搜 kmod-salist，没有任何结果……难道不是一个开源项目？


This actually worked, so I can just do this in the future


QwtUpdate (macumber) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Interesting. You're right, we don't need the dependable records. If I'm thinking correctly, we have choose between:
(i) making multiple count queries, which `@dependant.where(dependable: dependable).exists?` will make, or
(ii) just fetch all `dependables` in one query.

If there are 5 dependable contacts detected, and 3 already exist, i can't think of a way we'll be able to find the 2 yet-to-be-persisted dependable contacts by checking for presence. Can you help with some pseudo-code @heathd?


look at ruby string interpolation for a nicer way to concatenate strings!


@sshumake yes we had a discussion about this .. as previously we were using '>1.1.14' which was also pulling out latest version of stratajs, so we changed to '*' as we don't have to modify the package.json.

But I agree it is risky in some situations where stratajs is not stable. May be we can follow what you explained here ~. I  will test this out and create PR for this.


no need for parens


The idea here is to make the prng initialization method as common code. As a user of crypto_ops, he doesn't care about the underlying implementation of crypto_ops is openssl or libtomcrypt, and PRNGs each implementation provided. He just needs to know how many PRNG implementations provided from crypto_ops. That's why I don't put prng.init() into crypto_ops.init(). This could be changed to initialize PRNGs in crypto_ops.init(), but provide a query method to tell the user how many PRNGs implementations provided by crypto_ops.


you're looking at an old version, i changed all this


shipit


they arent missing the door node the triggers are just generated behind the collision blocker, you can easily make an offset for the trigger to make all of the doors work


i'd suggest return 1. as by inspecting the other places where radosgw fails it returns 1. and i don't see any docs regarding to the return code in doc/man/8/radosgw.rst. and i can not find any convention in ceph to return the errno as the return code of any of our CLI tools.


因為還沒merge，這邊可以直接回去修正錯誤的commit。


combine select statements


look up ruby concatenation for a nicer way to concatenate strings!


do logic in controller


這邊能用簡化的寫法嗎?例如
<pre>
<%= render "post_control", post: post %>
</pre>
手邊沒有環境，再麻煩試試看。


next time try blank? to look for empty objects


@ayine17, instead of overloading this conversation, could you use my @cheuschober tag to point me to your pull request for the warmup exercises? Also, fyi, hotel wifi at this conference has been terrible so much so that I wouldn't count on a speedy reply (before friday) from me.


dont need this


Minecraft != Minecraft Forge and mods


theres a sum method in rails!


put logic in controller


Yes i compiled the DLL myself. I have tried what you told. Its works like charm. But it failed to create sandbox URL. It returning Response url like "https://coinbase.com/checkouts/3e05cf768985dbe7f9ca9095f78fe4b3" for sandbox wallet. If we manually add "sandbox" term in the URL then its working fine.


另外，partial裡面盡量不要使用 instance variable，建議改成再render的時候傳值進去。


rails has sum method :o:


이 부분의 역할이 뭔가요? 나중에 쓰려고 미리 만들어 놓으신건가요?


aqw


convention: put logic in controllers, assign to variables


QwtUpdate (macumber) - x86_64-MacOS-10.9-clang: Tests Failed

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


Hello!
Welcome to JC Pedjeu's repository


`CMAKE_BUILD_TYPE` will tell you if `cmake` has been called with any specific build type. We set it to `Release` if it isn't explicitly set by the caller.

Concerning the depenency handling, see e.g. https://github.com/kodi-pvr/pvr.iptvsimple/tree/master/depends/common/zlib. Instead of the URL to a tarball you can also provide an URL to a git repository followed by the branch/hash/tag to get. Then you can provide your own `CMakeLists.txt` which will be executed in the downloaded source (which can still be an `externalproject_add()` call.


put logic in controllers


move logic into controller, use instance vars


Since you obvious have no common sense, you might want to take a look at all of the Trinity core source codes posted all over Github, https://github.com/cbunting99?tab=repositories

There was nothing stating in any other them, that they couldn't be used for projects.. Hence the reason why they are open source projects posted on github..

You obviously have little man syndrom cause honestly, I don't care about your 14 year old potty mouth, and I don't give a crap about Wow-source,,,, The biggest con and thief of the Wow Emulator community..

Don't like what I have to say? Where is the DMCA takedown? Show me a trademark or something?


What exactly wasn't working here?


@jbech-linaro, PRNG has its own descriptor table and indexes in ltc, and already has register functions. As I said in preview message, the code here is simply tried to let the crypto_ops user choice PRNG implementation he wants to use. And I think provide another interface for user to query how many PRNGs supported in the system could be a better approach. And we can completely eliminate the code block here.


move logic into controllers, assign to instance vars


can delete this!


The problem with this approach, that travis won't run jasmine on both of the unit tests and example tests. #62 I have to solve this somehow asap.


TeamCity Helios :: Helios PR Build [Build 1](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=157&buildTypeId=Helios_HeliosPrBuild&guest=1) is now running


TeamCity Helios :: Helios PR Build [Build 1](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=157&buildTypeId=Helios_HeliosPrBuild&guest=1) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 00:00:07


I think this is required to boot


Наверное, это уже излишне. Зачем в Loggable метод чтения из StdIn? SRP?


Here is a hint:   element ID is #rock, #paper, or #scissors, right?  You can use `$("#" + computerChoice).show();` as a shortcut so you don't need many `if` statements ;)

Doing good so far!


NAN MAIS OH


Usage of alerts is not recommended


Round corners?, we made everything square why do we have round buttons now?


WTF is going on. I have never heard of this guy and I was the one who wrote these commits.


space missing after ```.```


@jonathanrico check it out


QwtUpdate (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2083 of 2181 tests passed)

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


personally, i think this is an not-that-bad idiom to reuse the (non const) function parameter as a variable. maybe other guys have different idea.


remove the empty space on line 46


I had assumed wrongly that common use case you're mentioning is covered by this index. I've changed the index definition in: https://github.com/alphagov/whitehall/commit/2a706bd. Only the order of the column names in the index needed to be changed to cover the common use case:
> If the table has a multiple-column index, any leftmost prefix of the index can be used by the optimizer to look up rows. For example, if you have a three-column index on (col1, col2, col3), you have indexed search capabilities on (col1), (col1, col2), and (col1, col2, col3).

http://dev.mysql.com/doc/refman/5.0/en/multiple-column-indexes.html


The button class will probably need to be more specified:
.btn  ->  a.btn, button.btn, input[type="submit"].btn, input[type="button"].btn, etc.


The issue is fixed.


as a best practice deal with the special case first then the normal case
```python
if phone is None:
     return ...    #special case
....  #normal case
```


empty lines should not have space characters


might want to simply return 1. as the doc/man/8/radosgw.rst does not document the 5/EIO return code, and i failed to find any convention in ceph tools to use errno as the return code. AFAIK, some other widely used unix command line tools do not return errno either.


Then do it yourself


Because fuck the rules.


Same comment on best coding practice deal with all feedback message first
```python
if not self.collections['participants'].opting_in(invitee_phone):
    .... # send feedback
    return
... # run the invite
```


Hello Everyone,

I have uploaded the access database file in git hub (look in app_data
folder).

Please follow the instruction is the changelog file for the addin you need
to install to run the database.

Thanks,

Joseph

On Mon, Mar 2, 2015 at 9:38 PM, Joseph Balatbat <jbala972@mtroyal.ca> wrote:

> Textfile that says change log
> On Mar 2, 2015 9:25 PM, "jadeclan" <notifications@github.com> wrote:
>
>> Where is the log file? I will commit a very rough start to the about page
>> for everyone to comment on the direction I am planning to take and to get
>> feed back on improving the design.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/abcd134/JJS3512/commit/e9f3638ff5ed51cc695fb11088ae01d3b58d88b2#commitcomment-9999053>
>> .
>>
>

--
Joseph Balatbat


@cheuschober
when run the your example input i don't get True or False. the get the string value in the print

>>> import task_01.peanut
init package
>>> if task_01.peanut.BUTTER: print 'I am truthy'

I am truthy
>>>


this is also duplicated from route.rb


add focus functionality


hello


it's probably not a good idea to redefine these constants, which are already in route.rb


89438532_RadianceBugFix (rpg777) - i386-Windows-7-VisualStudio-12: OK (2198 of 2213 tests passed)

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


Em cai github for windows, dang nhap tai khoan va thu clone repository ewsd-proj vao thu muc bat ky. Nho cai ampps hoac easyphp de khi can chay app thi bo thu muc clone vao.

Sent from Outlook

On Wed, Mar 4, 2015 at 10:23 PM -0800, "namnguyen2014" <notifications@github.com> wrote:
cái này là demo thử thôi fai ko anh

---
Reply to this email directly or view it on GitHub:
https://github.com/tuananhit87/ewsd-proj/commit/83d3155470c2fe5fe03de8a87f4dc5d0de283b0f#commitcomment-10045381


there is no way with ``get`` method to receive more than one object, there will be always one or none


QwtUpdate (macumber) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


the ruby support code has grown over time, and probably could use some refactoring. at some point it would be great to extract an osrm ruby gem that could be used for easily interfacing with osrm servers. then gem could be used in the cucumber support code, as well as other projects.


89438532_RadianceBugFix (rpg777) - Win64-Windows-7-VisualStudio-12: OK (2190 of 2213 tests passed)

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


"%llu" won't compile on MSVC6. It needs "%I64d".

Note that `v7_array_length` is an exported function, would it be a good idea to keep `long` as a return type? We just can compare return value of `v7_array_length()` with 2^32, return -1 on overflow,  and keep the rest of the code unchanged?


almost identical to request_path in route.rb. should be possible to factor our the common parts?


Hi Hunter, thanks for the interest. The easiest thing is to come along to the regular meetings that are advertised on our meetup.com page. Were you looking for something more specific? If so, feel free to email the members of the board: board@fortcollinscreatorhub.org.


:D  Splendid! I think I'll post on Github ( in two weeks maybe ) the work-in-progress version of the episode 06 " The potion challenge" before coloring, in black and white line-art version, with speech-bubbles and text vector SVG ready. A sort of 'beta' version of Pepper&Carrot, two weeks before the release.
I'm thinking a lot on opening the production. It might be a good solution to let translators/correctors give feedbacks about improvement for the story, and translate the webcomics for the release. What do you think about it ?


Well done !


I see this targeted to .NET 4.5 only. Would this patch work also for .NET 4.0?


mem_e_ber ?


We had a long hard chat and it turns out JFact was ready and JFact 4 has been out a while ;-)


update the document 2. wiring guide. Content about change the wires'
color of power plug


@azturner removing this line is keeping the delete from working. Keeping it though causes the GetData to run twice though... Currently though in the v3.0 branch the delete isn't working though.


루비에서는 마지막 문장의 평가 결과를 반환합니다. 저 결과를 반환하면, 트윗 등록 성공시에 twitter 젬의 Tweet 객체가 반환될 텐데 이를 사용하지 않고, 등록된 트윗의 id와 올린 내용(tweet)을 반환해서 필요하면 활용할 수 있도록 해둔 겁니다.


TeamCity Akka.NET :: Akka.NET PR Build [Build 103](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=158&buildTypeId=AkkaNet_AkkaNetPrBuild&guest=1) is now running


Ah, sorry my bad. Right! You could also have used another condition inside the Prerequisite


To have the small time-indicator added changes in the time-ago settings: http://stackoverflow.com/a/4120896


develop (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1141 of 1141 tests passed)

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


Оказались смешанными вывод и вычисления среднего и прочего.


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

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


这个版本中有错误！


rgw_main::main() already returns error codes as e.g. ENOSYS or EINVAL.


Is chrome never in 'Program Files', but always in the home directory in windows?


Original commit is here https://github.com/VanirExodus/packages_apps_Settings/commit/8c0d7cb4eaba72b487cc1d1c250bb3b7bc7ba8be

In my defense that is not where I found this


Ok so this value is extremely fragile IMO.

Can you try setting `display: block` for bdi?


develop (jasondegraw) - 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)


why had you added this


Статистику собирает game. Может быть, рассмотреть вариант с событием плохого выстрела, чтобы желающий мог подписаться и инкрементить счётчик, например.


It seems to be not very common in the code. If I didn't oversee something this was the only case. At least it's bad style.


My B


This either needs modification of the matcher or transition to all lower-case, as the matcher does not accept camelcase from the elvis.syn

As merged, this won't work. Sadly


yeah, you are right, i just saw them! so could you use the constant errno? like
```
return EIO;
```
?


Can't believe I missed that, fixed in: https://github.com/alphagov/whitehall/commit/f41bab4.


Cool


Зачем сделал определение имени свойства на уровне переменной, а не на классе? НА уровне переменной делают для того, чтобы в последующем достать их по имени, когда компонент редактируется


metatype=false


Снова чуть-чуть небрежности. Файл помечен как изменённый, хотя в нём всего-навсего изменились табуляции. Но, в итоге, читаешь описание коммита, видишь список изменённых файлов и удивиляешь, как это изменение должно было затронуть этот файл.


Если это уже не фрагмент - поменяй имя на бандл


OK, I've reverted your patch for now.


Looks like you misspelled "canola"!


Money logic.


I'm assuming you wanted `limit` to equal `limit` here, coming in the next push.


TeamCity Akka.NET :: Akka.NET PR Build [Build 103](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=158&buildTypeId=AkkaNet_AkkaNetPrBuild&guest=1) outcome was **FAILURE**
Summary: System.Exception: xUnit failed for the following assemblies: D:\BuildAgent\work\49b164d63843fb4\src\core\Akka.Persistence.Tests\bin\Release\Akka.Persistence.Tests.dll   at Microsoft.FSharp.Core.Operators.FailWith[T](String message)   at Fake.XUnitHel... Build time: 00:10:16
### Failed tests
```
Akka.Persistence.Tests.dll: Akka.Persistence.Tests.GuaranteedDeliveryCrashSpec.GuaranteedDelivery_should_not_send_when_actor_crashes: <no details avaliable>

```


In my opinion these kinds of comments are superfluous, especially for cases of method length < threshold


Well, yes, but then you can't choose between CLI or GUI installation. However, to tell you the truth, I'd like to see the CLI installer disappear for good, so I'm OK with your changes  :grimacing:


Yes I will change it ...


Зачем тут сервис?


Thanks, now it's translated.

2015-03-04 18:20 GMT+01:00 Ivan Kwiatkowski <notifications@github.com>:

> Okay, it's a problem on my end. My IDE didn't automatically regenerate the
> resources when I created the new APK. I have updated the APKs, so this
> should be fixed.
> Thanks for reporting this!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/JusticeRage/ApkTrack/commit/c517ba859150ba6194fcb34b0ae62dc7c636cfe2#commitcomment-10034364>
> .
>


Don't recreate the repo.


Не понял махинаций, почему метод апдейт сразу не обозначить двумя аннотациями?


i fix the system.cpp like this, after ```make``` and ```make install```  ,but there is not ```python``` file in the ```/usr/local/lib```  directory


Okay, I can dig it!


develop (axelstudios) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2188 of 2213 tests passed)

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


Just want to note that `self.options.get(constants.EMBED_GEOMETRY)` currently seems to return `None`, leading to the following export:

```json
"geometries": [{ "type": "BufferGeometry",
  "attributes": { ... }
}]
```

instead of

```json
"geometries": [{ "type": "BufferGeometry",
  "data": {
    "attributes": { ... }
  }
}]
```


OK. Agreed. :+1:


Все ошибки пишем в лог, икаких принт стек трейсов


Do these tests actually run?


Could you copy the @DelegatesTo annotation from FileTreeBuilder#call ?


Yeah, this definitely should not be there. I can fix that or you submit a patch. Let me also point out that I have no idea if anybody is actually using this library. There are several good idea in here - some things I would change with respect to functional interfaces and defaults in interfaces but I am not really working with it - what shouldn't stop you but at least you should know about it.


BTW, if you won't mind to polish on the merge: https://travis-ci.org/spring-projects/spring-integration/builds/53088122
Looks like one more Java 8 forgiveness. We can't rethrow just `Exception` if we don't have `throws` on the method. So, we should do the same what Juergen did for the parent class - `catch (RuntimeException ex) {`


All looks good. Nice to be getting some of this cruft removed


Fixed


Actually, I forgot to remove the limit stuff cause then it messes up the pagination.  For now this just moved the previous/next buttons down via css


Error “vector iterator + offset out of range” occurs when I call pow function:
	bigint a = 159753;
        cout << a.pow(6) << endl;
There may be some error in operator * overloadng:     bigint bigint::operator*(bigint const &b)


Раз мы используем CQ как платформу, используй дополнительную конфигурация сервиса логирования для осуществления записей в лог. без своих надстроек


develop (jasondegraw) - x86_64-MacOS-10.9-clang: OK (1138 of 1138 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)


ну ты даешь)) зачем всем внутренним объектам делать такой же модификатор как родителю?)) такой модификатор нужны только родителю, все внутренние блоки остаются как есть


right now this filter expects 1 id. it should expect multiple, in some cases. and in that case it should be like:
as t WHERE ogc_fid IN


Do you want a new pull-request?