and you are sure that the default is different than applied in my commit?

On 06.10.2015 18:22, Tobias Preuss wrote:
> I use the default formatting settings of Android Studio. I set up a [macro on save](http://stackoverflow.com/a/5581992/356895).
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/criticalmaps/criticalmaps-android/commit/6bf9ee50001aa14772d19c7ffb133393ab1fb3db#commitcomment-13619128
>

--
best regards,
ligi
http://ligi.de


tienes onclick=clickCorto en el xml?


Looks like we don't even need this commit anymore, now that we're just adding a tx and then trimming after.  (`git grep` confirms this functions isn't used)


掃了一下英文原文，所有的 <clan> 都接著 clan，
我會改成 1. 的版本跟匪幫。


Do you have check the performance gap for this change ?


`$("#container").append("<%= escape_javascript(render("post.html", post: @post)) %>");` will not work this way.

It would be required to do something like

    <% {:safe, response} = escape_javascript(render("post.html", post: @post)) %>
    <%= $("#container").append("<%= response %>");

Is it necessary and normal or it's not and there's a way to avoid it?


si, e non fai nada


you must kidding me, i just install the old versions⋯⋯ and failed


```
+A first big study on smuggling was done in 2005 and lead to several corrections,
+ this was a study by
```
+A first big study on smuggling was done in 2005 and lead to several corrections.
This was done by


Hi thanks for the attempted fix, unfortunately it doesn't seem to have worked - master branch, 638270689679f3958d1b593c329e7c6e5cb7448e:
```
2015-10-06 17:26:06 INFO     SEARCHQUEUE-MANUAL-279121 :: [KickAssTorrents] :: Performing episode search for The Flash (2014)
2015-10-06 17:26:06 DEBUG    SEARCHQUEUE-MANUAL-279121 :: [KickAssTorrents] :: search_strings = [{'Episode': [u'The.Flash.2014 S02E01|2x01']}]
2015-10-06 17:26:06 DEBUG    SEARCHQUEUE-MANUAL-279121 :: [KickAssTorrents] :: Search string: {'q': u'The.Flash.2014 S02E01|2x01', 'field': 'seeders', 'category': 'tv', 'sorder': 'desc', 'rss': 1}
2015-10-06 17:26:06 DEBUG    SEARCHQUEUE-MANUAL-279121 :: [KickAssTorrents] :: Proxified URL: http://getprivate.eu/browse.php?u=https%3A%2F%2Fkat.cr%2Fusearch%2F&b=32&f=norefer
2015-10-06 17:26:11 ERROR    SEARCHQUEUE-MANUAL-279121 :: [KickAssTorrents] :: Failed to parsing KickAssTorrents Traceback: Traceback (most recent call last):
  File "/opt/sickbeard/sickbeard/providers/kat.py", line 138, in _doSearch
    data = xmltodict.parse(data)
  File "/opt/sickbeard/lib/xmltodict.py", line 248, in parse
    parser.Parse(xml_input, True)
ExpatError: undefined entity: line 5, column 26
Traceback (most recent call last):
  File "/opt/sickbeard/sickbeard/providers/kat.py", line 138, in _doSearch
    data = xmltodict.parse(data)
  File "/opt/sickbeard/lib/xmltodict.py", line 248, in parse
    parser.Parse(xml_input, True)
ExpatError: undefined entity: line 5, column 26
```

BTW, I keep having to `rm -fr cache/mako` each time I restart SickRage after an update - maybe this should happen automatically when updating SR?


Ok!!


Is it just the records slice that is affected or should we be initializing the whole nextResult struct?


This isn't necessary anymore with the built-in ForestDB encryption API. CBForest is no longer using `CBFOREST_ENCRYPTION` anywhere.


This can be a `Context` type instead, since that's all we need.


Reverse proxies, SSL Terminators, Load Balancers, etc


I think the 'correct' way is to also call Search::init() when this will be changed 'on the fly'.
```
void on_threads(const Option&) { Threads.read_uci_options(); Search::init(); }
```


Not sure I understand.
In https://gcc.gnu.org/onlinedocs/gcc-4.7.2/gcc/Type-Attributes.html#Type-Attributes;
`packed`
 `   This attribute, attached to struct or union type definition, specifies that each member (other than zero-width bitfields) of the structure or union is placed to minimize the memory required. When attached to an enum definition, it indicates that the smallest integral type should be used.`

and perhaps this is connected with the problem:
`You may only specify this attribute on the definition of an enum, struct or union, not on a typedef which does not also define the enumerated type, structure or union.`


аыфвафыавф


The only really practical advantage I see to using `class` in the router is that it forces us to think about what our API looks like without mixins. If we have good solutions for using `class`, we'll always have an answer for people using `createClass`.


Hm, forget that, it must at least re-raise internal SWF errors like `ExecutionBlocked` at first sight, else the decision tasks are never taken. I'll dig into that unless you have some immediate idea about optional activities not working :) (this is the faulty commit btw, I bisected the problem to this on a dummy workflow)


súbelo a github y lo miro a ver, porque ese método está bien


FWIW, NetBSD bsd.prog.mk v 1.1.1.1 (which is what Ubuntu 14.04 uses) contains

```
.if !empty(SRCS:N*.h:N*.sh:N*.fth)
OBJS+=          ${SRCS:N*.h:N*.sh:N*.fth:R:S/$/.o/g}
LOBJS+=         ${LSRCS:.c=.ln} ${SRCS:M*.c:.c=.ln}
.endif
```

but I'd argue that explicitly including that dependency isn't going to hurt.


wew


no "sync prefs" button for me in the 'newwindow' it is there in the original window


@MilhouseVH try and disable the proxy


Hmm are you sure about that?? The function seems to reverse the endianess here.
My 'head' tells me you're right, and the AVR is little endian, but... that code worked before so...

puzzling! :-)


Выводит второй по величине элемент набора чисел


In fact the `Context` can be retrieved from the `TextView` via the `getContext()`, so this parameter can just be removed I guess.


@duramato he cant, kat is blocked for him. We need to implement the custom urls


Yes. I double checked it here: `Settings > Editor > Code Style` ... `Scheme: Default`.


'return' is not a function, it doesn't need parenthesis...


`class` components have better performance, too


, and protocol


<html>
<head>
	<title>Javascript Box - OOP demo</title>
</head>
<body>

	<svg id="svg" xmlns="http://www.w3.org/2000/svg" width="100%" height="100%"></svg>

	<script>
	function Circle(cx, cy, html_id)
	{
		var html_id = html_id;
		this.info = { cx: cx,  cy: cy };

		//private function that generates a random number
		var randomNumberBetween = function(min, max){
			return Math.random()*(max-min) + min;
		}

		this.initialize = function(){
			//give a random velocity for the circle
			this.info.velocity = {
				x: randomNumberBetween(-3,3),
				y: randomNumberBetween(-3,3)
			}

			//create a circle
			var circle = makeSVG('circle',
				{ 	cx: this.info.cx,
				  	cy: this.info.cy,
				  	r:  10,
				  	id: html_id,
				  	style: "fill: black"
				});

			document.getElementById('svg').appendChild(circle);
		}

		this.update = function(time){
			var el = document.getElementById(html_id);

			//see if the circle is going outside the browser. if it is, reverse the velocity
			if( this.info.cx > document.body.clientWidth || this.info.cx < 0)
			{
				this.info.velocity.x = this.info.velocity.x * -1;
			}
			if( this.info.cy > document.body.clientHeight || this.info.cy < 0)
			{
				this.info.velocity.y = this.info.velocity.y * -1;
			}

			this.info.cx = this.info.cx + this.info.velocity.x*time;
			this.info.cy = this.info.cy + this.info.velocity.y*time;

			el.setAttribute("cx", this.info.cx);
			el.setAttribute("cy", this.info.cy);
		}

		//creates the SVG element and returns it
		var makeSVG = function(tag, attrs) {
	        var el= document.createElementNS('http://www.w3.org/2000/svg', tag);
	        for (var k in attrs)
	        {
	            el.setAttribute(k, attrs[k]);
	        }
	        return el;
	    }

	    this.initialize();
	}

	function PlayGround()
	{
		var counter = 0;  //counts the number of circles created
		var circles = [ ]; //array that will hold all the circles created in the app

		//a loop that updates the circle's position on the screen
		this.loop = function(){
			for(circle in circles)
			{
				circles[circle].update(1);
			}
		}

		this.createNewCircle = function(x,y){
			var new_circle = new Circle(x,y,counter++);
			circles.push(new_circle);
			// console.log('created a new circle!', new_circle);
		}

		//create one circle when the game starts
		this.createNewCircle(document.body.clientWidth/2, document.body.clientHeight/2);
	}

	var playground = new PlayGround();
	setInterval(playground.loop, 15);

	document.onclick = function(e) {
		playground.createNewCircle(e.x,e.y);
	}

	</script>

</body>
</html>


Not necessary: `C4SliceResult` is just a typedef of `C4Slice`.


Guys if hypervisor field set as "Automatic allocation" then it choose first one in pool

here is example below.
      if args[:hypervisor_host]
		host  = client.hosts.find { |host| host.name == args[:hypervisor_host] }
		logger.info "create_vm_from_builtin: host : #{ hypervisor_host }"
	  elsif
		host  = client.hosts.first
		logger.info "create_vm_from_builtin: host : #{ host }"
	  end


3 kinds of attacks


@ljbade Can you help me understand what this annotation does?  Does it add information to the generated JavaDoc or is it just an IDE warning?


of attack


Well, its entirelly possible the debug traces were wrong, since the real code definitely decodes it properly...


Возвращает n-ый по величине элемент набора чисел ( метод selectNMax)


This has broken RMG. Please fix. Thanks


the text should be "This system is currently not registered"


missing comma?


This is required by OpenShift.


User? I think this should be `Non-official`.


Разложение числа на набор простых множителей ( метод primeFactorizationFrom)


You realize that this is exactly the same as `return before == after;`, right?


you *can* add them in the request


RLE кодирование строки (метод runLengthEncoding)


RIP commit title, forgot my coffee this morning.


1) The unit test cases for checkDate(), checkTask() and generateNextDate() have been done.
checkDate() returns a boolean value.
checkTask() returns a boolean value.
generateNextDate() returns a String object

2) assert statements and logging statements have been inserted into Flex.java, as required for the project.
(Even though at this stage, they are truly redundant.)

3) I have made Flex Display such that:
1. Whenever it displays error messages due to an invalid user input command, the Command Line Prompt will do the same.

2. If the user input command is valid, and it specifically requests for Tasks to be displayed (This means "searching", and "display" or "showing"), the Command Line Prompt will also display messages to notify the user of the valid input, and/or the action done.

3. If the user input command is valid, but it does NOT specifically requet for Tasks to be displayed (This means "add", "delete", "change" and "undo"), the same messages will be displayed on both Command Line Prompt, and Flex Display, to notify the user of the valid input, and/or action done.

As far as possible, all display messages have been converted into "non-magic" Strings, for refactoring purposes, as required by the project's code quality standard(s).

4) changeTaskVariable() now makes sure that starting times are not later than ending times, and ending times are not earlier than starting times. I am allowing starting times to be the same as ending  times (This may be the case of the user attempting to indicate a "deadline" task, in my opinion.)


Do we still use avr_timer_cancel_all_cycle_timers by itself? If not, what about renaming the function to something shorter? I know I'm to blame originally, but it's getting a bit ridiculous now :-)


Would a suggested layout of a workshop file/folder wise make sense to put here too?


Deliver enhancement #73


Является ли слово полиндромом (метод isPolindrom)


We should probably just use the default as it follows the Google java style
guide which probably most people are used to and use. But I think that was
probably your intention to begin with.
On Oct 6, 2015 6:30 PM, "Tobias Preuss" <notifications@github.com> wrote:

> Yes. I double checked it here: Settings > Editor > Code Style ... Scheme:
> Default.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/criticalmaps/criticalmaps-android/commit/6bf9ee50001aa14772d19c7ffb133393ab1fb3db#commitcomment-13619290>
> .
>


Hi, is OTA still broken?


done.


advanced mode the keep-aliv => advanced mode, keep-aliv


Ah, I hadn't realized that.. In that case I'm fine with just dropping it, yeah.


Instead of abbreviating "text" to "txt", you can abbreviate "Resource" instead, making it `textResId`.


@wmertens: Funny you mention this now. Just a few days ago, I hit this again while setting up a new Jenkins instance :-)


I fixed this error on our fork, using  https://pythonhosted.org/kitchen/api-text-converters.html#kitchen.text.converters.getwriter. I'd recommend adding a second test so that you cover both `byte-string` type containing unicode and `unicode` type containing unicode, because the handling is different.

Unfortunately I can't push my patches easily because too much has diverged.


Indeed, kat.cr is blocked in the UK.


I am looking at these as a way to customize docker per OS.  IE To plug into the architecture that it uses.  I agree docker might want to allow specification at docker run time, but I don't see why this should not be allowed as a mechanism for people who are hounding docker for advanced features to have a true plugin architecture.


of requests


Students/30425/Popa T. Florin/Largest palindrome/palin.java

use `boolean`, we're in Java, not C.


and waiting => waiting


Sorry Taapat,

It's all working now . butt I really do not understand You're remaks.
Therefore, I think it is wrong that in one place Hotplug is without try. ???

What do You mean, If a potential problem is present it really must be solved.


@NixFifty I'm not that happy with this work around, but for some reason logException fails here, even if I can use a datawriter here. I'm left scratching my head, but I want to at least have a work around until I better understand why this is failing.


RLE декодирование строки (метод runLengthDecoding();)


Finally => Eventually


This parameter should probably be moved before the `textView`.


Students/30425/Popa T. Florin/Sum of multiples/sum.java

This can be done a lot more elegantly. Check it out [here](http://math.stackexchange.com/questions/9259/find-the-sum-of-all-the-multiples-of-3-or-5-below-1000)


@fxlb Perhaps a connection?

None AFAICS.


proxies


№1


Please see my post at:
https://groups.google.com/forum/?fromgroups=#!topic/fishcooking/8JBu7DQXIEA


№2


lgtm


Находит строку в подстроке (метод Find(); у класса Task5)


responses


Please see my post at
https://groups.google.com/forum/?fromgroups=#!topic/fishcooking/8JBu7DQXIEA


reading or parsing the query


I'd rather not have this empty line


Is the URL being "proxified" malformed? `https://kat.cr/usearch/&b=32`


"Updated Comments." is what was meant to be written in the commit message.


ends. This way, the agent can check if what's coming after is another query


@gvanem
Could you try this patch?
```patch
diff --git a/print-lisp.c b/print-lisp.c
index 6b5b815..6b0cb6f 100644
--- a/print-lisp.c
+++ b/print-lisp.c
@@ -185,7 +185,7 @@ static const struct tok lisp_loc_flag[] = {
        { 0, NULL }
 };

-typedef struct map_register_hdr {
+struct map_register_hdr {
        uint8_t type_and_flag;
        uint8_t reserved;
        uint8_t reserved_and_flag2;
@@ -193,11 +193,13 @@ typedef struct map_register_hdr {
        uint64_t nonce;
        uint16_t key_id;
        uint16_t auth_data_len;
-} UNALIGNED lisp_map_register_hdr;
+} UNALIGNED;
+
+typedef struct map_register_hdr lisp_map_register_hdr;

 #define MAP_REGISTER_HDR_LEN sizeof(lisp_map_register_hdr)

-typedef struct map_register_eid {
+struct map_register_eid {
        uint32_t ttl;
        uint8_t locator_count;
        uint8_t eid_prefix_mask_length;
@@ -206,18 +208,22 @@ typedef struct map_register_eid {
        uint8_t reserved_version_hi;
        uint8_t version_low;
        uint16_t eid_prefix_afi;
-} UNALIGNED lisp_map_register_eid;
+} UNALIGNED;
+
+typedef struct map_register_eid lisp_map_register_eid;

 #define MAP_REGISTER_EID_LEN sizeof(lisp_map_register_eid)

-typedef struct map_register_loc {
+struct map_register_loc {
        uint8_t priority;
        uint8_t weight;
        uint8_t m_priority;
        uint8_t m_weight;
        uint16_t unused_and_flag;
        uint16_t locator_afi;
-} UNALIGNED lisp_map_register_loc;
+} UNALIGNED;
+
+typedef struct map_register_loc lisp_map_register_loc;

 #define MAP_REGISTER_LOC_LEN sizeof(lisp_map_register_loc)

```


Yeah I appeared to pushed the wrong version. Should be fixed now


OK, I've introduced those types, and made the LISP dissector use them, in 1376682ba5650750600966ba910774ba26973b92.


My two cents:

String interpolation compiles to `string.Format` while concatenation translates to `string.Concat`. Concatenation *is* faster, so I would hesitate to start using string interpolation as a all-up replacement.


That class sound really interesting.

I didn't mention it in my response, but I totally agree with you. I don't think they were created from scratch either, so I'm still left wondering exactly what she meant by that comment.


The `if` test isn't necessary — `delete NULL` is a no-op in C++.


actor => composants


Seems like it'd be a good one to backport to 2.4.x


Interestingly I have a Default(1) and Default(2) there - would take whatever you want - I do not really care for this detail ..


`ReadConsole` 32768 fail, 32767 ok


Naming it `c4key_newReader` would be clearer.


please remove this line or document the return value


Your PR note says Julia 0.4 only but this suggests support for 0.5-. So the way to do this to *not* support 0.5- is to do `julia 0.4- 0.5-`


Sorry, I'm not used to update both setup.py and requirements.txt. Besides that, the development branch is not the branch you should use except you want to contribute.


I'd prefer a newline right here, just a suggestion though. I find it better readable.


So far we don't need to reinitialize the entire object, the other primitives seem to be behaving correctly and are overwritten. The Done, NextRecordsUrl, and TotalSize are being changed correctly by the unmashalling.

If you'd prefer I'm happy to adjust and reinitialize the entire object, I can't see it hurting because we're expecting it to be overwritten anyway.


Probably eventually, but for now I'd leave it as is. We'll probably end up eventually writing a guide for building workshops.


Vote for default settings by now. In the long term there should be a configuration file included in the repository which can be loaded in the IDE and integrated in the build process. IDE settings have been changed by Google in the past which caused reformatting a lot.


My argument for using the current default is preventing accidental changes
and making it easier for new contributors.
Otherwise I don't really care either. ;)
On Oct 6, 2015 6:42 PM, "ligi" <notifications@github.com> wrote:

> Interestingly I have a Default(1) and Default(2) there - would take
> whatever you want - I do not really care for this detail ..
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/criticalmaps/criticalmaps-android/commit/6bf9ee50001aa14772d19c7ffb133393ab1fb3db#commitcomment-13619556>
> .
>


some kind of


`/etc/gitattributes` won't be picked up if the prefix is `/mingw64/`...

But I have a better idea: how about integrating the `gitattributes` as well as both `.suggested` into `git-extra` proper?


Guys, this is a public notifications thing. Be careful with what you post.

On Tue, Oct 6, 2015 at 10:49 AM, Wayez <notifications@github.com> wrote:

> </3
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mks65/stringy/commit/4b63c93d109b7aadfab8b2de5b03bd48b5b10c40#commitcomment-13616712>
> .
>

--
Samuel Zhang
samuelzhang3@gmail.com


For anyone who reads this, getprivate is failing with it, proxite seems it might work.


we should add some logic here to actually uncheck some engines (to make sure that works) and in addition we should check that the event later on sends the declined engines


Thanks for the updated binaries.


please bump or move the 0.0.2 tag to this, and change the gemfile to point at it (see changes made before the revert)


we can do it in a different PR


I've been meaning to add couple other small things before the next release.

Sent from my iPhone

> On 06 Oct 2015, at 02:45, XhmikosR <notifications@github.com> wrote:
>
> @kangax: do you plan to release 0.8.1?
>
> —
> Reply to this email directly or view it on GitHub.
>


You might want to move the color into colors.xml.


`{:safe, _}` should not be required. If it is required, we need to fix something somewhere or set up some configuration.


Does this happen only with IE9? What about IE10/11 or Edge?


Why are you calling `${SHELL} path/to/run.sh` instead of using a shebang in the script and simply calling `path/to/run.sh`? If the latter was used, we could simply force the required shell in a per-use-case.

Also, at the time you shared that script I thought about this, but forgot to comment:
Are you sure that is worth your effort? From my (limited) experience, converting a regular expression parser into regex C parser is not that hard. Are we expecting so many conversions?
Also, while writing the C code the developer is forced to write test units to guarantee that the regular expressions are correctly implemented and if a converter is available a developer might think that is just a matter of generating the equivalent C code. Obviously, we would block such PR...
At the moment we have 4 files in optlib, 3 of those are regex based, which aren't that many.

Obviously, I will no oppose to the addition of the translator. _I'm just sayin'_... ;)


Also, this .... query to success.


@sbeaver-netgate Not quite properly - now if you edit a rule with a "Single host or alias" type, the rule edit screen shows it as "network" type.
I re-jigged the test in https://github.com/pfsense/pfsense/pull/1949 - I believe that works, see if you like it.


At school right now, but perhaps this should go in www/api/? just want to keep it seperate from root so root can have the web interface


SGTM.


Since you're already taking the `isTopicClosed` parameter here, if you also add the texts for both states, you can apply all the conditional logic here. The icon parameter can be removed then, since both states have a fixed icon.


Maybe change this from negative to positive. So audio = false when there is no audio available.


As far as l know, legacy_mineral isn't used, you can remove it.

your avatar looks similar to the one of t4im
https://github.com/t4im


Well, I got this error when I tried to do it without the first line

    (Protocol.UndefinedError) protocol String.Chars not implemented for {:safe, "<article ...

so it's either I got something wrong again or there's a fix required


need to be reformulated


done.


done


did some general rewording.


use AE to be HTML 5 compliant:
allows us to use <div data-read-more-button> instead of <read-more-button>
(also, the current setup with E breaks the view, so if we keep the change we need to change the html too (not big)


@aviadhahami
not mimicing a fade effect.. this is a fade effect with css transition


That's the point of the "Alternative" template. You can return "Alternative<std::vector<int>, void*>"

Alternatively I could add the concept of an "Option" which would act like an Alternative with "None" as the right hand type.


You can set the fields in `CourseDiscussionResponsesFragment` to private as well I guess.


This seems to ignore --local-patches ?


(print keys) is debug code?? Should that be removed?


+On a real life attack the at => In real life, the attacker


ok i see.
problem is that my drivers/char/random.c
is merged up to 3.16.y
and going back... is a big mess...
i will skip this stage for now :)
you are welcome to merge my state and forget frandom.


This wasn't in the rebase?


Latest version is broken for devs using non-ROMHUT sites. I'm trying to roll back to a working version before this breakage. There is currently no updates by the app dev.


Yeah, therefore my comment on generalizing the color definitions. It
doesn't really make sense to define another color that is black. Neither
does it to reuse one of the existing ones as the name doesn't match. IMHO
we should define a consistent set of a few general text styles and use a
defined palette of colors for these.
On Oct 6, 2015 6:47 PM, "Tobias Preuss" <notifications@github.com> wrote:

> You might want to move the color into colors.xml.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cbalster/criticalmaps-android/commit/34a5bcbe389ca52a8009e3f43cba0d2342764720#commitcomment-13619672>
> .
>


@acdlite :+1:


trasmi => transmi


Request


Just look at it as a quick fix for the broken text and let's worry about
that later ;)
On Oct 6, 2015 6:54 PM, "Christian Balster" <christian.balster@gmail.com>
wrote:

> Yeah, therefore my comment on generalizing the color definitions. It
> doesn't really make sense to define another color that is black. Neither
> does it to reuse one of the existing ones as the name doesn't match. IMHO
> we should define a consistent set of a few general text styles and use a
> defined palette of colors for these.
> On Oct 6, 2015 6:47 PM, "Tobias Preuss" <notifications@github.com> wrote:
>
>> You might want to move the color into colors.xml.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/cbalster/criticalmaps-android/commit/34a5bcbe389ca52a8009e3f43cba0d2342764720#commitcomment-13619672>
>> .
>>
>


@Arthur2e5 更确切的是“减少噪点”，当然，“降噪”也可以。在描述算法的时候（“去除噪声峰值的方法”）可以用“噪声”一词，没问题。具体术语应用参见《数字信号处理》、《数字图像处理》等。


Merged as 4ab863dc45f3e3055a9d9497bbeaf1400ebe269a after I fixed all the syntax errors.


quest body you would use several layer =quest body, you could use several layer


What does this do?


The fix works but it is not yet committed to the version on PyPI repository. As a result pip installing pycrypto on Windows platforms still rises https://bugs.launchpad.net/pycrypto/+bug/1118255.


What is this supposed to do?


Like The Highlander!

![http://i.imgur.com/gZvuqCO.png](http://i.imgur.com/gZvuqCO.png)


Not only that, but even when I added patches/master-patches I still got:

Patches branch patches/master-patches not found. Running in --bump-only mode.


The string should also be changed to "discussion_topic_closed" I guess.


yes it only happen on IE9.  IE10 and IE11 it is working correctly.


filter trying to decode the request body you would use several layers
of content-Transfer encoding
=>
filter trying to decode the request body, you could use several layers
of content-Transfer encoding


That makes sense then. :)

Out of curiosity, why is Exec closing them?  I wouldn't expect that to be the case (hence my original comment).  Also, I only see where stdin gets closed.  That's only done to send the EOF for the sake of the websocket, right?


But we already had UNALIGNED structures in other decoders without visible problem ???
```
$ git grep '\<UNALIGNED\>'
ip.h:} UNALIGNED;
ip.h:} UNALIGNED;
ip6.h:} UNALIGNED;
ip6.h:} UNALIGNED;
ip6.h:} UNALIGNED;
ip6.h:} UNALIGNED;
ip6.h:} UNALIGNED;
ip6.h:} UNALIGNED;
ip6.h:} UNALIGNED;
netdissect-stdinc.h:#if !(defined(_MSC_VER) && defined(UNALIGNED))
netdissect-stdinc.h:#undef UNALIGNED
netdissect-stdinc.h:#define UNALIGNED   __attribute__((packed))
print-bootp.c:} UNALIGNED;
print-bootp.c:} UNALIGNED;
print-dccp.c:} UNALIGNED;
print-dccp.c:} UNALIGNED;
print-dccp.c:} UNALIGNED;
print-dccp.c:} UNALIGNED;
print-dccp.c:} UNALIGNED;
print-lisp.c:} UNALIGNED;
print-lisp.c:} UNALIGNED;
print-lisp.c:} UNALIGNED;
tcp.h:} UNALIGNED;
```


光照类型


You need to add this to your config I believe:

```elixir
      config :phoenix, :format_encoders,
        js: Phoenix.HTML.Engine
```

@chrismccord should we add it to Phoenix by default?


Now the user inline editing is working fine :) :+1:


It's interesting that you're discussing the topic of languages, ideas and their origins. Isn't that the one superpower that all humans share? The power of imagination! Yeah, it seems a little bit far-fetched, but I think Haas really is pulling in these concepts with anything her Vgy mentality. We can see the gray fog before us and truly believe that there is a finish line at the end.

Brontes and Tolkien, Haas and Vgy, these are all authors and writers who have created their own stories. It's why Haas is willing to admit that her 'history' of Andrew could be wrong. We imagine tools, we theorize signs and we convince ourselves that we see patterns. We can be completely wrong, but at least it's there. I think that's wholly original. Cultural Influences is not the end-all statement. Our thoughts are our own.


1) The unit test cases for checkDate(), checkTask() and generateNextDate() have been done.
checkDate() returns a boolean value.
checkTask() returns a boolean value.
generateNextDate() returns a String object

2) assert statements and logging statements have been inserted into Flex.java, as required for the project.
(Even though at this stage, they are truly redundant.)

3) I have made Flex Display such that:
1. Whenever it displays error messages due to an invalid user input command, the Command Line Prompt will do the same.

2. If the user input command is valid, and it specifically requests for Tasks to be displayed (This means "searching", and "display" or "showing"), the Command Line Prompt will also display messages to notify the user of the valid input, and/or the action done.

3. If the user input command is valid, but it does NOT specifically request for Tasks to be displayed (This means "add", "delete", "change" and "undo"), the same messages will be displayed on both Command Line Prompt, and Flex Display, to notify the user of the valid input, and/or action done.

As far as possible, all display messages have been converted into "non-magic" Strings, for refactoring purposes, as required by the project's code quality standard(s).

4). changeTaskVariable() now makes sure that starting times are not later than ending times, and ending times are not earlier than starting times. I am allowing starting times to be the same as ending  times (This may be the case of the user attempting to indicate a "deadline" task, in my opinion.) Do take note that the validity of the new starting time or new ending time has been checked using checkTask(), by first setting the corresponding variable in the Task object to the new term/time. Any invalidity is countered by setting the corresponding  variable back to the original variable.

5. Corrected errors in changeTaskVariable(). Previously, if the variable of a Task to be changed was specified correctly, but no Tasks match the (matching or non-matching)valid date and the non-matching title provided by the user, the output would say that a change has been done, even though "show all" proves otherwise. This has been corrected with a boolean value in changeTaskVariable().

6. Some messages in changeTaskVariable() were only displayed in Command Line Prompt, but not Flex Display. This has been corrected.


allows


did you discuss with @popcornmix?


TeamCity Colore :: Main Build [Build 175](http://tc.sharpblade.net/viewLog.html?buildId=723&buildTypeId=colore_mainbuild) is now running


Na disponibilidade, otimizarei o mesmo.


i don't know about that


@guy and made the LISP dissector use them, in 1376682.

Ok, the `lisp_*` tests now works.

But I'd prefer the `#pragma pack(push,1)` syntax. GCC has supported that since forever. v2.95 I think (as opposed to the butt-ugly `__attribute__((xx))` syntax).


The result of `comment.getChildren().isEmpty()` could be stored in a final variable to be reused in the listener as an optimization. It's up to you though.


First _git_ commit. I have history going 4 years further back in monotone and mercurial repos, but it's terribly embarrassing :)


The only dumb mistakes are the ones that are found but never corrected.


discover
code seems still unchecked =code still need to be fixed.


Well it's very simple, the only problem is that inspektor is not easily configurable. We need per-project granularity (something like `.inspectrc`) file which overrides the default. I thought about it ~year ago, but had no time to actually do it...


TeamCity Colore :: Main Build [Build 175](http://tc.sharpblade.net/viewLog.html?buildId=723&buildTypeId=colore_mainbuild) outcome was **SUCCESS**
Summary: Tests passed: 111 Build time: 0:2:27


(it was the dictionary definition of "hobby project": spare time, here and there, weeks or months between commits ... not worth browsing)


TeamCity Colore :: Main Build [Build 176](http://tc.sharpblade.net/viewLog.html?buildId=724&buildTypeId=colore_mainbuild) is now running


content


+1
Tested and it is working fine


Yes, that's true; otherwise, it will behave as a duplicate for the ``skipAllTestsOnFailure`` flag.


is very strong at


NEEDS Doxygen


These should be numeric positions, not strings.


matlab


TeamCity Colore :: Main Build [Build 176](http://tc.sharpblade.net/viewLog.html?buildId=724&buildTypeId=colore_mainbuild) outcome was **SUCCESS**
Summary: Tests passed: 111 Build time: 0:1:53


I disagree, the noAudio should be proven to be false for safer handling in the program.


These should be both integer and the translation to letters should be done transparently.


using matlab to run the code


That would work as well.  I like to spell it out.


> 光照类型

@lmy441900 注意检查……


What's the use case for this? Why not provide a static method that parses a string and provides coordinates?


Why have this method and `getBarcode(int, int)`? By exposing the array, this is now mutable; is that intentional?


This isn't strings class. It's a collection of functions like memcpy, memchr, strncmp, and such that operate on C strings. Standing on the shoulders of giants, not reinventing the wheel and all that! Plus, they can operate on all sorts of objects, including void pointers to raw memory!

The difference is that it's not a class (no classes in C!) with all their overhead and extra RAM use and internal variables for bookkeeping and dubious hashes for comparison to other strings, and so forth.


I think `_VALIDATED` is undefined.


A deduplicate would only work if called...this manual stuff won't call it so would ignore it


Can we get rid of this NOQA?


Problme not resolved

 git fetch origin
remote: Counting objects: 3, done.
remote: Compressing objects: 100% (3/3), done.
remote: Total 3 (delta 0), reused 0 (delta 0), pack-reused 0
Unpacking objects: 100% (3/3), done.
From https://github.com/rafaeltuelho/jboss-eap-managed-on-docker
   3031594..a2525c8  master     -> origin/master
[jgammon612@jgammon612 jboss-eap-managed-on-docker]$ docker-compose up
Removing jbosseapmanagedondocker_dnsmasq_1...
Recreating ce1172a905_jbosseapmanagedondocker_dnsmasq_1...
Cannot start container 5d682e060d89c0526e8e6a5aa0b50a4d2cb8f5b6e773a8f2b475f44b3a0c67e7: iptables failed: iptables --wait -t nat -A DOCKER -p udp -d 172.17.42.1 --dport 53 -j DNAT --to-destination 172.17.0.6:53 ! -i docker0: iptables: No chain/target/match by that name.
 (exit status 1)
=======================================

Here is iptables:
 iptables -L -n
Chain INPUT (policy ACCEPT)
target     prot opt source               destination
ACCEPT     all  --  0.0.0.0/0            0.0.0.0/0            state RELATED,ESTABLISHED
ACCEPT     icmp --  0.0.0.0/0            0.0.0.0/0
ACCEPT     all  --  0.0.0.0/0            0.0.0.0/0
ACCEPT     tcp  --  0.0.0.0/0            0.0.0.0/0            state NEW tcp dpt:22
REJECT     all  --  0.0.0.0/0            0.0.0.0/0            reject-with icmp-host-prohibited

Chain FORWARD (policy ACCEPT)
target     prot opt source               destination
REJECT     all  --  0.0.0.0/0            0.0.0.0/0            reject-with icmp-host-prohibited

Chain OUTPUT (policy ACCEPT)
target     prot opt source               destination


Never calling set_record_owner - can this be correct?

Not to speak of update_super or onaccept (which may be unnecessary, though).


It's still epic as heck! :]


It wasn't in but will be in the next that @soltysh is doing


Haha, base-by-base comparison. Which certainly is base camp for a program like this, yes.


Again, why have this interface when `getBarcode(String)` exists, especially given this one is mutable and the other is not?


Right you are!!


Looks like Ben got .... strung out to dry ....


What does this change accomplish?


well i completely misinterpreted what `tags` were for. i've been sending all tags read from a file like this..

```javascript
{
    "listen_type": "single",
    "payload": [
        {
            "listened_at": 1444151052,
            "track_metadata": {
                "additional_info": {
                    "artist_mbids": [
                        "5167695e-3a2a-4311-9990-b7e2d13686a2"
                    ],
                    "recording_mbid": "31d07f30-b009-4838-87b9-a9d3e9015b63",
                    "release_mbid": "772857ca-fdb6-4cf6-8130-e2072e6527e2",
                    "tags": {
                        "album": "Greatest Hits",
                        "artist": "Goldie Lookin Chain",
                        "date": "2004",
                        "musicbrainz_albumid": "772857ca-fdb6-4cf6-8130-e2072e6527e2",
                        "musicbrainz_artistid": "5167695e-3a2a-4311-9990-b7e2d13686a2",
                        "musicbrainz_releasegroupid": "458bb148-7693-315a-95f4-23c985ff3fe8",
                        "musicbrainz_trackid": "31d07f30-b009-4838-87b9-a9d3e9015b63",
                        "releasetype": "Album",
                        "title": "21 Ounces",
                        "totaltracks": "13",
                        "tracknumber": "12"
                    },
                    "track_number": "12"
                },
                "artist_name": "Goldie Lookin Chain",
                "release_name": "Greatest Hits",
                "track_name": "21 Ounces"
            }
        }
    ]
}
```

i'm guessing you would prefer if i wasn't spamming your database like this?


style = row.style.append(sdata) looks very odd, are you sure?
This sets the result of row.style.append to style
Since the result is normally None then that is why you are setting style to None.
I( think what you're trying to do is:
if row.style:
    style = row.style.append(sdata)
    db(...).update(style=style)


dfgdfg


Where are the methods to get the row and column count?


I would NOT trust that - highly fragile!


This comment is a good start, but still doesn't say *what* the setup of the problem is, and *what* solution is supposed to be.  Imagine you are not the author of the test and give enough information to understand exactly what the output should be, and why. Also "trapezoidal" has an e in it, not two a's.


Dude...what was I thinking, that's a great point.  Ugh, I totally confused myself on this one because in another project for some reason I had been adding an alias for every folder inside my root source dir and that was a very bad approach.  But the alias as we had it is less fragile, as long as there aren't any packages with an `app` folder inside them.

Yeah this project is getting tough to manage... I have in mind a "meteor-webpack-react 2" based on Webpack plugins that will make it easier for users to keep up to date with our code while using whatever Webpack config they want.


This name is not intuitive. Perhaps something like `getMaximumTubeCount`?


NB style is a JSON string, so will rpopbably need to be json.loads(style) before you can append the new entry
- this does depend on the DB I think - best is to use the IS_JSONS3 validator for this as it understands the DB differences


How do I know where the errors occurred?


Maybe. I went for minimal space because it is not such an important feature.


Look into using:
https://www.npmjs.com/package/gulp-ngmin


saw approval from andytill
at https://github.com/basho/riak_core/commit/d26fcdeadd9035afeff2ecd36f4c92fab8edc2d3


merging basho/riak_core/fix/az/bump-clique-to-0.3.4 = d26fcdea into borshop-integration-787-fix/az/bump-clique-to-0.3.4


basho/riak_core/fix/az/bump-clique-to-0.3.4 = d26fcdea merged ok, testing candidate = 2adb459e


Use IS_JSONS3 both to read & also to write


I don't think any of these apply to CAP


I wouldn't delete migrations unless there's a really good reason to. The new migration caused a conflict and I had to reset the entire DB.


I think that makes sense - make sure to not break if there is no writable archive if you're adding the next checkpoint in there.


Thank you very much for noticing this.
It was such a mess.


Float like a 4.5 ... string like a 'b' ... the cache can't hit what the compiler can't C.


Not working properly


No docs, no foul. :) tags are more for folksonomy style tags -- that data you have should go under additional_info -- which is the part of the docs coming tomorrow. ;)


is it 'is'?


This interface should define the behaviour for a scanner. For instance, if I call `prepareScan` multiple times, and then call `getScan`, what should happens? If I call getScan without having `prepareScan` first, what should happen?


what's up


Are these print statements just for debugging? They can go.
Also, instead of this if-block, you can do:

    os.path.normpath(os.path.join(cwd, grm['path']))

I try to avoid direct string operations on paths, in case it makes a difference across platforms.


That generates a compile error like the following:

```
../../cpu/x86/drivers/legacy_pc/uart-16x50.c: In function ‘uart_16x50_init’:
../../cpu/x86/drivers/legacy_pc/uart-16x50.c:82:35: warning: passing argument 3 of ‘pci_init_bar0’ makes integer from pointer without a cast [-Wint-conversion]
   pci_init_bar0(c_this, pci_addr, NULL);
                                   ^
In file included from ../../cpu/x86/drivers/legacy_pc/uart-16x50.h:34:0,
                 from ../../cpu/x86/drivers/legacy_pc/uart-16x50.c:31:
../../cpu/x86/drivers/legacy_pc/pci.h:73:6: note: expected ‘uintptr_t {aka unsigned int}’ but argument is of type ‘void *’
 void pci_init_bar0(pci_driver_t *c_this,
```


Over two weeks and no-one complained CUDA build was b0rken. CUDA is dead!


This means the bundler is using traceur internally to go from es6 -> es5. This is something we want to avoid. What is preventing the bundler consuming our CJS es5 as output from TS?


Because the Mail.preventSending is already checked by the Mailer.send.


LUL
Did not see this one coming


Sure, that's fair enough.


This is the only line that differs (in my very quick check)…could be the original "sync" still be referenced somewhere instead of IslandoraSync? just an idea.


No. I do have a more complete fix on newclock5: https://github.com/popcornmix/xbmc/commit/b3ad9beabc4cd90b900a57f9b9a4931b6c20a5c1
which does fix the build errors. I've not fixed trickplay yet, so haven't submitted a PR.


Fix for https://github.com/Microsoft/TouchDevelop/commit/b2aaf351831d5a0f5827e9638cf4b62c48cec04f


file**s**
Ouch :(


`Processing` and `Complete` are in a different tense compared to the other statuses.


Ok, the renaming we had discussed during the Monday meeting.


Yeah, I was going to but it seems a bit weird to pin to a particular version when the implementation of `Range` is unlikely to change. Wouldn't it be more busywork to have to track versions within Atom?


Yes, is better use the $title var, I will update to use it.


I can merge this as is, but it would be nice to have it refactored to avoid code duplication...


Ah, ok. I did it because I thought this was a hack and wanted to just detect the specific case. Will change.


In general I like it, but see the comments I added in the commit above.

Also, in general I don't think we need the print statements for debugging left in the code. Instead, you could use `logging` so the messages can be silenced by changing the log level.


Hm, I was wrong actually. We can extract error string (I didn't remember that because it didn't say anything clear to understand that IPv6 was disabled). Will be fixed in next revision.


Does last_pos need to persist?


But why is this necessary? Why are you visiting the tree so many times?


cpu/avr/dev/lanc111.c is a similar type of driver, and it wraps these sorts of accesses in macros.  Now that I think about it, I prefer that style.  I'll revise my code.


I think this would be better as flashlight_tool. Sorry if I wasn't clear yesterday.


lgtm


This change is not needed.


`invalid strategy type; valid values ...` - don't create sentences


Yes, but that's more complex.  This simple approach actually seems to work pretty well in my tests, so my preference would be to keep this as-is for this initial pull request unless there's a compelling reason to change it.


teste


thanks for making this :+1:


i'll hold off updating until tomorrow then. thanks.


Does this mean we are disabling this test?  Easy to forget about this and end up with incomplete tests!  Please add a comment with a tracking bug to re-enable this test once the issue is fixed.


Rather, how about last_pos does persist, but as an ivec3. (block position of hit). If last_pos didn't change, don't update lightfield.


It's confusing that two of the parameters have the same value.  How about:

     lookup("fromCollection", "local", "from", "joinedOutput")


I really wish we didn't have to do this and @derekwaynecarr would help fix this


The appcast for El Capitan is the same at the moment, which is strange. But there is no need to duplicate it.


This is accompanied by the addition of the PROCINIT macro instantiation above.  These changes result in more concise code than using multiple process_start statements.


Please add a TODO to re-enable this, you can reference the same tracking bug.  (Or if you are able to re-enable this right away in the next commit, no need, of course.)


Did you really want to change this to an empty string too?


i think this sounds a bit awkward


This is P0 for me to fix -- I'm working on it right now.


> if src2 can be memory operand, using cmovnz would not save a read access to operand src2 because it is done BEFORE checking whether dest must be updated

Wow, I had absolutely no idea about that. I will update it as soon.


I'm not sure creating a second one of these is safe - why isn't this passed in with `c.KubernetesEtcdHelper`?


The appcast for El Capitan is the same at the moment. No need to duplicate it.


Hello out there.
This change is fixing encryption for old devices with old kernels;
however, such devices can run cm-12.1 pretty smooth (I've got a few), but encryption can't be used due to incompatible data format in ioctl for device mapper layer in newer versions of kernel.
Related bug report information - https://gist.github.com/ia/12d6e6ea3ba903ba923d


The X1000 includes two Ethernet MACs, so the selection of one or both of them is a platform-level consideration.


正常审计通过


NOTE: I said should be compatible with 3.0.1, I meant 2.0.1


审计通过


I have a vague memory of what your proposed fix was where you basically wanted to change how ```NewFromPlugin``` worked.  Anything to jog memory would help me put the fingers to the keyboard.


Where would you recommend this to be refactored to?


Gah I always forget this, since `break` isn't mandatory in cases in Go :(


:+1: Thank you!


req.headers.authorization is misspelled here


Скрипт, который вам нужен, находится в папке PyScripts, файл CyrillicToCode.py.


It's 0 here, not CERTCFG_ROOT_DIR ?


Added JHU 6.2 URL to Readme


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


To PRE.


I agree. But if we pass invalid code to the checks we need to mess up the code with hundreds/thousands of if-clauses because almost no assumption about the code (and maybe AST, symboldatabase, etc.) is guaranteed to hold!
So maybe we should try to catch syntax errors if the code is known to cause problems later on.


I also think that turning this into a generic service within the core of Contiki would be ideal.

I based this code on cpu/6502/net/ethernet-drv.[ch].  Thus, moving this to the Galileo platform will deviate from that precedent.  Do you still think that's a good idea?


Because the unit is what's checked, so it has to be done in different nodes, but sometimes through different nodes you get to the same unit (so a simple `return` won't work in some cases). You could have a `Java()` invocation, or just a ref to a Java decl.


- Предлагаю везде использовать по 2 пробела в качестве отступов
- Лучше, чтобы все тесты для одного класса были вместе на экране вывода при запуске, поэтому я внес все их в класс TEvolventTest


Added missing comment


Of course It's all working now.
First, I mean the coding style.
I think that an independent plugin someone can changed later without knowing that it's used in MediaPlayer. And now, instead to not work only cd (which maybe so many people are not used) will not work all plugin MediaPlayer.
So far MediaPlayer was protected from Hotplug problems but you will break it.

I wrote here just what I think. I'm not saying that it is 100% correct.
If others, including openpli team, thinks it is not right or important, then don't need change anything.


Cool, I suspected this would work but good to know for sure.


LGTM, although I did not verify any of your assertions that certain comments were fixed in future versions.


Maybe, giving a better error message than a random one from OS X might be good. @sindresorhus?


authorization is misspelled


hi, I have added the follow up question, please help me check again, thank you ~~~


Thanks for this. I was just trying to figure out why `'never'` was used instead of `'global'`. Makes sense now!


Just some another templating mechanism. I can push it to biegunka's branch if you're interested


dont want to delet moskito-control?!


always define access modifier explicitly:
private string _commandLine;

Please, start field names with single underline prefix.


Perhaps should use instead `vh`, e.g. "30vh" would be 30% of viewport height. 320px is really small on larger display.


You probably should "define" `connection` member in `__init__` method by setting it to `None` or describe it in comments somehow (that you expect it to be an instance of some class or provide you with some interface).


Please let it be so.


i added ejson after the previous chapter with "yield" not working. Afterwards, it worked fine. On this chapter after adding the spinner template, the posts are gone again even though I've already added ejson.


Is really correct though? This will make it difficult for everyone who use sqlite on envoyer


Yes, thats what I meant :).


all tests pass:
success: http://buildbot.bos1/builders/test-riak_core/builds/1768


I didn't focus on this statement when reading Haas' piece, but it's interesting to ponder. History can be forgotten and adjusted easily based on who covers it more extensively. For instance, Christopher Columbus returned to Spain and even Isabella (who led the Inquisition so was obviously not the kindest of people) thought he was a pretty evil and cruel man. However, years later, Columbus' nephew wrote a book about him that completely lionized the explorer and now we celebrate "Christopher Columbus Day". At first I thought that maybe this was getting too far removed from the argument at hand, but I think that in the context of historicizing a piece of technology it would be interesting to look at not only the history of the artifact itself, but also the history of its history. Does that make any sense? Or I am I too into my own head right now?


i would use "done" instead of over


Good plan -- I was just hoping to uncover what is unclear in the docs so far.


Is this line willingly left there?


parle plus jamais de mes commits


var commandLine =


@johncrossman: I think the `csv_staging_dir` in the `sftp_command` method:

https://github.com/ets-berkeley-edu/calcentral/blob/master/app/models/oec/publish_task.rb#L53

will now be a relative path. Will it need to have `expand_path` called on it, or is the system command smart enough to work things out?


reminder to @pweil- fix the sentence here too (in the PSP version of this validation)


new


This doesn't look backwards compatible - I don't see how an old client would post to /api/v1/securitycontextconstraints and get back a SecurityContextConstraints object?


Personally, if it doesn't make the build fail, I would leave the unused variable trigger a compilation warning, so that we remember there's still a mystery here.


fg


Hello, thanks for your reminder, I have modified it and delete the extra blank lines, thank you very much.


I have no problem submitting a PR, but your response makes me less inclined to do so, due to it's passive-aggressiveness. A simple "great idea, feel free to submit a PR" would go much further than the subtle hostility displayed above.


spurious tick


@gvanem
Please try the patch on the previous state.
I would want to know if the problem was connected to the use of typedef.
'https://github.com/fxlb/tcpdump/commit/20e79c9bb36e3907ab79e91d4b795ead208b2fc9?format=patch'


By moving it inside the if block, it helped to get rid of additional check later before deleting image.

if oldInfo != nil && !oldInfo.Initialized{
     Delete image
}

And overall understanding code is easier. If oldinfo != nil, then there are two actions. Either image is just fine or we need to delete it. And if oldinfo=nil, that means we need to setup new image.

So overall if structure simplifes a bit if we move oldinfo.Intialized check inside if.


By moving oldInfo.Intialized check inside if, I could get rid of above check.


Hi Anders,

Thanks so much for the pointer. I am busy with other stuff right now but
I'll try your config out as soon as I can.

m

On 06/10/15 04:09 AM, Anders Johansson wrote:
>
> Hi, saw your discussion in the Zotero forums and realised that we are
> aiming for the same goal. I have put up some code for my
> |org-zotxt|-configuration here:
> https://gist.github.com/andersjohansson/324a01364eb5a5435c65
> Perhaps you will find it useful? It parses link descriptions in pandoc
> citation format with |org-pdcite| to allow the features of suppressing
> authors and adding page numbers etc.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/titaniumbones/zotxt/commit/29fea8f9c4aedb0264195964101698dd865c59fe#commitcomment-13609107>.
>


If you're using envoyer, you probably shouldn't be using sqlite. The main purpose of sqlite in laravel is for development or low traffic sites. If you're using envoyer, then you don't fit that, so it's recommended that you use mysql or postgres.


s/on top of newest version/on top of the newest version/


It's just simpler - we don't need to handle all the parts of the URL separately and join them back together in the right order.


I was considering that but didn't know if that's widely supported but thinking about it it's probably not that important as devs use modern browsers.

I think the main reason was "don't clobber whole screen" which is achieved by either solution.


remove `foo=bar`


it's io_path in this function


Hi @qis.
What are your ideas about **Add thread support**?
Sqlite already supports threading (some global options when compiling there code).
Is that what you are thinking about? or are you planning something different :)


My reading has mostly been that everything involving the atomic bools is racy but that it kinda sorta works out because the maximum damage of the races are limited. I agree it is hard to reason about compared to, for example, if this function just checked `if c.claimed { // bool` while holding the lock. But I don't feel strongly enough about it to insist on it code-review wise.


impotant changes


LGTM


public class


ack


public class  .. etc


Great! :+1:


Ok, makes sense.


Should be `scope.dataB`.


Thank you!


Alright then I guess I don't have any concern beyond the one noted in the TODO. :)


Quick note - performance increases are only in cases where a large amount of processing goes into making the tooltip text; for example, if HTML is used in the Tooltip.


Ахахаха чёрт, совсем облажал с описанием, спасибо


I just want to say that you've done a really great job providing an insightful summary of Haas' piece. This is really thorough so big kudos to you.

The scroll bar wars were particularly interesting to me because of the goal of obtaining intuitiveness. Even within the members that were working towards creating a more functional and intuitive scrollbar, arguments occurred.  This is still relevant--when I first set up my macbook I could customize how I scrolled. When using someone else's computer, oftentimes scrolling feels very weird because while I mouse up to have the page move down, someone else may have it set up the opposite way. Up feels intuitive to me and down feels intuitive to them. Even when designers factor in usability and try to come together to make designs are intuitive as possible, arguments will still occur because every mind works differently.


Split up into one test per feature / binding type


Is it worth calling out that the the value we get from Open Source is part of the greater value that we get from the community?

and that we contribute back to the community. Open Source software is one big way.  But we also contribute back to the community in other ways (off the top of my mind for Stitchfix's case, we host meetups, we sponsor RubyConf, we pay for our employees to speak at conferences...etc.)


Why didn't you `setdefault(family, [])` for both AF_INET and AF_INET6 here?

This way you can explicitly specify what keys will be in the result.


what for?
just return true; - that is all for abstract operation.
And just one more moment: dont put user interaction logic (such as Console.WriteLine()) here.


Perhaps there's a reason for `[String*]`, but the other annotations use the variadic `String*`


Whatever the delicacies of forks (and I am sure that you can clone that in your docker; you may have to just remove ext/sTeX and re-clone it from your fork) using a feature branch is the better approach since you have push rights. Let's do that from now on.


But you're adding the error to the Node, not to the Unit...


LGTM


> But I'd prefer the `#pragma pack(push,1)` syntax.

I prefer something that doesn't rely on particular compiler features at all.  (Failing to compile if you try to fetch stuff directly from the packet is an additional benefit of using the new types.)


again, pl. remove Console. Write from here


This would be a bit more readable as:

```erb
<% unless no_money? %>
```


This page is really about open source software, but if you have a suggestion to replace line 18, I will gladly put it in here! :)


Awesome I'll give this a go shortly. Thanks for the quick turn around!


In addition, your happy path should be first. I would change the above to `if no_money?` and swap the order of these clauses.


iojs broke again as a result of removing this. You should bump the minor version if you're going to stop supporting iojs.


@tordans Es sind Buttons dazugekommen. Paßt Du die Google-Events an?


"This will be a non omnibus release. We will just be tagging"

Well, it showed up in Omnibus today.


May be just SumOperation.
Let it inherit from MathOperation.
Please, refactor MathBinaryOperation to MathOperation with no restriction on operands count. The only restriction: they must be a numbers


The other option would be to have a `has_money?` helper, which would probably be the best option. Then the only change would be `if has_money?` above.


I also found this really interesting. It's nice to see a human side to technology. There's a really nice article in The Economist about rebecca purple that you may be interesting in reading: http://www.economist.com/blogs/babbage/2014/06/digital-remembrance

I'm also planning on focusing on colors for my historicizing project with a sub-focus on the use of hexadecimal code.


Good call. I would move the `player_lost` and `player_won` calls to their own erb tags though:

```erb
<% if check_bust(player_hand) %>
  <% player_lost %>
  <h5>Player Busted!</h5>
```


We convert to radians anyway, so might as well.


It does not compile, there's an extra parenthesis


@JT1989 should've caught this earlier, but for future reference, this is redundant because the title is already defined in the lines above. I'll remove it.

![screen shot 2015-10-06 at 10 51 08](https://cloud.githubusercontent.com/assets/1744567/10317209/2f39e87a-6c18-11e5-8c42-2205e67968f3.png)


Thanks for the comments @glouppe . I hope all are fixed now.


1) The unit test cases for checkDate(), checkTask() and generateNextDate() have been done.
checkDate() returns a boolean value.
checkTask() returns a boolean value.
generateNextDate() returns a String object

2) assert statements and logging statements have been inserted into Flex.java, as required for the project.
(Even though at this stage, they are truly redundant.)

3) I have made Flex Display such that:

Whenever it displays error messages due to an invalid user input command, the Command Line Prompt will do the same.

1. If the user input command is valid, and it specifically requests for Tasks to be displayed (This means "searching", and "display" or "showing"), the Command Line Prompt will also display messages to notify the user of the valid input, and/or the action done.

2. If the user input command is valid, but it does NOT specifically request for Tasks to be displayed (This means "add", "delete", "change" and "undo"), the same messages will be displayed on both Command Line Prompt, and Flex Display, to notify the user of the valid input, and/or action done.

3. As far as possible, all display messages have been converted into "non-magic" Strings, for refactoring purposes, as required by the project's code quality standard(s).

4) changeTaskVariable() now makes sure that starting times are not later than ending times, and ending times are not earlier than starting times. I am allowing starting times to be the same as ending  times (This may be the case of the user attempting to indicate a "deadline" task, in my opinion.) Do take note that the validity of the new starting time or new ending time has been checked using checkTask(), by first setting the corresponding variable in the Task object to the new term/time. Any invalidity is countered by setting the corresponding  variable back to the original variable.

5. Corrected errors in changeTaskVariable(). Previously, if the variable of a Task to be changed was specified correctly, but no Tasks match the (matching or non-matching)valid date and the non-matching title provided by the user, the output would say that a change has been done, even though "show all" proves otherwise. This has been corrected with a boolean value in changeTaskVariable().

6. Some messages in changeTaskVariable() were only displayed in Command Line Prompt, but not Flex Display. This has been corrected.

7. starting times and ending times are now not allowed to have more than 23 hours, and more than 59 minutes, as acceptable time input values.


I think I can do away with visiting `QualifiedMemberOrTypeExpression` and just visit `MemberOrTypeExpression` instead of `BaseMemberOrTypeExpression`. And by doing this I can probably also remove the method for `BaseType`.


you can remove try/catch block from here (it is very expensive operation actually).

foreach (var arg in args)
{
   double number;
   if (!double.TryParse(arg, out number))
  {
     Console.WriteLine("Invalid numeric argument: {0}", arg)
return false;;
   }
NumericArguments.Add(number);
}

return true;


@junghuk Can this code be coupled with the JunitListener? The reason i bring this up is because not all instrumenters will want DB support. So might it be better to tie the DB support with the listeners which are actually producing the logs? So if a listener indeed wants to store the logs that it produces to a DB then it is the listener's responsibility to do so.


Note the allocations on lines 345 and 346. We wand to dispose of them too when changing configuration.


I think expression (or dir (projectile-project-root)) will be more idiomatic.


I think this should be here. As this for search not for indexing.


> Why are you calling ${SHELL} path/to/run.sh instead of using a shebang in the script and simply calling path/to/run.sh? If the latter was used, we could simply force the required shell in a per-use-case.

sorted-help-message.d is very special one. Generally a script used in a test case is simple and /bin/sh is enough. Therefore I assumed we don't need per-use-case shell implementation switching is not needed.

> Also, at the time you shared that script I thought about this, but forgot to comment:
> Are you sure that is worth your effort? From my (limited) experience, converting a regular expression parser into regex C parser is not that hard. Are we expecting so many conversions?

I'm thiking converting a simple parser writtien in C(= crafted parser) using only regex patterns to a *.ctags. I think such simple parser should not be written in C. In git repository we will maintain *.ctags files.
When building they are converted to *.C file.

> Also, while writing the C code the developer is forced to write test units to guarantee that the regular expressions are correctly implemented and if a converter is available a developer might think that is just a matter of generating the equivalent C code. Obviously, we would block such PR...

We require test units anyway. It is nothing to do with implementation language.

> At the moment we have 4 files in optlib, 3 of those are regex based, which aren't that many.

Yes, I wrote them. I'm very disappointed at this result. I took much time to implement optlib mechanism.

> Obviously, I will no oppose to the addition of the translator. I'm just sayin'... ;)

This is just an idea. Stabilizing tag file format has higher priority.


I see no reason against adding it by default?


@dougseven Nope. Go ahead and submit a PR if you like.


You don't need the escape for the single quote, since your string is surrounded by double quotes.


Inappropriate comment.


Function to configure $JAVA_HOME and add $JAVA_HOME to path using /usr/libexec/java_home utility.


Yeah, this is just the HIR specs and unrelated to the PPU/SPU frontends.

The specs just say that the instructions defined by `OPCODE_CMP` take two values of the same integer type, a flag that tells the comparison type (`COMPARE_EQ`, `COMPARE_NE`, `COMPARE_ULE`, `COMPARE_SGT`, etc.) and return a `TYPE_I8` value that contains `1` if the condition is true or `0` if the condition is false.


search method expects data to be a single word.


and what if NumericArguments will be of 1 length? Or even 3 - it is incorrect too. This is where outer try/cathc block comes into play.
Nevertheless, you will join this class with SummListOperation


fixed in the PSP commit


Need to check `events[i].events & EPOLLIN`


done


You should be VERY, VERY careful deleting your Gemfile.lock. Basically, you should never do it. If you do that on a production app, you are asking for trouble. Unless your `Gemfile` locks each gem at a specific version (which mine do, in my applications), all gems will be updated to the latest version. Your code is not necessarily going to be compatible. Upgrading gems should be done one at a time after reading each gem's change log. I have done this three times and once it took me two days. I found  that I couldn't upgrade to certain versions due to incompatibilities. I cannot stress this enough!


Okay. I don't really care that much about this as I easily can change it myself


Just wanted to suppress a GCC warning quickly to get to the ones that mattered. I'll remove that.


using Operation;

But please, use Operation folder only for Operation classes, not for the factory or smth else


@burke552, @maothao  I agree with your statements that it's a bit brazen of developers to compile a list of complaints about the usability of a feature and then disregard them in favor of a design that worked for them. Engineers designing for engineers and not average people is a consistent usability problem across various industries.


Unfortunately `socket.getaddrinfo` doesn't support kwargs in Python2.

What do you think about defining variables with the same (or similar) names as function's arguments' and passing them to the function like so:
```
port = None
family = socket.AF_UNSPEC
socktype = socket.SOL_TCP
records = socket.getaddrinfo(hostname, port, family, socktype)
```


#Popular data structure's implementation codes


crashlog? ^^


Sure thing. Just wanted to suppress a GCC warning in order to get the warnings that mattered.


Thank you for this. I'd forgotten.


you shoul put try/catch block here:
try
{
operation.Execute();
}
catch (Exception ex)
{
 console.Writeline(ex.Message);
operation.GetHelp();  <----- this is the last challenge: add GetHelp method to each operation
}


The main module has the main Sub for general extract plus plus it has cleaning macro and public variables.


Already fixed it in a later commit, after finally getting it to build.


well that's not the greatest thing, thanks for the heads up


I saw this less as an affirmation of the newer is not better argument and more as a "budgetary restriction" and "creatures of habit" argument. I am not averse to new technologies, but I also hesitate before applying the new facebook update or downloading the lated ios version for my phone because I'm used to the current layout and don't want to rock the boat. These updates aren't necessarily inferior to the current technology in the applications.


this doesn't work on Mono sadly and throws NotImplementedException!!!


Nice


ack


@jblaine how did you get it. I'm not seeing it on the downloads page or being served out of omnitruck


I'm glad you're interesting in digging into this topic. After Week 4, I am very interested in responsiveness and adaptivity.

This has also made we wonder if there are any sort of standards across browsers. We've mentioned the "browser wars" once or twice in class, but it seems that they're all over the place in terms of what technologies are and are not compatible. I'd like to look further into this.


it still exists under the `/api` prefix, are you looking for something specific?

https://github.com/pweil-/origin/blob/cb7ce2171d5475a5ae5d0fce80496f4b693620fb/api/swagger-spec/api-v1.json#L11409

```
[pweil@localhost origin]$ ./old-oc get psp --loglevel=6
F1006 14:01:27.406527   18157 helpers.go:95] error: no resource "psp" has been defined

[pweil@localhost origin]$ ./old-oc get scc --loglevel=6
I1006 13:58:42.334869   17923 debugging.go:120] GET https://192.168.1.139:8443/api 200 OK in 18 milliseconds
I1006 13:58:42.336982   17923 debugging.go:120] GET https://192.168.1.139:8443/api/v1/securitycontextconstraints 200 OK in 1 milliseconds
NAME         PRIV      CAPS      HOSTDIR   SELINUX     RUNASUSER
privileged   true      []        true      RunAsAny    RunAsAny
restricted   false     []        false     MustRunAs   MustRunAsRange

[pweil@localhost origin]$ ./old-oc get scc --loglevel=6 -o json
I1006 13:59:40.086394   18010 debugging.go:120] GET https://192.168.1.139:8443/api 200 OK in 17 milliseconds
I1006 13:59:40.087582   18010 debugging.go:120] GET https://192.168.1.139:8443/api 200 OK in 0 milliseconds
I1006 13:59:40.089371   18010 debugging.go:120] GET https://192.168.1.139:8443/api/v1/securitycontextconstraints 200 OK in 1 milliseconds
{
    "kind": "List",
    "apiVersion": "v1",
    "metadata": {},
    "items": [
        {
            "kind": "SecurityContextConstraints",
```


Sorry but The change I made will NOT make Mediaplarer victim off problems with hotplug issues.
 I did nothing to change the working off media player and that's really 0 nada Null nothing.
The only and ONLY changed code (first removal off the code dependent on the very old and now extreme buggy cdfs support a hugh amount off boxes could not work any more with the current patched (hacked kernel module from in other words's old age deprecated stuff.) Sorry ..... You should really study what's all about this before issuing such not founded comments


this is going to be the challenging part to test :)


You need to add `CRlibm` to `REQUIRE`.


I am very concerned about the performance of this and the remove method as well. I suspect switching the SimpleMessageGroup to use a concurrenthashmap would fix it. Will check that myself


logic is right. I dont' like the idiom.  Lemme think on this.


Whoops! Removed in 380e881bc53f0448d7ed4bffb63fee9977651f9b.


wow so big.


you can just append printf and then use sol_buffer_ensure_nul_byte()


I can't use this version because

Package polymer requires SDK version >=1.9.0 <=1.13.0-dev.3.1 but the current SDK is 1.13.0-edge.b98cd400550e100c8f63f48338a2d04c4fef4c68.


What about small comment before:
```
# (family, type, proto, canonname, sockaddr)
```


@twooten - great to see you practicing your Git skills! :tada:

We recommend [forking](https://help.github.com/articles/fork-a-repo/) the repository and creating a [branch](https://help.github.com/articles/about-branches/) to test the workflow. Use [pull requests](https://help.github.com/articles/creating-a-pull-request/) when you're ready to introduce changes to the docs - let me know if you have any questions!


I would be very interesting in reading what you'd put together should you chooe either of these topics. Gifs are obviously fun--who doens't love a good cat gif?--but there recent reemergence into popular culture would be interesting to look into.
In terms of headers, it had never occurred to me to historicize heading tags, but I think the way that you've framed it in terms of SEO puts it in the really relevant and interesting context of the business and marketing worlds.


Why not just allow an arbitrary amount of whitespace? E.g. `\\s*`.


I see it is fixed number of games versus master branch. You compare it with another  test?  Which one?


Fussiness:  maybe all of these consts will need to be a big "ReleaseDateOversampling" object.  Will make it easier to test.


Can you turn `#if defined(E_REGULATOR)` into `#if ENABLED(E_REGULATOR)` ? For some reason this is what Marlin uses.


Is this allowed for the other tags?


What are these breaking changes anyway?


TeamCity Colore :: Main Build [Build 177](http://tc.sharpblade.net/viewLog.html?buildId=725&buildTypeId=colore_mainbuild) is now running


Also, gods this must have been tedious!  Thinking through is we want multiplier or to ajust the sample some other way.  Lots of ways for this to lead to samples over '100%'


is this a doublicate fix of https://github.com/Itseez/opencv/commit/f2decec3e887dd1539e9613a6eaafe4980e25252


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


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


LGTM, aside from the fact that this is exactly what we're telling everybody else to _not_ do. :)


Actually, I think we can drop the +1


LGTM


TeamCity Colore :: Main Build [Build 177](http://tc.sharpblade.net/viewLog.html?buildId=725&buildTypeId=colore_mainbuild) outcome was **SUCCESS**
Summary: Tests passed: 111 Build time: 0:1:49


I think this might be a bit more idiosyncratic to write with maps, something like
```ruby
response.map do |brand|
  issuers = brand['issuers'].map do |issuer|
    form_issue(issuer, order, payment_method, brand)
  end
  form_payment_method(brand, order, payment_method, brand, issuers)
end
```
maybe? What do you think? Of course the above code would set issuers to `[]` in the event that there are none instead of `nil`, but I don't think that would effect anything.


It's actually funny, since forks don't exist in git, but are an entirely syntactic sugar from GitHub.

Forks are just remote branches with special conventions. All of this boils down to keeping in mind to **never merge a branch twice**, and to delete branches immediately after they get merged. If one does that consistently, it's impossible to mess up the merge flows.

(It's still possible to mess up in other ways of course, but they usually involve using ```--force```. Don't use ```--force``` unless you absolutely have to, and then still document that in as many places as possible.)


Needs comment before the `url`:
```ruby
  # amazonaws.com is the official download host per the vendor homepage
  url "https://s3.amazonaws.com/dialer_downloads/PureVPN-v#{version}.dmg"
```


Well someone has to provide the nice abstractions somehow! :)


`userstate={}, extras={}`


You're probably going to tell me to remove this `goto err3` because the program flow will continue down to the `default` case anyway because there's no `break`.  I will do so, but under protest because IMO code is easier to read if we can assume that each `case` statement is distinct.  (and I hate the fact that `break` isn't automatically implied by C)


A cor da fonte tá feia eu sei que tá, mas fazer oq?


TeamCity Colore :: Main Build [Build 178](http://tc.sharpblade.net/viewLog.html?buildId=726&buildTypeId=colore_mainbuild) is now running


Ok, just mentioning it: not calling set_record_owner makes ownership/realm-based authorization impossible.

Probably a ToDo?


hmm... I dont really like that magic constants like "summ" appears in many places.
The main challenge to you:
I want that any operation knows about its literal name:
public abstarct string Name {get;}

public overidde string Name {  get {return "summ"} }

I also want to simplyfy adding new operation to application. If I want to add a new operation, ALL THAT I NEED TO DO is:
1. Add new operation class and inherit it from base Operation (or its descendants no matter)
2. Register it to application's operation repository. Repository will contains in your factory class. You will fill it in factory's contructor:
public OperationFactory()
{
   _repository = new List<Operation>();
  _repository.Add(new SummOperation());
  _repository.Add(new SubtrationOperation());
  ....
}

As a consequence, you will no need to use switch below in factory, because you will have a Name property in each operation.


I think i have a better 'fx short version' fn than this.


Thanks @jmcarp for the pull request. To guarantee that nothing magical is happening with binaries, all the buildpacks have a `manifest.yml`, which specifies that support binaries. We do this for transparency (supported versions) and security updates (MD5 hashes).

You are free to fork, modify the `manifest.yml` and build your own buildpack with `buildpack-packager`.


This being unsynchronized means we could Heartbeat offsets out of order, right? Do we know what the rationalizer will do if it sees heartbeats in the order 1,3,2 ?


Thanks :) :+1:


TeamCity Colore :: Main Build [Build 178](http://tc.sharpblade.net/viewLog.html?buildId=726&buildTypeId=colore_mainbuild) outcome was **SUCCESS**
Summary: Tests passed: 111 Build time: 0:1:48


Might save you some refactoring later to just make this offset blob a struct.


The app itself is free, the service, however, is not. The `:gratis` is better suited here because the `license` stanza should only represent the app itself, which is free.


This the thing holding an update back, or is that something else?


Should be `crcCheckChance`


Please do, I'd like to look at what you've come up with.


O Hapi está presente em dependencies e devdependencies, em package.json!


PS. the issue is that removeMessageFromGroup does not seem to return anything useful. Why is it returning the messagegroup and not a boolean like removeMessage does? There is a different method that returns messagegroup if you want this. Its not consistant for some reason with any remove api I have seen. You need to know if the removal succeeded. Also it is removing by object instead and not by key which is awkward and requires message to correctly implement equals. Why does this class know about the internals of SimpleMessageStore at all and why is it doing an instanceof?

I think this method should just be
```java
 	private boolean removeDelayedMessageFromMessageStoreWithoutGroup(Message<?> message) {
 		if (!this.messageStore instanceof SimpleMessageStore) {
                         return ((MessageStore) this.messageStore).removeMessage(message.getHeaders().getId()) != null;
                }

```


I’ll look at each of these again.

Generally where I use extract my intention is to wipe out or populate the current set of variables with the new set.
I am trying to not use this on any field/table that is outside the scope of the eye_mag form.

The ones I removed so far were there in the first place because it was easier…  Laziness on my part.

Truthfully I am a novice so I need to spend a little time thinking about this EXTR_SKIP flag for each one.  I can probably come up with a way to create each variable programmatically/manually.

One extract line of code needs to be replaced with hundreds of lines of code.

In the past each time I looked at doing this, when I needed all the variables, I just seemed to be writing a function that is the same as the @extract function.  eg. list the fieldnames of a table in an array and iterate through processing each variable.  Is there something else besides possibly overwriting the current variable that you would like that extract is not doing, eg. text(VARIABLE)?

> On Oct 4, 2015, at 10:12 PM, Brady Miller <notifications@github.com> wrote:
>
> For all your extracts, would it make sense to use the EXTR_SKIP flag in the extract command, so don't overwrite current variables.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/bradymiller/openemr/commit/87c95fa272aa1b4169a78009b727c1fc777d662b#commitcomment-13583786>.
>


first thing, lets move this to an exported module as part of `recipe-utils` maybe?


I'm trying doxygen. I put a doxygen config con src dir and testing the output. I'll put on http://www.evilmind.es/research/sphereserverdoc/ and wait your feedback!


Seems like a good change. Can't spot anything wrong. You can merge it as far as I'm concerned.
<review


@rhymes did you mean to use `~0.9.2`? As it stands, bundler will go to a major release of faraday unattended.


This reasoning is why random.Int(0) panics but I don't actually want you to rewrite it to use topics. :)


Welcome to the project George!


This still has both the tryAquire and release inside the lock which has issues


That's a good point.

The rationalizer's logic for heartbeats is "last write wins". It has no concept of "ordering" to them. As long as the timestamp is valid, it could see 1,3,2 and it would record the "last" offset as 2. That's definitely not what we want.

The old code would do it synchronously and hold the lock, which is also suboptimal. I'll take another pass at this.


Ok, as long as you want those two different header files also sorted alphabetically, I think this PR is still good.


Sorry, should be 'It's'.


> But we already had UNALIGNED structures in other decoders without visible problem ???

`UNALIGNED` doesn't *introduce* problems, it merely fails to *remove* certain problems, namely the problem that some structures have members that, if the structure's layout is to reflect the way a particular packet type is transmitted on the network, need not to be aligned on the "natural" boundary for integral data types of the size of the member in question.  That other structures didn't have any such members, so `UNALIGNED` not doing anything on some compilers didn't cause a problem, but it *did* cause a problem for the data structures in the LISP dissector.


parentWindow was removed in approximately jsdom 1.0.0...


I don't get what you're trying to say.


Okay. Agree. Sounds a lot better actually


Well, that may make sense for the `DelayHandler`...
The main feature of the `delayer` that it is based only on single `group`...


why change this again?


@octopushugs Man thank you for this!!! I've been making a PipBoy in AngularJS and this was exactly what I needed. By the way on line 73 you need escape characters around the word tag. Now I just need to grab the images, that should be fun..


@rileyjshaw this is a little less suspicious (as suggested by @spicyj)


OK, no description what `config` is... But there's import!
Let's see what's there:
```
import json

CONFIG_PATH = '/etc/elliptics/mastermind.conf'

try:

    with open(CONFIG_PATH, 'r') as config_file:
        config = json.load(config_file)

except Exception as e:
    raise ValueError('Failed to load config file %s: %s' % (CONFIG_PATH, e))
```
Yeah, nice. No validation. No description.
I will just hope that you put these numbers in the right order.


alert and dropdown don't have options, and button has a string option which is already checked for being a string, so nothing has to be changed there.


Right, I said: it isn't finished...


Thank you, fixed.


ah, I agree that it makes sense to keep this as `MaterializedView` then, thanks :)


What is the scope of this function?


You should really start closing your entities… also what's with the name game XD


Hallo Sidey,

ich habe das Autocreate getestet. Es funktioniert nicht. Beim Autocreate kommt im log eine Fehlermeldung.

2015.10.06 19:30:45 2: autocreate: define Hideki_30_4 Hideki Hideki_30_4
2015.10.06 19:30:45 2: autocreate: define FileLog_Hideki_30_4 FileLog ./log/Hideki_30_4-%Y-%m.log
2015.10.06 19:30:45 1: define FileLog_Hideki_30_4 FileLog_Hideki_30_4 FileLog ./log/Hideki_30_4-%Y-%m.log : wrong syntax: define <name> FileLog filename regexp [readonly]
2015.10.06 19:30:45 1: ERROR: wrong syntax: define <name> FileLog filename regexp [readonly]

in fhem.cfg steht:
define autocreate autocreate
attr autocreate filelog ./log/%NAME-%Y-%m.log
# Fake FileLog entry, to access the fhem log from FHEMWEB
define Logfile FileLog ./log/fhem-%Y-%m.log fakelog

Gruß Ralf


Foi atualizado um commit acima


Should have been #233


yeah, we can :smile: thanks!


Flexibility/Ease of use for the consumer. Such static methods already exist to do the conversion. Implementations should use those so that its consumer doesn't have to worry about conversion.


Thanks... Will check!


<!DOCTYPE html>

	<title>
	Creator Information
	</title>
	<body background="lubuntu.jpg"

 The Creators

 Dhiva Krishna

 Cameron Lau-Scrum Master

 Alisha Laroia

 Sachin Shah

 Chelsea Truesdell

 Jacob Autrey

 Ethan Towner

 Jay Ivan Gentallan


Any reason why we're locking cucumber?


> I think this method should just be

The `SimpleMessageStore` and others has different internal implementations. And when we store `Message` to a `group` we don't store it to the `idToMessage`. Hence raw `removeMessage` knows nothing about that message in the group, so it always returns `false`.

Other `MessageGroupStore`s do a bit different logic and really store message in the separate location. E.g. JDBC has a structure: `INT_MESSAGE`, `INT_MESSAGE_GROUP` and `INT_GROUP_TO_MESSAGE` between them, if needed, of course.
That's why we have such a different logic here in the `DelayHandler`.
I agree with you about an inconsistency with `removeMessageFromGroup`.
But... the API is here for a while and we really can't change it for the `maint` releases.


Please use keyword args wherever possible.


Thanks! I didn't notice that. Now I know that the table defines the Headline/Title, we don't need another line of markdown to create a headline.

I created another commit to take out the word, "Testing." So disregard that if this commit will take care of that.


I think this debug statement snuck in here. :-)


TeamCity Colore :: Main Build [Build 179](http://tc.sharpblade.net/viewLog.html?buildId=727&buildTypeId=colore_mainbuild) is now running


You can add a 3..2..1 countdown or Sending hotbar popup while it's loading


when looking for how to make a select with *CMOVcc*, there were a lot of complains about them as being rather  inefficient even against *Jcc* version on modern CPU. One complain addresses the issue I told you. Another complain is why Intel did not choose the operand memory for destination instead of source: it was explained because the memory operand was inconditionnaly done. Well I do understand it is easier to do so from hardware viewpoint.


well done


@cbalit renamed


this else is unneeded since you return from the if (I thought this was linted).  Also, maybe this branch should be in its own function.    if (strategyCond) return strategyA(); strategyB()


TeamCity Colore :: Main Build [Build 179](http://tc.sharpblade.net/viewLog.html?buildId=727&buildTypeId=colore_mainbuild) outcome was **SUCCESS**
Summary: Tests passed: 111 Build time: 0:1:53


TeamCity Colore :: Main Build [Build 180](http://tc.sharpblade.net/viewLog.html?buildId=728&buildTypeId=colore_mainbuild) is now running


Can you reference the static method that does the conversion in the documentation?


This looks a lot nicer to me :+1:


@sderickson watch out for testing as an admin.


Should this line be removed? (I'm thinking it's a YAPF formatting issue?)


On Mon, 05 Oct 2015 15:46:29 -0700, Jim Huang wrote
> Why did you do the following?mov r4,
#4294967295
mov r5, #1

mov r6,
#0
mov r7,
#0

> —
> Reply to this email directly or view it on GitHub.
>
>
把previous result sum 設定初始值

我不太懂老師說的可以省略  還是說r3 r4 r5 本來就設定好是初始值了呢?  因為我是看出來r0 就是x

我是照的PPT上面的code來改寫的

--
Open WebMail Project (http://openwebmail.org)


It appears to be:

```r
> roxygen2:::parse.preref("#'      @param foo bar")
$param
$param$name
[1] "foo"

$param$description
[1] "bar"
```

Although that's not explicitly indicated by the [`LINE.DELIMITER`](https://github.com/klutometis/roxygen/blob/master/R/parse-preref.R#L22) object.


Provided for flexibility. The translation can't be transparent from every perspective. It must be done somewhere, and the best place is in the implementation, so that it is transparent to consumers


@lp-- http://tests.stockfishchess.org/tests/view/56080eff0ebc597e4f23e2e7


Should this comparison use COMP()?


Mariano estos commits son viejos, estuviste comentando en un merge que ya hicimos. Hay cosas que ya se corrigieron.


TeamCity Colore :: Main Build [Build 180](http://tc.sharpblade.net/viewLog.html?buildId=728&buildTypeId=colore_mainbuild) outcome was **SUCCESS**
Summary: Tests passed: 111 Build time: 0:1:49


Thank you for this excellent patch! There were a couple of nits (two files that might not need to be here). After those are done, then I'll merge it. :-)


TeamCity Colore :: Main Build [Build 181](http://tc.sharpblade.net/viewLog.html?buildId=729&buildTypeId=colore_mainbuild) is now running


getBarcode(int, int) is for a single position. This is to get all of the barcodes. Whether it is mutable depends on the implementation. This is what defensive copies are for


Why are the the numbers being dropped from the links?


Improve error message like :
 self.assertTrue(line.procurement_ids, "Procurement should be created")


It seems like we cannot afford to widen at low threads as expected. I guess it only begins to payoff after 8 threads(maybe higher, needed to be tested). As we will have lots of available threads, brute force prevails over efficiency.


Good point, thanks.


And Taapat ,

You really should study what was before and what is now after my patch. Nothing is broken. And using having or not will never break Mediaplayer. Seems that you really do not test and have any clue about the changes.


Single barcode vs. Collection, flexibility, and mutability depends on implementation


Ya hablamos de este archivo, fuiste vos el que lo commiteó y ya fue corregido previo al merge. :)


TeamCity Colore :: Main Build [Build 181](http://tc.sharpblade.net/viewLog.html?buildId=729&buildTypeId=colore_mainbuild) outcome was **SUCCESS**
Summary: Tests passed: 111 Build time: 0:1:47


TeamCity Colore :: Main Build [Build 182](http://tc.sharpblade.net/viewLog.html?buildId=730&buildTypeId=colore_mainbuild) is now running


They're being added....


Will change


Поясни что ты имеешь в виду под 2560, конвертирование char в int, и ты разве за границу не выходишь?


y to this email directly or view it on GitHub
<https://github.com/RFD-FHEM/RFFHEM/commit/d38f22967c0f54e57dd151c99b2b3dae6d8cb84d#commitcomment-13621585>.

Hi Ralf,
Danke. kannst Du mir mal den Dispatch String, welchen der Sduino
generiert (aus dem Log) dazu schicken? Dann kann ich das dann austesten.

Grüße Sidey


Classic. Ignore me :sweat_smile:


You have to remove `_fix` from the URL:
```ruby
  url "https://github.com/pallotron/yubiswitch/releases/download/v#{version}/yubiswitch_#{version}.dmg"
```


Occasionally, taking quantiles can result in duplicate cut points. If this is the case, use the unique() function to drop any duplicate values before discretizing.


@gl2748 can you push this to the https://29-e3teachers-3mhev6qb5ihtc.us.platform.sh branch WITHOUT affecting the changes that I have made? This would be helpful in finalizing what I would like to see committed to code and pushed live tomorrow.


Yes it seems a bit extreme. But maybe i´m not the best to maintain the repository. At least not alone i´m sure there are people using this more than me. Your for example :D An organization would allow you or others to maintain it. e.g. doing merges


@cbalit et al. Should these dependencies be added to google-recaptcha's bower.json like this? `iron-form` should only be necessary for the demo.

As for `iron-validatable-behavior` and `iron-form-element-behavior`, do you want to keep those out of a pure google-recaptcha implementation, and instead add them to a separate, iron-form-specific implementation? (See [here](https://github.com/GoogleWebComponents/google-recaptcha/pull/12#issue-109702171) 2nd ¶)


TeamCity Colore :: Main Build [Build 182](http://tc.sharpblade.net/viewLog.html?buildId=730&buildTypeId=colore_mainbuild) outcome was **SUCCESS**
Summary: Tests passed: 111 Build time: 0:1:49


Первая проблема - это глобальная функция. Она одна и делает всю работу. То есть, по сути мы имеем ситуацию где код написан сверху вниз одним куском. Это плохо, потому что каждое новое изменение повлечет за собой изменение этой функции, она будет расти и её будет сложно поддерживать и расширять.
Нужно провести декомпозицию - разделение на части. Например: обработчик добавления тудушки, обработчик удаления тудушки, создание html строки и т.д.


Why this? Maybe the problem is "4.0" instead of "4".


Quite right, thanks Victor!


creo deberiamos tener una sola const struct y por ende este array no deberia ser necesario, no?


This commit LGTM.


Why do you define `IpAddressesCacheItem` later (the same question about other `*CacheItem` classes)?
If there's no reason, what about defining all dependencies (imports, functions, classes, etc.) before?


I was trying the above command and it seemingly doesn't work:

    avocado run --loaders file:?
    No tests found for given urls, try 'avocado list -V' for details


dvb? You mean deb right?


@grimbald nie powinieneś wrzucać zahardkodowanych ścieżek jak "C:/WorkspaceMars/Library/JUnit/junit-4.12.jar" przez to po imporcie miałem błędy, proponuję przejść na Mavena - tam podaje się z jakich bibliotek się korzysta a mvn podczas builda sam je sciąga z zewnętrznych repozytoriów


You may be right, the problem is we don't know how much to increase/decrease futility movecounts/reductions and other pruning logic. I just made a rough attempt at 2 threads because I don't want to use up resources at 7 threads and above.


input - глобальная переменная. не забывай var


plik  .settings/org.eclipse.jdt.core.prefs  też mołeś do gitignora dodać


wcięcia powinny być standardowo ustawione na 2 spacje


Add `RUN ruby-switch --set ruby2.1` so you can use `gem install` etc.


one nasty bug with *operator ? :* using *CMOVxx* is that having a C code like: `return ptr ? *ptr : 0;` will lead to an exception when ptr is 0. See the asm code:
```
xor eax, eax
test rdx,rdx
cmovnz rax, qword ptr [rdx]
```
guess what happens...


When we update to OpenSHMEM 1.3, can we delete the bounce buffer, frag, and total ordering code?  This stuff should all be unnecessary, and we'll have nonblocking communication in 1.3, so users who ask for blocking should get what they asked for.


En het werkte :smile:  :+1:


@web2solutions, this is much better.

From https://github.com/web2solutions/Veritaseum-proof-of-concept/pull/1#issuecomment-145652737:
>  Note:

> Now, the close method from resource will attempt to delete NAMESPACE._all_ids[_id] 2 times when you call

> ```
resource.close();
resource2.close();
```

> It does not make sense until you have 2 resources with different IDs

Ah, now there's a different problem. What if one code path is still using `resource2` after another code path calls `resource.close()`? Consider the following:

```javascript
QUnit.test( "resource is closed at the appropriate time", function( assert ) {
    var rsrc1 = NAMESPACE.resource(42);
    var rsrc2 = NAMESPACE.resource(42);
    rsrc1.close();
    var rsrc3 = NAMESPACE.resource(42);
    assert.ok( rsrc2.getExpensiveResource() === rsrc3.getExpensiveResource(), "unnecessarily reallocating resource(42)" );
    rsrc2.close();
    rsrc3.close();
    var rsrc4 = NAMESPACE.resource(42);
    assert.ok( rsrc4.getExpensiveResource() !== rsrc1.getExpensiveResource(), "unnecessarily keeping resource(42) alive" );
});
```

From https://github.com/web2solutions/Veritaseum-proof-of-concept/pull/1#issuecomment-145653117:

> Shouldn't the module to handle the passed id when you call NAMESPACE.resource and avoid multiple equal IDs?

Is this always the case? What if two higher level modules that don't know about each other are each using the `NAMESPACE` entity to access similar expensive resources? For example, what if the resource was a database connection, and two different modules were using the connection to create its own cursors and perform its own queries? Should those modules have to coordinate with each other if they want to share a single connection?

Also, there are a few unanswered questions.

From https://github.com/web2solutions/Veritaseum-proof-of-concept/commit/e6cb8cc8e6dff9c22c65dd3468a76731f13fd842#commitcomment-13580894:

> > No, it does not require browser functionality ... it will require only if you use window to declare globals

> How would I run the tests without using a browser or phantom?

From https://github.com/web2solutions/Veritaseum-proof-of-concept/commit/e6cb8cc8e6dff9c22c65dd3468a76731f13fd842#commitcomment-13581141:

> > > Can you think of a way to to provide the intended functionality of `_all_ids`, but without exposing it to the global namespace?

> > No. no functionality

> I'm not sure I understand. Are you suggesting there's no way to hide shared state from the global namespace, or are you suggesting that in this case, it is unnecessary because you've eliminated `_all_ids`?